Skip to content

Add enlarge images button#248

Merged
dgarcia360 merged 2 commits intomasterfrom
enlarge-image
Nov 19, 2021
Merged

Add enlarge images button#248
dgarcia360 merged 2 commits intomasterfrom
enlarge-image

Conversation

@dgarcia360
Copy link
Copy Markdown
Collaborator

@dgarcia360 dgarcia360 commented Nov 17, 2021

Closes #187

How to test this PR

  1. Clone this PR. For more information, see Cloning pull requests locally.

  2. Enter the docs folder, and run:

make preview
  1. Open http://127.0.0.1:5500/ with your favorite browser. Then, navigate to "Examples" > "Images". At the end of the page, you will find an example on how to use the custom class "enlarge-image".

Screenshots

image

image

@dgarcia360 dgarcia360 requested a review from tzach November 17, 2021 07:49
@tzach
Copy link
Copy Markdown
Collaborator

tzach commented Nov 17, 2021

@tzach
Copy link
Copy Markdown
Collaborator

tzach commented Nov 17, 2021

Do we need to add directive to each image to support this?

@tzach
Copy link
Copy Markdown
Collaborator

tzach commented Nov 17, 2021

WARNING: error while formatting arguments for sphinx_scylladb_theme.extensions.substitutions.substitution_code_role: type object 'list' has no attribute '_special'

@tzach
Copy link
Copy Markdown
Collaborator

tzach commented Nov 17, 2021

image

@tzach
Copy link
Copy Markdown
Collaborator

tzach commented Nov 17, 2021

Make this option default for all images with width

@tzach
Copy link
Copy Markdown
Collaborator

tzach commented Nov 17, 2021

@dgarcia360
Copy link
Copy Markdown
Collaborator Author

@tzach I've updated the PR with a new proposal. You can test it pulling the latest commit.

If it does not work as expected, please remove the cache or open the documentation site in incognito mode.

@tzach
Copy link
Copy Markdown
Collaborator

tzach commented Nov 18, 2021

LGTM

@dgarcia360 dgarcia360 merged commit c0a0166 into master Nov 19, 2021
@dgarcia360 dgarcia360 deleted the enlarge-image branch November 19, 2021 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enlarge content images

2 participants