-
Notifications
You must be signed in to change notification settings - Fork 860
simple solution for basic gallery styles #1228
Comments
Might be a good thing to add to FP by default. Could you please take some before/after screenshots? thanks |
Sure. Still looks a bit rough, but here you go.
} .gallery-icon { |
The next challenge is to figure out how to add modals. Not sure if it's worth the trouble though. |
Thanks @AmyGail. You can make modals with the Reveal plugin for Foundation. |
You bet. |
@AmyGail I'll add the help wanted tag and maybe somebody will help add this feature to FP. |
In my opinion the Reveal plugin isn't a good option for image modals. With other plugins we can get best results and features like responsive size, next/prev and others things. I think the best is implement a basic solution to fix CSS and flexibilize the use of the other modal like the Venobox (http://veno.es/venobox/). I can add a PHP function to change default Wordpress gallery markup with Foundation classes and html atributes for flexibilize usage of other plugins. What you think? |
I agree that Reveal is not the way to go for this goal, but in general I think we should leave image/gallery-modals to third party plugins (for example: https://wordpress.org/plugins/w3dev-fancybox/). Imho this kind of functionality should not depend on theme functionality anyway. |
I was just thinking of having Reveal pop-up the full size image and have no other functionality. I like @emaia's idea to make the markup Foundation compatible so it can be used in 3rd party plugins. |
Yes; that's certainly better then the standard image-link (like WP does it). Still Venobox looks insteresting too :-) |
Emaia & Colin-Marshall’s ideas sound good to me as well. |
I'm working on a site and noticed that WP galleries are unstyled.
There's probably a more elegant solution - but on the off chance anyone else is dealing with this
here's what's working for me.
https://foundation.zurb.com/sites/docs/xy-grid.html#xy-grid-layout
The text was updated successfully, but these errors were encountered: