Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hiding labels do not work as expected #240

Open
nknguyenhc opened this issue Feb 6, 2024 · 5 comments
Open

Hiding labels do not work as expected #240

nknguyenhc opened this issue Feb 6, 2024 · 5 comments

Comments

@nknguyenhc
Copy link
Contributor

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

  1. Open any repository with active labels.
  2. Click on hiding any label.

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

image

After hiding the label difficulty.Easy:

image

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser Chrome
  • Version 1.1.0

Additional context
As discussed in #230

@nknguyenhc
Copy link
Contributor Author

What do you guys think? @CATcher-org/2324s2

@cheehongw
Copy link
Contributor

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.
image

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 not difficulty.Easy.


Taking inspiration from my favorite manga site, my preference would be to overload these checkboxes to allow for include, exclude, no preference
image

image image image

@gycgabriel
Copy link
Collaborator

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,

  • How can we present this feature in a better way
  • Would it be too much effort for the user to click through each label that they are not interested in just to hide a UI

Or,

  • What are some alternative to minimize clutter from the labels?
    Unlike Github, WATcher displays the issues in shorter width, so it is much easier to run out of horizontal space to display all the labels

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

@nknguyenhc
Copy link
Contributor Author

nknguyenhc commented Feb 7, 2024

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.

@nknguyenhc nknguyenhc self-assigned this Feb 19, 2024
@nknguyenhc nknguyenhc mentioned this issue Mar 4, 2024
4 tasks
@nknguyenhc
Copy link
Contributor Author

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.

image

@nknguyenhc nknguyenhc added the good first issue Good for newcomers label Jun 17, 2024
@nknguyenhc nknguyenhc removed their assignment Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants