Update font definitions to remove font-weight: light

[gfellerph]

Font definitions have been updated and the light font variant has been removed for most use cases.

Design: https://www.figma.com/file/ojCcgC5Zd12eUSzq6V5m24/Foundations-Post?type=design&node-id=3%3A950&mode=design&t=mfN0rhDtnbsAvzHT-1

Tasks

Preview Give feedback

1. Update all text except subtitle to regular
2. Update typography documentation
3. Design review

[oliverschuerch]

There is still a light font-face in the figma file. Clearification wanted!
@rouvenpost can you please have a look.

[gfellerph]

@oliverschuerch the change is scoped to body copy. Subtitles still use light.

[gfellerph]

@oliverschuerch can you check if font styles are updated to regular and close this ticket if they are?

[oliverschuerch]

Font styles are not updated now.
Selectors/Variables that still use $font-weight-light, but shouldn't:

• header.navbar .search-textbox input (file: components/intranet-header/_searchbox.scss)
• #navbarsDefault > ul > .nav-item (file: components/intranet-header/_top-navigation.scss)
• @mixin styles { p } (file: layouts/portal/_type.scss)
• %list-adjustment { ul li } (file: placeholders/_text.scss)
• %text-container { ul li } (file: placeholders/_text.scss)
• %text-container { ol li } (file: placeholders/_text.scss)
• $blockquote-cite-font-weight (file: variables/_type.scss)
• $accordion-body-font-weight (file: variables/components/_accordion.scss)
• $notification-font-weight (file: variables/components/_notification.scss)

[oliverschuerch]

@gfellerph The .carousel-caption .light class is used like a subtitle and the whole selector looks like this:

.carousel-caption {
  .subtitle,
  .light {
    font-size: type.$font-size-small-big;
    font-weight: type.$font-weight-light;
  }
}

Should we keep this?

[oliverschuerch]

@gfellerph What should be updated in the typography documentation?
Do you just want to remove the light font face from the docs page?

[gfellerph]

@gfellerph The .carousel-caption .light class is used like a subtitle and the whole selector looks like this:

.carousel-caption {
  .subtitle,
  .light {
    font-size: type.$font-size-small-big;
    font-weight: type.$font-weight-light;
  }
}

Should we keep this?

The carousel is deprecated and will be removed soon, no need to do anything right now.

[gfellerph]

@gfellerph What should be updated in the typography documentation?
Do you just want to remove the light font face from the docs page?

As the font is still available and used, I'll keep this as is. Might change with the new font and v2 anyway.