#36873 closed Cleanup/optimization (fixed)
Improve visual aspects of related-widget-wrapper icons.
| Reported by: | Antoliny | Owned by: | Antoliny |
|---|---|---|---|
| Component: | contrib.admin | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | yes |
Description
(Please refer to the discussion in the Discord accessibility channel.)
Currently, the icons in the related-widget-wrapper on the admin page are not visually appealing.
First, they are not vertically aligned with the field, and there seems to be a slight difference in size between the icons. (The "pencil" appears slightly larger, and the "X" appears smaller.)
After adding vertical alignment, making the icon sizes consistent would likely improve the visual appearance.
Attachments (2)
Change History (15)
comment:1 by , 2 months ago
| Owner: | set to |
|---|---|
| Status: | new → assigned |
by , 2 months ago
| Attachment: | current_related_widget_wrapper_icons.png added |
|---|
by , 2 months ago
| Attachment: | after_related_widget_wrapper.png added |
|---|
comment:2 by , 2 months ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:3 by , 7 weeks ago
| Keywords: | font-awesome added; icon removed |
|---|
comment:4 by , 7 weeks ago
| Has patch: | set |
|---|
comment:5 by , 7 weeks ago
| Needs documentation: | set |
|---|
Need a release note for the font-awesome upgrade, like last time.
comment:6 by , 6 weeks ago
| Needs documentation: | unset |
|---|
comment:7 by , 6 weeks ago
| Patch needs improvement: | set |
|---|
comment:8 by , 6 weeks ago
| Patch needs improvement: | unset |
|---|
comment:9 by , 6 weeks ago
| Patch needs improvement: | set |
|---|
comment:10 by , 6 weeks ago
| Keywords: | font-awesome removed |
|---|
comment:11 by , 5 weeks ago
| Patch needs improvement: | unset |
|---|---|
| Triage Stage: | Accepted → Ready for checkin |
| Version: | 6.0 → dev |

PR