Twenty Nineteen: Long submenu item titles do not wrap

[kjellr]

In the latest version of Twenty Nineteen (1.1), long submenu item titles do not wrap on desktop screens (looks like there's a whitespace: nowrap; rule applied to .main-navigation .sub-menu > li > a).

These should wrap when necessary, and wrapped lines should be indented as in the following mockup:

[mukesh27]

In my case i have to remove whitespace: nowrap; from .main-navigation .sub-menu > li > a and also needs to change min-width: min-width: max-content; to min-width: min-width: 250px;

[mohsinrasool]

Patch to set fix width issue of the submenu item in both .scss and .css files.

[mohsinrasool]

Added a patch to set fix width issue of the submenu item in both .scss and .css files. I am starting to contribute in WordPress. Do let me know if something wrong with my fixes.

Thanks

[samful]

The patch needs a very quick refresh since it hasn't been committed for 9 months and some lines have moved about, but the rules work and fixes the problem. Hope this is committed sooner rather than later.

[mukesh27]

Updated patch.

[ianbelanger]

Reviewing for commit.

[ianbelanger]

In 48032:

Bundled Themes: Twenty Nineteen long submenu item titles do not wrap.

Fixes the issue of long submenu item titles that do not wrap by adding a max-width: 20rem; and removing white-space: nowrap;.

Props kjellr, mukesh27, mohsinrasool, samful.
Fixes #45866.

[ianbelanger]

Reopening for backport.

[desrosj]

With 5.5 just under 5 weeks away, there is no 5.4.3 planned. Moving back to the 5.5 milestone.