Make WordPress Core

Opened 7 years ago

Closed 4 years ago

#42201 closed defect (bug) (fixed)

Admin Sidebar Text length Issue

Reported by: jagirbaheshwp's profile jagirbaheshwp Owned by: audrasjb's profile audrasjb
Milestone: 5.6 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-screenshots has-patch early commit
Focuses: ui, css Cc:

Description

Hello Team,

As this is really awesome as working with the community of wordpress CMS.

While creating custom post type my post type text is too large and i see text are not properly aligned in sidebar and it's look not good.

For Ref. Screen Shot Attach

https://i.imgur.com/qX4OTCW.png

Attachments (40)

screenshot-103.8.216.142-2017-10-12-21-29-36.png (20.6 KB) - added by jagirbaheshwp 7 years ago.
42201.diff (354 bytes) - added by munyagu 7 years ago.
Padding added to the left side of the label.
42201_1.patch (290 bytes) - added by jagirbaheshwp 7 years ago.
Added Display flex
42201-2.diff (605 bytes) - added by pratikkry 6 years ago.
42201.3.diff (612 bytes) - added by Hareesh Pillai 6 years ago.
Patch refreshed
42201.1.png (110.6 KB) - added by Hareesh Pillai 6 years ago.
Screenshot of the page after applying the patch
42201 - Responsive Issue.png (71.2 KB) - added by mukesh27 6 years ago.
Responsive design issue
Schermafbeelding 2019-06-07 om 13.34.13.png (50.2 KB) - added by dushanthi 6 years ago.
42201.4.diff (936 bytes) - added by worldweb 5 years ago.
Added display:flex, word-wrap: break-word and height: auto for responsive
desktop.png (10.8 KB) - added by worldweb 5 years ago.
Tested in desktop and mobile. Working fine.
Capture d’écran 2019-08-16 à 14.50.33.png (21.3 KB) - added by audrasjb 5 years ago.
Works fine with 42201.5.diff
42201.5.diff (482 bytes) - added by audrasjb 5 years ago.
Patch refresh and correct diff file
menu counters display flex.png (26.6 KB) - added by afercia 5 years ago.
menu longer display flex.png (16.8 KB) - added by afercia 5 years ago.
42201.6.patch (631 bytes) - added by amolv 5 years ago.
Patch refresh and correct diff file
before, after - 42201.6.patch .png (324.6 KB) - added by amolv 5 years ago.
Admin Menus Before and After - 42201.6.patch
contact.png (33.7 KB) - added by iqbalbary 5 years ago.
Plugin Number Counter
42201.7.patch (800 bytes) - added by amolv 5 years ago.
Patch refresh and correct diff file.
menu items visual diff.gif (149.5 KB) - added by afercia 5 years ago.
WordPress trunk 5.3 vs. WordPress 5.2.3
42201.8.patch (1.4 KB) - added by amolv 5 years ago.
Patch refresh and correct diff file.
Screen Shot 2019-10-09 at 3.34.28 PM.png (115.2 KB) - added by amolv 5 years ago.
Latest patch fixes longer Menus and submenus text, rounded circle to counts till 9 number and it's 18px, spacing between menu text and count.
42201.9.patch (1.4 KB) - added by amolv 5 years ago.
Patch refresh and correct diff file.
42201 menu collapsed.png (277.2 KB) - added by afercia 5 years ago.
Collapsed menu counters not styled
42201.10.patch (1.4 KB) - added by amolv 5 years ago.
Latest patch diff file, counter fix on menu collapsed
42201.updates-count.PNG (2.5 KB) - added by SergeyBiryukov 5 years ago.
notification-circle-after-42201.10-chrome.png (32.5 KB) - added by sabernhardt 5 years ago.
Notification circles after patch 42201.10 in Chrome (Windows 10)
notification-circle-after-42201.10-folded-chrome.png (14.0 KB) - added by sabernhardt 5 years ago.
Notification counters in folded menu after patch 42201.10 (Chrome, Windows 10)
r46555-regression.png (42.3 KB) - added by johnjamesjacoby 5 years ago.
Bildschirmfoto 2019-10-17 um 16.07.32.png (17.4 KB) - added by zodiac1978 5 years ago.
UI glitch for every color schema besides the default
42201.11.patch (1019 bytes) - added by amolv 5 years ago.
Patch refresh to support admin menu counter/bubble in different color schemes
lont title menu test.png (328.9 KB) - added by amolv 4 years ago.
Patch Refresh, without using flex long titles/menus displayed nicely.
42201.12.patch (377 bytes) - added by amolv 4 years ago.
Patch Refresh.
Capture d’écran 2020-09-10 à 17.10.04.png (34.0 KB) - added by audrasjb 4 years ago.
Last patch works fine
42201-patch12-narrow-screenshots.png (75.5 KB) - added by helen 4 years ago.
42201.13.patch (526 bytes) - added by amolv 4 years ago.
Patch Refresh.
42201.14.diff (736 bytes) - added by audrasjb 4 years ago.
New patch to address a small responsive issue
Capture d’écran 2020-09-15 à 00.07.57.png (126.2 KB) - added by audrasjb 4 years ago.
42201.14.diff on narrow screen
IE11.png (68.9 KB) - added by mukesh27 4 years ago.
IE11 issue
42201.15.patch (843 bytes) - added by amolv 4 years ago.
Patch refresh and IE fix
chrome.gif (111.0 KB) - added by metalandcoffee 4 years ago.
Latest Patch Testing

