Open nknguyenhc opened 10 months ago
What do you guys think? @CATcher-org/2324s2
I think the original intention of that icon is to hide labels on items that are cluttered with labels, to allow for easy viewing.
For example, an item like this has 4 labels, and users might want to hide some of them from view.
However, I do see a use case for what you are proposing. I think our filter does not achieve that yet.
i want to see all items with
priority.High
that is notdifficulty.Easy
.
Taking inspiration from my favorite manga site, my preference would be to overload these checkboxes to allow for include
, exclude
, no preference
This is certainly something worth thinking about.
Is there a use case for hiding the UI label without excluding it from search? If there is,
Or,
Three-state checkbox
Kind of goes against what users expect of a checkbox, but it is a possible solution
I would suggest together with that, also change the label colours or some bigger indicator on the label name itself so it is more obvious to the user if it is negated or selected. As the checkbox is quite small
I would agree with the 3-state checkbox as a solution to incorporate the feature of hiding relevant issues & PRs. Perhaps instead of presenting it as a checkbox, we can indicate green border around the label in the dropdown when filtering the label in (selected), and red when filtering the label out (negated)? I think by incorporating this 3-state button, we can also better present hiding just the label better as a feature than a bug.
For the feature of hiding just the label to reduce clutter, I think the feature is fine as it is right now. It is sometimes necessary to reduce the clutter when there are too many labels used on an issue. When the user wants to unhide the label, the user would still have to go back to the list to click on the eye icon. However, it would still be too much effort for the user to go through a long list of labels just to hide a label from the table. Perhaps a possible enhancement to this feature is to have an eye icon on the label on each issue/PR card, which the user can just click to hide the label from the table? Probably the eye icon should only be visible when the user hover the mouse over the label.
First part of 3-state labels has been done. Second part to this is to create an "eye icon" besides each label, so that the user can click on the eye icon to hide the label from the table. It should do the same thing as pressing the eye icon from the label dropdown.
Describe the bug When hiding a label, from the dropdown on the top right corner, only the labels are hidden, but not the related issues & PRs.
To Reproduce
Expected behavior It should hide the issues & PRs that have the label. The current behaviour is that only the label becomes hidden, but not the related issues & PRs.
I think it makes more sense that when a user clicks on hiding a label, the user wants to hide the relevant issues & PRs with the label.
Screenshots
After hiding the label
difficulty.Easy
:Desktop (please complete the following information):
Additional context As discussed in #230