PECE-project / drupal-pece

PECE development repository
GNU General Public License v3.0
21 stars 28 forks source link

Replace owlcarousel for carousel slides (or not) #345

Open MegaKeegMan opened 9 months ago

MegaKeegMan commented 9 months ago

See https://github.com/OwlCarousel2/OwlCarousel2 "YEAH SO THIS IS PRETTY MUCH DEAD, DO YOURSELF A FAVOR AND SWITCH TO tiny-slider"

While we could switch to tiny slider, as recommended by the developers of owlcarousel, we should ideally avoid custom integration of js, and even more ideally avoid use of carousels at all. This can still be done, but noteworthy that carousels are often recommended against, despite their popularity.

Screenshot_101

mikefortun commented 9 months ago

I'm all for avoiding customization when possible, but this may be a case where we will have to settle for the less-than-ideal and go with tiny-slider. The front page sliders have been popular because they've been a good and easy way for the Little Admins of each instance to add quick portals from the front page to new projects, and to provide a good sense of the many different things that are going on. If there are other ways to do similar things without depending on the custom integration of carousels then that makes it easy, but I suspect not. I can also poll the various instance admins to see how they would feel about losing the front page sliders. Maybe we can defer a decision on this for a while?

MegaKeegMan commented 9 months ago

Thanks for sharing the value this feature provides. I don't think there is any rush to make a decision on this. If there are any non-custom alternatives, we will document them here.

MegaKeegMan commented 5 months ago

Does the carousel need to autoplay?

mikefortun commented 5 months ago

isn't that kind of the definition of carousel? Or are you talking about a kind of slider that would only slide if you click of hover over the arrows on either side -- a Netflixxy-kinda thing?

mikefortun commented 5 months ago

so I guess the question is, is there anything in between carousel and no carousel? and/or can you say more now about why carousels are recommended against? because they are customized and therefore subject to becoming unsupported like owlcarousel has?

MegaKeegMan commented 2 months ago

A cool module that allows animating paragraphs withiin a container paragraph https://www.drupal.org/project/drowl_paragraphs

Note that this module requires a theme based on Zurb Foundation 6.x, which was in the top 3 considerations for theme

MegaKeegMan commented 1 month ago

The slides are coming along. I was looking at the image sizing so I can get into actually styling the images here, and it looks like all of the images that I see are 960 x 230. These dimensions make sense on desktop, but not so much on mobile. I know there is not a lot of emphasis on mobile for this platform, which makes sense for people who are using it to perform research and collaborate, but it makes less sense for the visitor who heard about the platform at a conference and wants to check it out—The text on the slides cannot be read on mobile.

While there probably won't be very many of them per site, given that they will be the first thing a visitor sees when they land on the home page, maybe the words in the image should be legible on mobile. I am recommending that perhaps we actually have 2 image fields for slides, one for large and one for small screens.

Screenshot_%T_%d-4

MegaKeegMan commented 1 month ago

Decided in chat on the 2 image option