Patch #31640
Add clear query icon next to selected query in sidebar
| Status: | New | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | - | % Done: | 0% | |
| Category: | UI | |||
| Target version: | Candidate for next major release |
Description
Now that "All issues" link was removed from the sidebar, we should have an easier way in sidebar to clear the current query.
My proposal is to add a Clear link next to selected query link:
On hover:
clear_query.png (43.4 KB)
clear_query_hover.png (45.2 KB)
0001-Add-clear-link-icon-for-selected-query-in-sidebar.patch 
(3.64 KB)
clear_with_close.png (41.2 KB)
0001-Add-clear-link-icon-for-selected-query-in-sidebar_2.patch
(3.64 KB)
Related issues
History
#1
Updated by Marius BALTEANU 22 days ago
- File 0001-Add-clear-link-icon-for-selected-query-in-sidebar.patch added
#2
Updated by Marius BALTEANU 22 days ago
- Related to Patch #30294: Move the links (View all issues, Summary, Import) from the Issues section of the issues list sidebar under a dropdown added
#3
Updated by Marius BALTEANU 22 days ago
- File deleted (
0001-Add-clear-link-icon-for-selected-query-in-sidebar.patch)
#4
Updated by Marius BALTEANU 22 days ago
Fixed some Rubocop warnings.
#5
Updated by Marius BALTEANU 22 days ago
All tests pass: https://gitlab.com/redmine-org/redmine/pipelines/68625213
Updated by #7
Updated by Marius BALTEANU 21 days ago
- File clear_with_close.png added
- File 0001-Add-clear-link-icon-for-selected-query-in-sidebar_2.patch added
Go MAEDA wrote:
Although I understand the reason why you use
(reload.png), I think the current icon used in your patch is not appropriate because it looks like a button to reload the current query. In my opinion, the icon should be the one similar to
(close.png).
Agree, but (close.png) icon is not so visible:
Attached the updated patch. Go Maeda, if you have a better idea, please fell free to update my patch.