Long spam link mobile view

[Marius84]

I got some spam comments with long link. When I access admin dashboard trough mobile phone and go to the spam comment section the text "Item" cover "Empty Trash" text.

Screenshot: ​https://prnt.sc/mp1m7i

[earnjam]

Thanks @Marius84! I am able to replicate this.

It looks like it's due to this rule on line 1900 in /wp-admin/css/list-tables.css

 css
.widefat * {
	word-wrap: normal;
}

It overrides the default break-word value for .widefat * in the common css file, but I'm not sure why specifically. I'd be a bit weary of changing that for how many things it would affect. Might be better to target the comments list table directly, or even the specific paragraph tag where these things are overrunning the container.

[SergeyBiryukov]

In my testing, the overlapping of "Empty Trash" and "1 item" happens regardless of the long URL. Looks like absolute positioning is not ideal for smaller screens like this.

common.css.patch​ fixes the URL wrapping, but not the overlapping.

[donmhico]

common.css.patch doesn't work for me. See my screenshot - ​https://prnt.sc/opdcvf

[donmhico]

Update the common.css.patch.

[donmhico]

Fix overflowing long link and overlapping comment count in Empty Trash button issue.

[davidbaumwald]

@SergeyBiryukov Are you still up for including this one in 5.3?

[davidbaumwald]

This one was brought up in today's scrub. This looks like a good candidate for 5.3, so marking for commit.

[davidbaumwald]

With 5.3 RC1 releasing today, this ticket is being moved to Future Release. If any committer feels this can be worked in to 5.3 before the deadline or can assume ownership in the 5.4 cycle, feel free to move it back up.

[audrasjb]

Hi, the patch needs to be refreshed against trunk but it looks like it solves the issue (I reproduced the issue recently). Adding this on the 5.7 list.

[audrasjb]

patch refreshed against trunk

[hellofromTonya]

From Sergey:

There's a refreshed patch, seems like a good commit candidate

Marking as a commit candidate.

[hellofromTonya]

With 5.7 RC1 minutes away, moving this to 5.8 which will allow the fix to go through a full beta cycle.

[davidbaumwald]

@audrasjb Looking to move this one forward. Can you provide some information on how to reproduce the original issue and how to test the latest patch resolves the issue?

[audrasjb]

@davidbaumwald to reproduce the original issue, put long links into the content of some comments marked as spam and try to visit the page using a small screen browser. See the first screenshot provided by the reporter.

[audrasjb]

Hmmm… looks like the issue was already solved by another patch.
Now we have this in common.css:

.widefat * {
	word-wrap: break-word;
}

[davidbaumwald]

@audrasjb Thanks for the information! I am unable to reproduce the original issue locally in Chrome and Firefox at any break point.

@Marius84 Can you confirm this is still an issue in WordPress 5.7?

[chaion07]

Thanks to @Marius84 for reporting this. We recently reviewed this during a recent [bug-scrub session]​https://wordpress.slack.com/archives/C02RQBWTW/p1623098868382200. With Beta 1 coming up in a day and still waiting on Reporter Feedback we feel that this could be a good candidate for future Releases. Milestone updated. Thanks

[sabernhardt]

Spam comments screen (Firefox responsive mode, extended view, WordPress 5.7)

[sabernhardt]

Long links wrap to the next line, apparently since WordPress 5.4 with [46790]/#48805, and the positioning overlap was fixed for 5.6 in [49178]/#49246.

(The screenshot shows 414x896 dimensions)