Closed flukeout closed 8 years ago
@gvn - Let me know if this makes sense and reflects what we talked about earlier today.
Sounds like @gvn has already done some mobile styling for this component - let us know when it hits the mofo-ui demo page and we can take a look there and provide feedback as needed.
Edit - a demo of the carousel is available here https://gvn.github.io/mofo-ui/demo/
I think it works pretty well in a narrow format with 4 photos, so I think we're in the clear until we need to add more photos.
@flukeout I made a new branch with a 3 tab scaffold for you to populate.
It's here: https://github.com/mozilla/learning.mozilla.org/tree/clubs-big-kahuna
We should probably meet quickly to discuss, but this should get you started!
Thanks @gvn - I'll give you a ping for a walkthrough before stand tomorrow sometime.
tagging @hannahkane to get updates
Feature requests & questions for @gvn
Can we
/clubs
for the first tab/clubs/start
for the second tab/clubs/running
for third tabLet's do the deep linking as a follow up once this is landed.
Need link for, don't see it in the content doc
Oh wow. I totally dropped the ball on that. The plan for that page was originally to build it with Calypso. Now that that's not going to happen, I'll file a follow-up ticket to build that page. In the meantime, I'd suggest we drop that sentence.
Will do, thanks!
I opened an issue on Learning for the carousel integration. I don't think it's a mofo-ui issue since the carousel works fine on Science. It seems to be a problem with how Learning is compiled with Webpack, but I'm having a hard time nailing it down.
@flukeout
Border radius for tabs - It should be similar to all buttonborder-radius on the learning site border-radius: 4px;
Link colour - On the learning site link colour is #337AB7
Footer - Yes, we do not need to introduce the dark footer and continue to use the existing side nav footer.
Thanks @sabrinang - I made the changes.
@sabrinang Since our mofo-ui component needs icons for the narrow versions of the tabs, I've decided to use the following...
Let me know if you'd like to change any of these.
@gvn Okay - I think this is READY! and my branch has all of the latest. What's the best course of action?
Ze Branchè - https://github.com/flukeout/learning.mozilla.org/tree/clubs-big-kahuna
Sweet! Just make a PR to the master
branch of the mozilla
repo and flag someone (probably me?) for a review.
Closing as PR is now merged. Will file follow-up issue to QA on staging.
What's left to do
Left before Launch
Big Stuff
Styling & Tweaks
Follow up / not critical
Some handy links
Glossary