Opened 9 months ago

Last modified 9 months ago

#35319 assigned Cleanup/optimization

Move the link to add a related object inside the .selector-available column for FilteredSelectMultiple.

Reported by: Hrushikesh Vaidya Owned by: Hrushikesh Vaidya
Component: contrib.admin Version: 5.0
Severity: Normal Keywords: accessibility
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: yes
Needs tests: yes Patch needs improvement: yes
Easy pickings: no UI/UX: yes

Description

This came up during discussion on ​this PR trying to fix #34643, specifically in ​this comment.

The patch for #34643 involves allowing the related widget wrapper to span the entire width of the screen, which causes the JavaScript widget for FilteredSelectMultiple to span the entire width of the screen. However, in case the widget has a link to add another object, that link now seems disconnected from the widget, especially on mobile layouts (see attached).

It would improve UX if the link to add a related object is moved to the header of the widget instead of at the end.

Attachments (3)

filtered-select-multiple-full-width.png​ (127.2 KB ) - added by Hrushikesh Vaidya 9 months ago.
The current position of the add link, after the filtered select widget stretches the full width of the screen
filtered-select-multiple-full-width-mobile.png​ (40.5 KB ) - added by Hrushikesh Vaidya 9 months ago.
The current position of the add link on mobile, after the patch
filtered-select-multiple-new-add-link-position.png​ (36.0 KB ) - added by Hrushikesh Vaidya 9 months ago.
Proposed new location of the add link

Download all attachments as: .zip

Change History (7)

by Hrushikesh Vaidya, 9 months ago

Attachment: filtered-select-multiple-full-width.png​ added

The current position of the add link, after the filtered select widget stretches the full width of the screen

by Hrushikesh Vaidya, 9 months ago

Attachment: filtered-select-multiple-full-width-mobile.png​ added

The current position of the add link on mobile, after the patch

by Hrushikesh Vaidya, 9 months ago

Attachment: filtered-select-multiple-new-add-link-position.png​ added

Proposed new location of the add link

comment:1 by Mariusz Felisiak, 9 months ago

Keywords: accessibility added
Owner: changed from nobody to Hrushikesh Vaidya
Status: new → assigned
Summary: Move the link to add a related object inside the .selector-available column instead of after the selector widget for FilteredSelectMultiple → Move the link to add a related object inside the .selector-available column for FilteredSelectMultiple.
Triage Stage: Unreviewed → Accepted

comment:2 by Hrushikesh Vaidya, 9 months ago

Has patch: set

​Draft PR

comment:3 by Mariusz Felisiak, 9 months ago

Needs tests: set

comment:4 by Natalia Bidart, 9 months ago

Needs documentation: set
Patch needs improvement: set
Note: See TracTickets for help on using tickets.
Back to Top