Change History (128)

#1 @SergeyBiryukov
7 years ago

  • Component changed from General to Administration

@munyagu
7 years ago

Padding added to the left side of the label.

#2 @munyagu
7 years ago

  • Keywords has-patch added

@jagirbaheshwp
7 years ago

Added Display flex

#3 @harshbarach
7 years ago

Aha @jagirbaheshwp , it's working like charm & glossy.

#4 @Hareesh Pillai
6 years ago

  • Keywords needs-patch added; has-patch removed

The submitted patch does not fix the bug in responsive width.

@pratikkry
6 years ago

#5 @pratikkry
6 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

#6 @naveenkharwar
6 years ago

  • Resolution set to worksforme
  • Status changed from new to closed

The submitted patch fixed the bug in responsive width too.
Tested in WordPress( 5.1-beta3-20190207.100728 ).

#7 @naveenkharwar
6 years ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

#8 @naveenkharwar
6 years ago

  • Resolution set to worksforme
  • Status changed from reopened to closed

#9 @pratikkry
6 years ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

@Hareesh Pillai
6 years ago

Patch refreshed

This ticket was mentioned in Slack in #core by hareesh-pillai. View the logs.


6 years ago

#11 @SergeyBiryukov
6 years ago

  • Milestone changed from Awaiting Review to 5.3

@Hareesh Pillai
6 years ago

Screenshot of the page after applying the patch

#12 @Hareesh Pillai
6 years ago

  • Keywords has-screenshots added

@mukesh27
6 years ago

Responsive design issue

#13 @mukesh27
6 years ago

  • Keywords needs-refresh added

Below css generate issue in responsive design but if we use height: auto then it will fixed issue in responsive design.

@media only screen and (max-width: 960px) {
	.auto-fold #adminmenu a.menu-top {
		height: 34px;
	}
}

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


6 years ago

#15 @marybaum
6 years ago

  • Owner set to marybaum
  • Status changed from reopened to assigned

#16 @chetan200891
6 years ago

For me adding below padding fixes issue.

#adminmenu div.wp-menu-name {
	padding: 8px 8px 8px 35px;
}

#17 @marybaum
6 years ago

  • Milestone changed from 5.3 to 5.2.2

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


6 years ago

#19 @audrasjb
6 years ago

  • Milestone changed from 5.2.2 to 5.3

#20 @dushanthi
6 years ago

Run the patch and created some long custom post types texts, seems so work fine. One thing I noticed is that really long "words" are too close to the edge of the sidebar. https://core.trac.wordpress.org/raw-attachment/ticket/42201/Schermafbeelding%202019-06-07%20om%2013.34.13.png

#21 @dushanthi
6 years ago

  • Keywords needs-testing removed

@worldweb
5 years ago

Added display:flex, word-wrap: break-word and height: auto for responsive

@worldweb
5 years ago

Tested in desktop and mobile. Working fine.

#22 @worldweb
5 years ago

  • Resolution set to worksforme
  • Status changed from assigned to closed

#23 @worldweb
5 years ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

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


5 years ago

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


5 years ago

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


5 years ago

This ticket was mentioned in Slack in #forums by worldweb. View the logs.


5 years ago

This ticket was mentioned in Slack in #core-themes by worldweb. View the logs.


5 years ago

This ticket was mentioned in Slack in #core-editor by worldweb. View the logs.


5 years ago

#30 @audrasjb
5 years ago

  • Owner changed from marybaum to audrasjb
  • Status changed from reopened to assigned

@audrasjb
5 years ago

Works fine with 42201.5.diff

#31 @audrasjb
5 years ago

  • Keywords commit added; needs-refresh removed

Changing owner of the ticket.

I tested the proposed patch and it works fine (see screenshot).

However I refreshed the patch since the previous one doesn't apply to WordPress Dev sourcecode.
42201.5.diff fix the implementation and refreshes the proposed patch against trunk.

Also @worldweb, youdon't need to copy patches to RTL CSS files since they are automatically generated during build process ;-)

Adding commit keyword.

@audrasjb
5 years ago

Patch refresh and correct diff file

#32 @whyisjake
5 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 46332:

Administration: Ensure visual consistency with large titles in the sidebar.

Custom posts, and other menu titles that are long aren't wrapping correctly.

Props jagirbaheshwp, munyagu, harshbarach, hareesh-pillai, pratikkry, naveenkharwar, mukesh27, dushanthi, worldweb, audrasjb
Fixes #42201

#33 @afercia
5 years ago

  • Keywords needs-patch added; has-patch commit removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

Looks like this change introduced a small regression when the admin menu items display the "counters": the spacing is now clearly smaller and the counters are almost attached to the menu item text.

See in the screenshot below: left: before the change, right: after the change that added display: flex.

#34 @afercia
5 years ago

Also, with longer translations and counters with multiple digits, the new display: flexand word-break: break-word; properties make the counter break.

On the left: the previous CSS where, at least, the counter goes to a new line and doesn't break.
On the right: the new CSS.

@amolv
5 years ago

Patch refresh and correct diff file

@amolv
5 years ago

Admin Menus Before and After - 42201.6.patch

#35 @amolv
5 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

@iqbalbary
5 years ago

Plugin Number Counter

#36 follow-up: @iqbalbary
5 years ago

  • Keywords needs-patch added; has-patch removed

42201.6.patch fixes the plugin counter problem only, still need to add #adminmenu .awaiting-mod, #adminmenu .update-plugins for comment counter.

Also if we can add css to .wp-menu-name > span then it will cover other menu(theme & plugins) too which have counter option.

Last edited 5 years ago by iqbalbary (previous) (diff)

#37 in reply to: ↑ 36 @amolv
5 years ago

Replying to iqbalbary:

42201.6.patch fixes the plugin counter problem only, still need to add #adminmenu .awaiting-mod, #adminmenu .update-plugins for comment counter.

Also if we can add css to .wp-menu-name > span then it will cover other menu(theme & plugins) too which have counter option.

Thank you for testing, updating CSS.

@amolv
5 years ago

Patch refresh and correct diff file.

#38 @amolv
5 years ago

  • Keywords has-patch added; needs-patch removed

#39 @davidbaumwald
5 years ago

  • Keywords commit added; needs-testing removed

Tested the most recent patch, and it works as intended. Marking this for commit.

#40 @whyisjake
5 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 46453:

Administration: Ensure visual consistency with large titles in the sidebar.

Custom posts, and other menu titles that are long aren't wrapping correctly.

Props jagirbaheshwp, munyagu, harshbarach, hareesh-pillai, pratikkry, naveenkharwar, mukesh27, dushanthi, worldweb, audrasjb, afercia, amolv, iqbalbary, davidbaumwald.
Fixes #42201

#41 @afercia
5 years ago

  • Keywords needs-patch added; has-patch commit removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

