WordPress.org

Make WordPress Core

Opened 5 weeks ago

Last modified 8 days ago

#48970 accepted defect (bug)

Twenty Twenty : Menu rendering problem Unicode (Bangla)

Reported by: smallprogrammers Owned by: audrasjb
Milestone: 5.4 Priority: normal
Severity: normal Version: 5.3
Component: Bundled Theme Keywords: has-patch
Focuses: css Cc:
PR Number:

Description

Hello i found a bug for menu rendering in Bangla language.
menu got broken and also text got broken if i write this : সাহায্য (it means "Help").

It works fine with Twenty nineteen theme.

Here is my forum post : https://wordpress.org/support/topic/unicode-rendering-problem-menu/#post-12236743

My website is : blog.aparajeyo.com

Thanks in advanced.

Attachments (5)

Capture d’écran 2019-12-16 à 09.52.02.png (43.7 KB) - added by audrasjb 5 weeks ago.
can't reproduce the issue on Mac OSX / Chrome
48970.diff (410 bytes) - added by Shital Patel 5 weeks ago.
i just removed word-break: break-word; word-wrap: break-word; in *, ::before, ::after { } style.css
Untitled.png (22.7 KB) - added by Shital Patel 5 weeks ago.
Before : Screenshot of windows PC firefox browser
Untitled1.png (55.3 KB) - added by Shital Patel 5 weeks ago.
After : Screenshot of windows PC firefox browser
48970.1.diff (1.2 KB) - added by audrasjb 2 weeks ago.
Reset word-break and word-wrap properties for Twenty Twenty primary and footer navigation menus

Download all attachments as: .zip

Change History (17)

#1 @SergeyBiryukov
5 weeks ago

  • Component changed from General to Bundled Theme

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


5 weeks ago

#3 follow-up: @audrasjb
5 weeks ago

  • Keywords needs-screenshots reporter-feedback added
  • Owner set to audrasjb
  • Status changed from assigned to reviewing

Hi @smallprogrammers thank you for opening this ticket and welcome to WordPress core Trac!

Would you mind uploading some screenshots of the issue?
I can't see what's wrong on the linked website.

Thanks

@audrasjb
5 weeks ago

can't reproduce the issue on Mac OSX / Chrome

#4 @audrasjb
5 weeks ago

  • Milestone changed from Awaiting Review to 5.3.2

@Shital Patel
5 weeks ago

i just removed word-break: break-word; word-wrap: break-word; in *, ::before, ::after { } style.css

@Shital Patel
5 weeks ago

Before : Screenshot of windows PC firefox browser

@Shital Patel
5 weeks ago

After : Screenshot of windows PC firefox browser

#5 @mukesh27
5 weeks ago

  • Focuses css added
  • Keywords has-patch added; needs-screenshots reporter-feedback removed

@audrasjb Replicated the same issue on the Firefox browser in Linux.

@shital-patel If we remove CSS word-break: break-word; word-wrap: break-word; from below then it will create some other text rendering issues.

*, *::before, *::after {
    box-sizing: inherit;
    -webkit-font-smoothing: antialiased;
    word-break: break-word;
    word-wrap: break-word;
}
Last edited 5 weeks ago by mukesh27 (previous) (diff)

#6 in reply to: ↑ 3 @smallprogrammers
5 weeks ago

Replying to audrasjb:

Hi @smallprogrammers thank you for opening this ticket and welcome to WordPress core Trac!

Would you mind uploading some screenshots of the issue?
I can't see what's wrong on the linked website.

Thanks

Problem seems with Firefox, i noticed. Not with Chrome or Other browser.:)

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


5 weeks ago

#8 follow-up: @audrasjb
5 weeks ago

  • Keywords needs-refresh needs-testing added
  • Milestone changed from 5.3.2 to 5.3.3

Hi there,

Thanks for working on this ticket.
Moving to 5.3.3 as this ticket still needs some work and testing.

Cheers,
Jb

#9 in reply to: ↑ 8 @smallprogrammers
5 weeks ago

Replying to audrasjb:

Thank you !

#10 @ianbelanger
3 weeks ago

  • Version changed from 5.3.1 to 5.3

@audrasjb
2 weeks ago

Reset word-break and word-wrap properties for Twenty Twenty primary and footer navigation menus

#11 @audrasjb
2 weeks ago

  • Keywords needs-refresh needs-testing removed
  • Status changed from reviewing to accepted

Hi,

In 48970.1.diff I propose to reset word-break and word-wrap properties for Twenty Twenty primary and footer navigation menus. It avoid side effects that could occur with a global reset of those properties.

Tested in various browsers with the reported string সাহায্য, it fixes the issue.

#12 @ianbelanger
8 days ago

  • Milestone changed from 5.3.3 to 5.4

Putting this into the 5.4 milestone so it can be tested and committed in time.

Note: See TracTickets for help on using tickets.