Skip to content
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

CDPT-636: Update heading styles in the rich text component #789

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -150,39 +150,24 @@ img.aligncenter { display: block; margin-left: auto; margin-right: auto; }
border: 1px solid $border;
}

h2,
h3,
h4,
h5,
h6 {
ff-title();
margin-bottom: $spacing;
font-weight: 700;
}

h1 {
ff-title()
font-size: $crust;
h1();
}

h2 {
font-size: $mantle;
margin-top: 50px;
h2();
}

h3 {
font-size: 24px;
line-height: 30px;
h3();
}

h4 {
font-size: 19px;
line-height: 25px;
h4();
}

h5 {
font-size: 16px;
line-height: 20px;
h5();
}

h6 {
Expand Down
69 changes: 69 additions & 0 deletions public/app/themes/clarity/src/globals/css/_mixins.styl
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,75 @@ ff-body() {
margin-bottom: $bodyCopy;
}

// Heading styles for c-rich-text component and the o-page--title object, adapted from https://design-system.service.gov.uk/styles/type-scale/
h1() {
ff-title();
font-size: 3.2rem;
line-height: 1.1875;
margin-bottom: 2.5rem;
@media screen and (min-width: 640px) {
font-size: 4.8rem;
line-height: 1.125;
margin-bottom: 3.125rem;
}
}

h2() {
ff-title();
font-size: 2.7rem;
line-height: 1.22222;
margin-top: 3rem;
margin-bottom: 1.25rem;
@media screen and (min-width: 640px) {
font-size: 3.6rem;
line-height: 1.16667;
margin-bottom: 1.875rem;
}
}

h3() {
ff-title();
font-size: 2.2rem;
line-height: 1.31818;
margin-top: 3rem;
margin-bottom: 1rem;
@media screen and (min-width: 640px) {
font-size: 2.6rem;
line-height: 1.25;
margin-bottom: 1.25rem;
}
}

h4() {
ff-title();
font-size: 1.9rem;
line-height: 1.42105;
margin-top: 3rem;
margin-bottom: 1rem;
@media screen and (min-width: 640px) {
font-size: 2.2rem;
line-height: 1.36364;
margin-bottom: 1.25rem;
}
}

h5() {
ff-title();
font-size: 1.6rem;
line-height: 1.5;
margin-top: 3rem;
margin-bottom: 1rem;
@media screen and (min-width: 640px) {
font-size: 1.9rem;
line-height: 1.47368;
margin-bottom: 1.25rem;
}
}

body() {
ff-body();
}

// For a real content list with bullet points
real-list() {
ff-body();
Expand Down
4 changes: 2 additions & 2 deletions public/app/themes/clarity/src/globals/css/objects.styl
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
margin-bottom: $spacing*2;
// Main page title
&--page {
font-size: $crust;
h1();
margin-top: $spacing*2;
}
// Section title
Expand All @@ -22,7 +22,7 @@
}

&--headline {
font-size: $crust;
h1();
}

&--byline {
Expand Down
5 changes: 5 additions & 0 deletions public/app/themes/clarity/src/globals/css/reset.styl
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,11 @@ select::-ms-expand {
display: none;
} // Remove default drop down style in IE
html {
/* This is an old trick to make rems map to pixel values.
* The browser default is 16px, 62.5% sets it to 10px so 1.6 rem is equivalent to 16px.
* See this article for a better explanation: https://www.aleksandrhovhannisyan.com/blog/62-5-percent-font-size-trick/
* TODO: remove this in future as it's confusing and doesn't integrate well
*/
// Reset font size to 10px
font-size: 62.5%;
}
Expand Down
Loading