-
Task
-
Resolution: Merged
-
Should have
-
None
-
None
-
3
-
Block: Carousel Header
-
Sprint #160, Sprint #161
-
rhea
Test page: https://www-dev.greenpeace.org/test-rhea/carousel-header-new-wysiwyg/
Feedback by aradu
Frontend
- on desktop I think it works pretty much as intended, I haven't spot any problem
- on mobile, there are a couple of bugs
- on scroll, it blocks the page
- i am not really able to change to the next slide
- the description and title are not visible
- the selectors indicating on which slide we are are very low on the page, there is a big space between the image and those selector (I'll add ss)
Backend
- there are some challenges with the display - probably some of these won't even matter when the carousel block is the first one loaded on a page, but I still wanted to have it documented (I'll add a video for this on drive and add the link for it -> https://drive.google.com/file/d/1_4aETlgEyaVJpFhPaE51NR6D9ntJyKNh/view)
- not able to see the menu b/c of the block above
- maybe the arrow from description can be moved so it does not overlap with the field
- the space where the editor needs to click to write the CTA is very small - different from I get is the case for the description, where the space is bigger - it is a bit confusing and frustrating for editors, I guess if we make the behaviors consistent we could make it easier for editors
- has to be done before
-
PLANET-6120 Release Carousel Header WYSIWYG block
- CLOSED
- relates to
-
PLANET-4809 Convert Carousel Header block to WYSIWYG (Beta)
- CLOSED