Implement CTA carousel for the footer.

RESOLVED FIXED

Status

Webmaker
webmaker.org
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cassiemc, Assigned: daleee)

Tracking

Details

(Whiteboard: frontend)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Created attachment 751149 [details]
footer CTAs carousel

* CTAs need the left and right arrows implemented as part of a carousel. 
* Carousel arrows should be stuck to left and right hand sides of the browser window.
(Reporter)

Updated

5 years ago
Blocks: 869204
(Assignee)

Comment 1

5 years ago
I was just told by Beltzner that because we only have 4 CTA, this isn't happening? Can I get a confirm/deny on this? It's the last thing I have to do for the footer fixes so I'd like to know if I can move on & open a PR for this stuff.
Flags: needinfo?(mbeltzner)
Flags: needinfo?(cassie)
Flags: needinfo?(brett)
I don't actually know if there is a list of CTAs for the footer carousel. If there are only four, doesn't make sense to have scroll arrows. If there are more than four, kinda does, though I doubt anyone will click 'em.

Cassie: do you ever think we ought to have more than 4 CTAs?

Dale: what you can do, if you want, is implement the carousel, with each CTA being described as a JSON object or other data structure that can be edited. Make up a 5th for now, or repeat them. Then have the code say "if there are more than 4 items in this structure, show the arrows; otherwise, do not." Sensible?
Flags: needinfo?(mbeltzner)
(Assignee)

Comment 3

5 years ago
Thanks Mike, that helps.
Flags: needinfo?(cassie)
Flags: needinfo?(brett)
(Assignee)

Comment 4

5 years ago
Implemented
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.