Opened 4 years ago
Closed 4 years ago
#52749 closed defect (bug) (fixed)
Wrap long lines on Posts screen (search results), Media Library, and Permalink Settings
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | has-screenshots has-patch commit |
Focuses: | ui, css, administration | Cc: |
Description
Hi,
i have find the UI issue on WordPress 5.7 running Twenty Nineteen theme. you can see mentioned screenshot.
Attachments (10)
Change History (34)
#1
@
4 years ago
- Component changed from General to Administration
- Focuses ui css administration added
- Summary changed from issue WordPress 5.7 running Twenty Nineteen theme with search post in post list admin side to Wrap long lines on Posts screen (search results), Media Library, and Permalink Settings
#3
@
4 years ago
- Keywords needs-patch added; needs-design removed
Replace needs-design
with needs-patch
as we can directly propose a CSS patch for those small text overflow edge cases, no need for specific design work here.
@
4 years ago
In the media library list mode on mobile, when the file title is too long, it doesn't wrap correctly.
#6
@
4 years ago
- Milestone changed from Awaiting Review to 5.7.1
Putting this on 5.7.1, it's a desirable fix but if adding the word wrapping rules in CSS proves too fragile it might need to slip to a later release.
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
4 years ago
#9
@
4 years ago
- Keywords needs-testing added
Adding needs-testing
keyword, to ensure the proposed patch doesn't introduce any side effect.
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
4 years ago
#12
@
4 years ago
- Version 5.7 deleted
Hello there,
Thanks for the patch. Let's move this one to the next major release (5.8) as it would be nice to give it some time for better beta testing, to make sure it doesn't introduce any side effect. Also, it's not clear the issue was introduced in 5.7.
Thanks
This ticket was mentioned in Slack in #core-css by ryelle. View the logs.
4 years ago
#16
@
4 years ago
Tested the patch on Desktop and mobile. While the issue is resolved in the list tables, I could still see it in the Permalink Settings page (on desktop).
#17
@
4 years ago
@hareesh-pillai I am not able to reproduce the issue you mentioned. Can you provide me a screenshot please?
#18
@
4 years ago
As we can see in the above screenshots, the text overflows out of the screen in the Permalink Settings page.
However, it seems to be fine when I'm using a really long URL. The issue presents only when I tested with a long text.
#19
@
4 years ago
@hareesh-pillai The permalink radio option text you mentioned is not editable from the backend. The only way to edit the text is by using inspect tool. In my humble opinion this is not a real or even edge case scenario.
#21
@
4 years ago
- Keywords commit added; needs-testing removed
- Owner set to audrasjb
- Status changed from new to reviewing
Thanks you all for your tests. Marking for commit
.
#23
@
4 years ago
The changes to the row actions (removing that 70px margin) has an unexpected side-effect on larger screens, where the row actions wrap under the image. There's another ticket, #47895, which has to do with these row actions on smaller screens.
The search term change is a good one though, so I'll commit that :)
And with the decision to leave permalinks alone, moving the row actions changes to #47895, & the commit here, I think that addresses all the issues.
Issue