Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding a few parameters to customize the gallery and adding the option to display the first image as a larger title image #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zuzi-m
Copy link

@zuzi-m zuzi-m commented Dec 4, 2022

Added a few parameters to the GalleryImage widget to be able to customize things that are currently hard-coded:

  • customizing the grid spacing and column count (gridHorizontalSpacing, gridVerticalSpacing, gridColumns)
  • customizing the border radius of gallery item thumbnails (thumbnailBorderRadius )
    The above parameters are optional and their default values are set to reasonable values similar to what it was hard-coded to originally (the only exception is gridVerticalSpacing which was originally 0 but now it defaults to 5 to make the gallery look pretty when there are multiple rows of thumbnails)

Also added a parameter showTitleImage that, when set to true, makes the gallery show the first image as a larger single thumbnail on the top, whith the rest of the gallery being as usual in a grid below that. Additional optional parameters titleImageWidth and titleImageHeight can be set to control the size of the title thumbnail.

…n to display the first image as a larger title image
@nisargsuthar
Copy link

@BeshoyMelika This is a great enhancement to the existing package, kindly merge it for ease of use!

@zuzi-m
Copy link
Author

zuzi-m commented Jun 17, 2023

@BeshoyMelika any updates on this? Could you please review and consider merging? Thanks

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.

2 participants