#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)
Change History (24)
comment:3 @bradt — 11 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.
comment:4 @bradt — 11 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.
@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)
@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.
comment:6 @SergeyBiryukov — 7 months ago
- Milestone changed from Awaiting Review to 4.2
comment:8 @slackbot — 7 months ago
This ticket was mentioned in Slack in #core-flow by boren. View the logs.
comment:9 follow-up: ↓ 11 @bradt — 6 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
comment:10 @DrewAPicture — 6 months ago
- Priority changed from normal to high
comment:11 in reply to: ↑ 9 @ryan — 6 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 @DrewAPicture — 6 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 @helen — 6 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.
comment:14 @ryan — 5 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 @slackbot — 5 weeks ago
This ticket was mentioned in Slack in #core-flow by boren. View the logs.
Working on a patch to hide the "In Response To" column.