New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update K5 PL marketing page #57904
Update K5 PL marketing page #57904
Conversation
…l-development-workshops
these will get added when I merge in another PR
@@ -23,6 +23,7 @@ theme: responsive_full_width | |||
%li=hoc_s(:pl_middle_high_top_list_01) | |||
%li=hoc_s(:pl_middle_high_top_list_02) | |||
%li=hoc_s(:pl_middle_high_top_list_03) | |||
%li=hoc_s("pl_page_k5.top_list_04") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dmcavoy added this re: this Figma comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here's the updated map styles including responsive:
Screen.Recording.2024-04-10.at.1.13.47.PM.mov
@@ -77,10 +77,10 @@ section.bg-pattern-dark { | |||
|
|||
// Adds 2rem of space above or below an element | |||
.add-margin-top { | |||
margin-top: 2rem; | |||
margin-top: 2rem !important; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you add a comment as to why !important
is needed here?
Also, more broadly, sorry for being a stickler here -- as you know, this is an anti-pattern and I'd like to understand why we need to do this. I completely believe that it's needed, just want to know why :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes definitely! I'll do it in a followup, but the short answer is that this can be used as a utility class that will override existing margin-top on any element.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One ask for a comment -- feel free to follow-up if you want to get this in sooner!
Updates the K-5 Professional Learning marketing page https://code.org/professional-learning/elementary
Related PRs:
Jira ticket: ACQ-1412