WordPress.org

Make WordPress Core

Opened 4 months ago

Last modified 6 weeks ago

#46758 assigned defect (bug)

Media Trash: Primary button(s) should be on the left

Reported by: garrett-eclipse Owned by: afercia
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Media Keywords: has-screenshots needs-design-feedback
Focuses: ui, accessibility, administration Cc:

Description

Following the #design chat it was flagged that the Media Trash primary button(s) shouldn't be on the right;
https://wordpress.slack.com/archives/C02S78ZAL/p1554138401066300

Quoting @afercia;

Also: why primary buttons are on the right? :)

This came from the design discussion surrounding #44675 and is related to #46757 which denotes there should only be one primary button.

Attachments (1)

2019-04-01 at 18.01.jpg (18.5 KB) - added by garrett-eclipse 4 months ago.
Bulk Media Trash Select showing primary buttons on the right, should be on the left

Download all attachments as: .zip

Change History (8)

@garrett-eclipse
4 months ago

Bulk Media Trash Select showing primary buttons on the right, should be on the left

#1 @garrett-eclipse
4 months ago

  • Focuses accessibility added

#2 @ianbelanger
4 months ago

  • Keywords has-screenshots added

I'm not sure that we can move the buttons around yet, at least until #46757 is resolved. Mainly because we don't know which button will be the primary

#3 @garrett-eclipse
4 months ago

  • Keywords needs-design-feedback added

Thanks @ianbelanger I agree with you there. That being said with what info we have now I would suggest the following order;
Restore from Trash | Delete Permanently | Cancel
*And in #46757 I suggested the Delete get a new button class to mark it important as it's potentially irreversible.

Flagging for design review as well in hopes both tickets end up in the same scrub discussion.

This ticket was mentioned in Slack in #accessibility by afercia. View the logs.


3 months ago

#5 @afercia
3 months ago

  • Milestone changed from Awaiting Review to Future Release

Moving to Future Release, pending resolution of #46757.

#6 @afercia
6 weeks ago

  • Milestone changed from Future Release to 5.3
  • Owner set to afercia
  • Status changed from new to assigned

As in this release cycle there's some focus on improvements for the media views, due also to the WPCampus accessibility report, I'd like to propose this ticket for 5.3 consideration together with #46757.

#7 @afercia
6 weeks ago

Note: a patch that fixes this issue is uploaded to #46757.

Note: See TracTickets for help on using tickets.