WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 5 weeks ago

#42201 assigned defect (bug)

Admin Sidebar Text length Issue

Reported by: jagirbaheshwp Owned by: audrasjb
Milestone: 5.3 Priority: normal
Severity: normal Version: 4.8.2
Component: Administration Keywords: has-patch has-screenshots commit
Focuses: ui 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 (12)

screenshot-103.8.216.142-2017-10-12-21-29-36.png (20.6 KB) - added by jagirbaheshwp 2 years ago.
42201.diff (354 bytes) - added by munyagu 2 years ago.
Padding added to the left side of the label.
42201_1.patch (290 bytes) - added by jagirbaheshwp 2 years ago.
Added Display flex
42201-2.diff (605 bytes) - added by pratikkry 9 months ago.
42201.3.diff (612 bytes) - added by Hareesh Pillai 4 months ago.
Patch refreshed
42201.1.png (110.6 KB) - added by Hareesh Pillai 4 months ago.
Screenshot of the page after applying the patch
42201 - Responsive Issue.png (71.2 KB) - added by mukesh27 4 months ago.
Responsive design issue
Schermafbeelding 2019-06-07 om 13.34.13.png (50.2 KB) - added by dushanthi 4 months ago.
42201.4.diff (936 bytes) - added by worldweb 2 months ago.
Added display:flex, word-wrap: break-word and height: auto for responsive
desktop.png (10.8 KB) - added by worldweb 2 months 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 weeks ago.
Works fine with 42201.5.diff
42201.5.diff (482 bytes) - added by audrasjb 5 weeks ago.
Patch refresh and correct diff file

Download all attachments as: .zip

Change History (43)

#1 @SergeyBiryukov
2 years ago

  • Component changed from General to Administration

@munyagu
2 years ago

Padding added to the left side of the label.

#2 @munyagu
2 years ago

  • Keywords has-patch added

@jagirbaheshwp
2 years ago

Added Display flex

#3 @harshbarach
2 years ago

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

#4 @Hareesh Pillai
9 months ago

  • Keywords needs-patch added; has-patch removed

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

@pratikkry
9 months ago

#5 @pratikkry
9 months ago

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

#6 @naveenkharwar
8 months 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
8 months ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

#8 @naveenkharwar
8 months ago

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

#9 @pratikkry
8 months ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

@Hareesh Pillai
4 months ago

Patch refreshed

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


4 months ago

#11 @SergeyBiryukov
4 months ago

  • Milestone changed from Awaiting Review to 5.3

@Hareesh Pillai
4 months ago

Screenshot of the page after applying the patch

#12 @Hareesh Pillai
4 months ago

  • Keywords has-screenshots added

@mukesh27
4 months ago

Responsive design issue

#13 @mukesh27
4 months 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.


4 months ago

#15 @marybaum
4 months ago

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

#16 @chetan200891
4 months ago

For me adding below padding fixes issue.

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

#17 @marybaum
4 months ago

  • Milestone changed from 5.3 to 5.2.2

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


4 months ago

#19 @audrasjb
4 months ago

  • Milestone changed from 5.2.2 to 5.3

#20 @dushanthi
4 months 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
4 months ago

  • Keywords needs-testing removed

@worldweb
2 months ago

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

@worldweb
2 months ago

Tested in desktop and mobile. Working fine.

#22 @worldweb
2 months ago

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

#23 @worldweb
2 months ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

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


2 months ago

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


2 months ago

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


2 months ago

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


2 months ago

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


2 months ago

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


2 months ago

#30 @audrasjb
5 weeks ago

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

@audrasjb
5 weeks ago

Works fine with 42201.5.diff

#31 @audrasjb
5 weeks 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 weeks ago

Patch refresh and correct diff file

Note: See TracTickets for help on using tickets.