Sorry to reopen but [46453] doesn't seem to fully address the unnecessary visual changes introduced with the previous commit.

1
On WordPress 5.2 when the counters have only 1 digit, they're a 18 by 18 pixels circle. This is not a coincidence: it's intentional and the CSS should make sure it's a perfect circle. Instead, now the counters aren't a perfect circle: they have a slightly oval shape and the number is not perfectly centered. Screenshot:

http://cldup.com/k2JO9FcvIt.png

2
The spacing between the text and the counters is still clearly different and doesn't help readability. Not to mention it's not great visually.

3
With longer menu items or longer translations the counter ends up in the middle of the page: this is undesirable. On WordPress 5.2 wasn't perfect as well but at least it went to a new line leaving just the text to overflow into the page.

Screenshot for 2 and 3:

http://cldup.com/zbBRKZgyBH.png

4
#adminmenu .wp-menu-name > span is now a flex item thus these properties applied on it:

display: inline-block;
vertical-align: top;

don't make much sense. Alignment should be controlled with the properties that pertain to flexbox.

See also the attached animated GIF that better highlights the visual changes.

@afercia
5 years ago

WordPress trunk 5.3 vs. WordPress 5.2.3

@amolv
5 years ago

Patch refresh and correct diff file.

#42 @amolv
5 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

@amolv
5 years ago

Latest patch fixes longer Menus and submenus text, rounded circle to counts till 9 number and it's 18px, spacing between menu text and count.

@amolv
5 years ago

Patch refresh and correct diff file.

#43 @afercia
5 years ago

  • Keywords needs-patch added; has-patch removed

One more thing to address that happens also on trunk before applying 42201.9.patch:

When the menu is collapsed, the counters are not styled properly. See screenshot below. Actually, there is another counter within .wp-submenu-head that is shown only when the menu is collapsed.

@afercia
5 years ago

Collapsed menu counters not styled

@amolv
5 years ago

Latest patch diff file, counter fix on menu collapsed

#44 @amolv
5 years ago

  • Keywords has-patch added; needs-patch removed

When menus is collapsed .wp-submenu-head fix added in recent patch.

#45 @SergeyBiryukov
5 years ago

The updates count under Dashboard is no longer styled properly after [46453], see 42201.updates-count.PNG.

42201.10.patch appears to resolve the issue.

@sabernhardt
5 years ago

Notification circles after patch 42201.10 in Chrome (Windows 10)

#46 @sabernhardt
5 years ago

The notification circle style is restored with 42201.10 in Chrome, Edge, Internet Explorer and Firefox (Windows 10).

@sabernhardt
5 years ago

Notification counters in folded menu after patch 42201.10 (Chrome, Windows 10)

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


5 years ago

#48 @johnbillion
5 years ago

  • Keywords commit added; needs-testing removed

Thanks for the testing @sabernhardt .

Safari 12, Chrome 77, and Firefox 69 all look good on macOS, as does latest Chrome on Android.

Those nested selectors aren't optimal but I can't immediately see a better way to do it, so +1 from me to get this in.

#49 @johnbillion
5 years ago

  • Owner changed from audrasjb to johnbillion
  • Status changed from reopened to accepted

#50 @johnbillion
5 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 46555:

Administration: Fix the appearance of update bubbles in submenus and when the admin menu is collapsed.

Props afercia, amolv, sabernhardt

Fixes #42201, #48294

#51 @johnjamesjacoby
5 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

r46555 broke the use of :after on #adminmenu .wp-submenu a.

Specifically, changing from display: block; to display: flex;.

I help build or maintain several plugins that add styling to a pseudo element after some specific anchors. This styling is applied specifically based on user feedback for plugins with large add-on ecosystems that add many sub-menus to a single top-level menu item.

/* Secondary Separators */
#adminmenu ul > li:not(:last-child) a[href^="admin.php?page=sugar-calendar"]::after,
#adminmenu ul > li:not(:last-child) a[href^="post-new.php?post_type=sc_event"]::after
{
	display: block;
	border-bottom: 1px solid rgba(255,255,255,0.2);
	height: 14px;
	margin: 0 -13px;
	content: '';
}

