carbon icon indicating copy to clipboard operation
carbon copied to clipboard

Typography style 'Segment Sub-header' line height issue

Open graham-blake opened this issue 2 years ago • 1 comments
trafficstars

Current behaviour

The typography style 'Segment Sub-header' is 16px medium (700) weight text and has 31px line height. Is the line height an oversight or is there a particular reason for the 31px? typography-segment-subheader-carbon

Expected behaviour

The Design System (via Figma) defines the line height to be 24px, which looks about right. typography-segment-subheader-design-system

CodeSandbox or Storybook URL

https://carbon.sage.com/?path=/docs/typography--variants-story

JIRA Ticket (Sage Only)

No response

Suggested Solution

Note from Daniel Dipper in the Slack Carbon channel:

It looks like the line height value we have is the same value we were given in 2020 when Typography was first created. We followed these specs here. I believe all we require is an updated spec from the DS, so that we can update our component with the new values Typography requires.

Carbon Version

Latest

Design Tokens Version

N/A

What browsers are you seeing the problem on?

Chrome

What Operating System are you seeing the problem on?

MacOS

Anything else we should know?

I'm hoping this is an oversight but there may have been a specific reason for the 31px line-height in a particular case in a particular product.

Confidentiality

  • [X] I confirm there is no confidential or commercially sensitive information included.

graham-blake avatar Sep 29 '23 08:09 graham-blake

FE-6232

robinzigmond avatar Oct 03 '23 13:10 robinzigmond