Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#55788 closed defect (bug) (reported-upstream)

Changing the font size to larger font size breaks the layout in sidebar settings panel

Reported by: juhise's profile juhise Owned by:
Milestone: Priority: normal
Severity: normal Version: 5.9
Component: Editor Keywords:
Focuses: ui, administration Cc:

Description

If the font size is set to 90 px and someone switches it to em or rem it breaks the layout. The font size is of course too big in em and rem but maybe we can either set the limits and can give an alert/message to enter font size is given limit to the user or doesn't reflect these changes for preview.

Steps to reproduce

  1. Click on Site Editor
  2. Click on Styles
  3. Click on Typography
  4. Click on Text
  5. Use Custom Font Size
  6. Enter the font size 90 in px
  7. Now select rem

It breaks the layout

Let me know if this also should be reported on Gutenberg Repo.

Attachments (1)

font-break-layout.gif (898.4 KB) - added by juhise 2 years ago.

Download all attachments as: .zip

Change History (5)

#1 @costdev
2 years ago

  • Keywords reporter-feedback added
  • Milestone Awaiting Review deleted
  • Version changed from trunk to 5.9

Hi @juhise, thanks for this report!

If you could report this on the Gutenberg repository that would be great! In general, all Blocks/Block Editor/Site Editor issues should be reported on the Gutenberg repository, then the fixes can be brought WordPress Core.

Also, a note for your report: This also occurs on WordPress 5.9.0, with or without the Gutenberg plugin installed.

Can you drop a link to the Gutenberg issue here once it's created and we'll close this ticket as reported-upstream? Thanks!

Last edited 2 years ago by costdev (previous) (diff)

#2 @juhise
2 years ago

@costdev
Thanks for your prompt response.

Added here https://github.com/WordPress/gutenberg/issues/41233

#3 @costdev
2 years ago

  • Resolution set to reported-upstream
  • Status changed from new to closed

Thanks for opening the issue and dropping the link here @juhise!

Closing as reported-upstream.

#4 @costdev
2 years ago

  • Keywords reporter-feedback removed
Note: See TracTickets for help on using tickets.