WordPress.org

Make WordPress Core

Opened 12 months ago

Closed 5 weeks ago

Last modified 5 weeks ago

#29991 closed defect (bug) (fixed)

Comment action links are quite cramped on small screens

Reported by: pento Owned by:
Milestone: 4.3 Priority: normal
Severity: minor Version:
Component: Comments Keywords: make-flow has-patch needs-testing
Focuses: ui, administration Cc:

Description

On mobile devices, the comment action links are quite small and cramped - it's pretty easy to mis-tap.

Attachments (9)

IMG_2900.png (79.9 KB) - added by pento 12 months ago.
29991.diff (3.9 KB) - added by bradt 11 months ago.
Screen Shot 2014-11-14 at 3.52.09 PM.png (83.9 KB) - added by bradt 11 months ago.
29991.2.diff (3.9 KB) - added by bradt 11 months ago.
29991.3.diff (1.5 KB) - added by coreymcollins 10 months ago.
All of the above fixes and added: Increased line-height of .row-actions for comments on single post; fixed typo on .fixed .column-comment .comment-post-link (awhite-space -> white-space)
29991.4.diff (3.7 KB) - added by coreymcollins 10 months ago.
All of the above fixes and: fixing my diff. Accidentally created it while in the /css/ directory, so it was missing the additional changes.
29991.5.diff (2.3 KB) - added by gregrickaby 10 months ago.
Fixes "undefined variable" errors
29991.6.diff (4.0 KB) - added by bradt 10 months ago.
Previous patch was missing CSS.
IMG_4348.PNG (277.5 KB) - added by ryan 5 weeks ago.
Comment action links now have room

Download all attachments as: .zip

Change History (24)

@pento12 months ago

comment:1 @soficgr12 months ago

  • Severity changed from normal to minor

comment:2 @bradt11 months ago

Working on a patch to hide the "In Response To" column.

@bradt11 months ago

comment:3 @bradt11 months ago

  • Keywords has-patch added

When screen is < 782px wide, it now:

  • Adds more spacing between the action links so that it's harder to tap the wrong one
  • Hides the "In Response To" column
  • Unhides a new "In Reponse To" section in the comment column

I've removed the "View Post" link as it didn't really fit well here.

@bradt11 months ago

comment:4 @bradt11 months ago

  • Keywords needs-testing added

Revised patch. Styles were unintentionally affecting action links in the entire dashboard instead of just the comments list table.

@coreymcollins10 months ago

All of the above fixes and added: Increased line-height of .row-actions for comments on single post; fixed typo on .fixed .column-comment .comment-post-link (awhite-space -> white-space)

@coreymcollins10 months ago

All of the above fixes and: fixing my diff. Accidentally created it while in the /css/ directory, so it was missing the additional changes.

@gregrickaby10 months ago

Fixes "undefined variable" errors

@bradt10 months ago

Previous patch was missing CSS.

comment:5 @bradt7 months ago

Still applies cleanly against trunk. No need for a refresh.

comment:6 @SergeyBiryukov7 months ago

  • Milestone changed from Awaiting Review to 4.2

Moving for review along with #29992 and #29993.

comment:8 @slackbot7 months ago

This ticket was mentioned in Slack in #core-flow by boren. View the logs.

comment:9 follow-up: @bradt6 months ago

Just read the Slack convo from 3 weeks ago. So is the concensus to go ahead with an implementation like @helen's screenshot? http://cl.ly/3F1L0N3z1M0F0Z3t0X3W

Last edited 6 months ago by bradt (previous) (diff)

comment:10 @DrewAPicture6 months ago

  • Priority changed from normal to high

comment:11 in reply to: ↑ 9 @ryan6 months ago

Replying to bradt:

Just read the Slack convo from 3 weeks ago. So is the concensus to go ahead with an implementation like @helen's screenshot? http://cl.ly/3F1L0N3z1M0F0Z3t0X3W

I wouldn't mind moving to single column mobile layouts for all of our list table screens. My inclination is to run with Helen's screenshot and iterate from there.

comment:12 @DrewAPicture6 months ago

  • Milestone changed from 4.2 to Future Release
  • Priority changed from high to normal

No patch activity in the last few weeks. Punting to future release.

comment:13 @helen6 months ago

I missed this - this is one of the screens that I think needs more of a rethought to work on a narrow screen. Interim fixes are great, I'm just not convinced that comments really gets very much better with them. Happy to lay out what I think could be the path with my very old experiment going forward, if we think that's a route we want to pursue.

@ryan5 weeks ago

Comment action links now have room

comment:14 @ryan5 weeks ago

  • Milestone changed from Future Release to 4.3
  • Resolution set to fixed
  • Status changed from new to closed

Fixed by the work on #32395.

comment:15 @slackbot5 weeks ago

This ticket was mentioned in Slack in #core-flow by boren. View the logs.

Note: See TracTickets for help on using tickets.