With these anchors now displaying as flexible instead of blocks, the :after pseudo elements flex after them, even when display: block; is applied to them.

#52 @johnjamesjacoby
5 years ago

Some research from: https://www.w3.org/TR/css-flexbox-1/#flex-items

Each in-flow child of a flex container becomes a flex item, and each contiguous sequence of child text runs is wrapped in an anonymous block container flex item. However, if the entire sequence of child text runs contains only white space (i.e. characters that can be affected by the white-space property) it is instead not rendered (just as if its text nodes were display:none).

If I am reading this correctly, anything now "inside" that now-flexed anchor is forced to be a flex element.

Using display: block !important; even gets ignored in all browsers, confirming (to me) that child pseudo elements of a flexed element are forced to flex, meaning there is no alternative CSS to get this styling working again (a more strict hack could override the core anchor styling, but whack-a-mole priority juggling isn't a solution IMO.)

A flex item establishes an independent formatting context for its contents. However, flex items themselves are flex-level boxes, not block-level boxes: they participate in their container’s flex formatting context, not in a block formatting context.

Last edited 5 years ago by johnjamesjacoby (previous) (diff)

@zodiac1978
5 years ago

UI glitch for every color schema besides the default

#53 @zodiac1978
5 years ago

I get a wrong styling in every color scheme besides the default one (looking at 5.3-RC1-46560).

Reason from colors.min.css (in this example):

#adminmenu .awaiting-mod, #adminmenu .update-plugins {
    color: #fff;
    background: #e1a948;
}

#54 follow-up: @afercia
5 years ago

Personally, at this point of the release cycle (WordPress 5.3 is now in Release Candidate) I'd lean towards a complete revert of all the changes from this ticket.

This is not to blame anyone :) The intent was good but the WordPress admin menu is tricky, made of various "layers" of functionalities, HTML, and CSS that stratified across the years. It's a very good example of a place where changing even a very small detail can break many things.

Overall, this change doesn't seem ready for 5.3 and it can be explored better in the next release cycle.

#55 in reply to: ↑ 54 @SergeyBiryukov
5 years ago

Replying to afercia:

Personally, at this point of the release cycle (WordPress 5.3 is now in Release Candidate) I'd lean towards a complete revert of all the changes from this ticket.

I tend to agree. Given that we're still discovering regressions after two follow-up commits, seems like we should consider reverting [46332], [46453], and [46555] for now, and trying again in a future release to allow for more testing.

@amolv
5 years ago

Patch refresh to support admin menu counter/bubble in different color schemes

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


5 years ago

#57 @johnbillion
5 years ago

  • Keywords revert added; has-patch commit removed
  • Status changed from reopened to accepted

Agreed on the revert. Let's get this back into 5.4 as soon as we branch so it can be tested as-is and iterated upon.

#58 @johnbillion
5 years ago

Reverting given agreement from three committers.

#59 @johnbillion
5 years ago

In 46561:

Administration: Revert changes to the admin menu styles for long line lengths.

This reverts [46555], [46453], and [46332].

See #42201

#60 @johnbillion
5 years ago

  • Keywords early added; revert removed
  • Milestone changed from 5.3 to 5.4

#61 @pratikkry
5 years ago

  • Focuses css added

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


5 years ago

#63 @davidbaumwald
5 years ago

  • Keywords needs-testing added
  • Owner changed from johnbillion to desrosj
  • Status changed from accepted to assigned

#64 @davidbaumwald
5 years ago

  • Keywords needs-testing removed
  • Milestone changed from 5.4 to Future Release
  • Owner changed from desrosj to johnbillion

Edited wrong ticket and falsely reassigned. Reassigning this to @johnbillion.

Since this ticket doesn't have any momentum, it's being moved to Future Release. If any committer feels this can be moved up or can be owned during a specific cycle, feel free to update the milestone accordingly.

@amolv
4 years ago

Patch Refresh, without using flex long titles/menus displayed nicely.

@amolv
4 years ago

Patch Refresh.

#65 @amolv
4 years ago

  • Keywords has-patch needs-testing added

