components icon indicating copy to clipboard operation
components copied to clipboard

Improve line height: dialog component

Open ftakuipers opened this issue 9 months ago โ€ข 1 comments

Provide a general summary of the issue here

The heading text is cut off in a strange way, visually it seems like there should be enough space to place the word 'blokkeren' before it jumps to the next line. In Figma this is the case. Not sure where the issue lies, but this leads to discrepancies between designs and development.

Screenshot developed dialog: Image

Screenshot same Figma dialog (also width 400): Image

๐Ÿค” Expected Behavior?

Same behaviour in both Figma and storybook component

๐Ÿ˜ฏ Current Behavior

Not the same behaviour

๐Ÿ’ Possible Solution

Make it the same behaviour :)

๐Ÿ–ฅ๏ธ Steps to Reproduce

In Magister zentest environment, go to Cijfers - (2nd) Beheer tab - Select a course (e.g. Nederlands) from a study (e.g. L BRUG 1) and select two columns via bulk select - Click 'Blokkeren' bulk action

What browsers are you seeing the problem on? (only for bugs in code)

Other

If other, please specify.

No response

What operating system are you using?

MacOS

๐Ÿ‘ค Your name

Floor Kuipers

๐Ÿงข Your product/team

Magister

๐ŸŽจ Your Theme(s)

Magister

  • [ ] https://developer.chrome.com/blog/css-text-wrap-pretty
  • [ ] use CSS text-wrap: pretty

ftakuipers avatar Mar 28 '25 09:03 ftakuipers

@ftakuipers we used CSS text-wrap: balance to prefend an orpahn on the second line, but I agree we can improve the line-height here.

DanielleRameau avatar Mar 31 '25 07:03 DanielleRameau

problem is solved within the new figma 2.0 component.

DanielleRameau avatar Aug 28 '25 13:08 DanielleRameau