Make WordPress Core

Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#29991 closed defect (bug) (fixed)

Comment action links are quite cramped on small screens

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

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 years ago.
29991.diff​ (3.9 KB ) - added by bradt 12 years ago.
Screen Shot 2014-11-14 at 3.52.09 PM.png​ (83.9 KB ) - added by bradt 12 years ago.
29991.2.diff​ (3.9 KB ) - added by bradt 12 years ago.
29991.3.diff​ (1.5 KB ) - added by coreymcollins 12 years 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 12 years 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 12 years ago.
Fixes "undefined variable" errors
29991.6.diff​ (4.0 KB ) - added by bradt 12 years ago.
Previous patch was missing CSS.
IMG_4348.PNG​ (277.5 KB ) - added by ryan 11 years ago.
Comment action links now have room

Download all attachments as: .zip

Change History (24)

@pento
12 years ago

#1 @soficgr
12 years ago

  • Severity normal → minor

#2 @bradt
12 years ago

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

@bradt
12 years ago

#3 @bradt
12 years 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.

@bradt
12 years ago

#4 @bradt
12 years ago

  • Keywords needs-testing added

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

@coreymcollins
12 years 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)

@coreymcollins
12 years 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.

@gregrickaby
12 years ago

Fixes "undefined variable" errors

@bradt
12 years ago

Previous patch was missing CSS.

#5 @bradt
12 years ago

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

#6 @SergeyBiryukov
12 years ago

  • Milestone Awaiting Review → 4.2

Moving for review along with #29992 and #29993.

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


12 years ago

#9 follow-up: @bradt
12 years 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

Version 0, edited 12 years ago by bradt (next)

#10 @DrewAPicture
12 years ago

  • Priority normal → high

#11 in reply to: ↑ 9 @ryan
12 years 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.

#12 @DrewAPicture
12 years ago

  • Milestone 4.2 → Future Release
  • Priority high → normal

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

#13 @helen
12 years 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.

@ryan
11 years ago

Comment action links now have room

#14 @ryan
11 years ago

  • Milestone Future Release → 4.3
  • Resolution → fixed
  • Status new → closed

Fixed by the work on #32395.

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


11 years ago

Note: See TracTickets for help on using tickets.