#66 @johnbillion
4 years ago

  • Keywords early removed
  • Milestone changed from Future Release to 5.6

#67 @johnbillion
4 years ago

  • Keywords early added
  • Status changed from assigned to reviewing
  • Version 4.8.2 deleted

@audrasjb
4 years ago

Last patch works fine

#68 @audrasjb
4 years ago

  • Keywords commit added; needs-testing removed

I tested the last proposed patch and it looks good to go on my side. Marking this for commit.

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


4 years ago

#70 @johnbillion
4 years ago

Agreed, I tested this last night and it looks like a good approach. I think we can commit it so it gets a good amount of testing across browsers.

#71 @helen
4 years ago

  • Keywords commit removed

I've uploaded a screenshot of the effects in the narrow viewport/mobile menu view, it's not great now but it seems even worse after. I'd like for that to be addressed before committing a patch here.

#72 @garrett-eclipse
4 years ago

  • Keywords needs-refresh added

@amolv
4 years ago

Patch Refresh.

#73 @amolv
4 years ago

  • Keywords needs-testing added; needs-refresh removed

@helen thank you for test, patch refreshed to fix issue on narrow viewport.

#74 @audrasjb
4 years ago

Looks good to me, thank you @amolv 👍

#75 @SergeyBiryukov
4 years ago

Could we have a screenshot of 42201.13.patch on a narrow viewport?

@audrasjb
4 years ago

New patch to address a small responsive issue

@audrasjb
4 years ago

42201.14.diff on narrow screen

#76 @audrasjb
4 years ago

  • Keywords needs-testing removed

While I was taking screenshot of the last patch, I noticed a small remaining CSS issue. 42201.14.diff fixes this issue. See screenshot of the new patch above.

Last edited 4 years ago by audrasjb (previous) (diff)

#77 @mukesh27
4 years ago

42201.14.diff working fine in WP 5.6-alpha-48958 version.

#78 follow-up: @garrett-eclipse
4 years ago

With the use of word-break: break-word; has this been tested on IE11? To my understanding it doesn't support that css;
https://caniuse.com/mdn-css_properties_word-break_break-word
*I would but am on a mac.

#79 in reply to: ↑ 78 @mukesh27
4 years ago

I have tested 42201.14.diff in IE11 and word-break: break-word; is not working in IE11.

Replying to garrett-eclipse:

With the use of word-break: break-word; has this been tested on IE11? To my understanding it doesn't support that css;
https://caniuse.com/mdn-css_properties_word-break_break-word
*I would but am on a mac.

@mukesh27
4 years ago

IE11 issue

@amolv
4 years ago

Patch refresh and IE fix

This ticket was mentioned in Slack in #core-css by kirstyburgoine. View the logs.


4 years ago

#81 @garrett-eclipse
4 years ago

  • Keywords needs-testing added

Latest patch should be retested across browsers but looks good.

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


4 years ago

#83 @johnbillion
4 years ago

  • Owner johnbillion deleted

#84 @audrasjb
4 years ago

  • Keywords close commit added; needs-testing removed
  • Owner set to audrasjb
  • Status changed from reviewing to accepted

The refreshed patch looks good to go on my side.
It's basically the same than mine, but it adds cross browsers support.

#85 @audrasjb
4 years ago

  • Keywords close removed

#86 @metalandcoffee
4 years ago

I tested 42201.15.patch on the latest version of Chrome, Firefox and Safari. It looks good to go. Attaching screenshot. Still needs to be verified on IE11 since I'm a mac user :)

@metalandcoffee
4 years ago

Latest Patch Testing

#87 @helen
4 years ago

#44938 was marked as a duplicate.

#88 @helen
4 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 49149:

Admin Menu: Better wrapping for long menu item names.

Props munyagu, jagirbaheshwp, harshbarach, pratikkry, hareesh-pillai, naveenkharwar, mukesh27, chetan200891, dushanthi, worldweb, audrasjb, afercia, amolv, iqbalbary, davidbaumwald, sabernhardt, johnjamesjacoby, garrett-eclipse, garrett-eclipse.
Fixes #42201.

Note: See TracTickets for help on using tickets.