Skip to content

Fix labels display #32758

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

Closed
wants to merge 5 commits into from
Closed

Conversation

stuzer05
Copy link
Contributor

@stuzer05 stuzer05 commented Dec 8, 2024

Fixes #32758

image
image
image

also fixes exclusive from repo and org grouping
image
image

search fixed using css
image

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 8, 2024
@github-actions github-actions bot added the modifies/templates This PR modifies the template files label Dec 8, 2024
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Dec 8, 2024
@wxiaoguang
Copy link
Contributor

The divider between "repo" labels and "org" labels are added intentionally, if you hide it, people are not able to distinguish "reop/org" labels.

bug (repo)
bug (org)
...

@stuzer05
Copy link
Contributor Author

stuzer05 commented Dec 8, 2024

The divider between "repo" labels and "org" labels are added intentionally, if you hide it, people are not able to distinguish "reop/org" labels.

bug (repo)
bug (org)
...

Is there a use case to have identical labels under org and repo? repo labls are more like "extension" of general, org labels

@stuzer05
Copy link
Contributor Author

stuzer05 commented Dec 8, 2024

Besides, divider between repo and org labels doesn't hint which labels are org and repo owned. It only "breaks" the sorting/grouping

@wxiaoguang
Copy link
Contributor

Besides the problem above, I do no think it could work. I have explained in the issue, the "filter" problem is one of the root problems.

image

@stuzer05
Copy link
Contributor Author

stuzer05 commented Dec 8, 2024

Besides the problem above, I do no think it could work. I have explained in the issue, the "filter" problem is one of the root problems.

image

Maybe js dropdown on refresh and template fix could be combined. I understand, dropdown js fix is a "crutch" of a sort

@stuzer05
Copy link
Contributor Author

stuzer05 commented Dec 8, 2024

Besides the problem above, I do no think it could work. I have explained in the issue, the "filter" problem is one of the root problems.
image

Maybe js dropdown on refresh and template fix could be combined. I understand, dropdown js fix is a "crutch" of a sort

This could be fixed by css

.filtered + .divider {
    display: none;
}

@stuzer05
Copy link
Contributor Author

stuzer05 commented Dec 8, 2024

image
image

@pull-request-size pull-request-size bot added size/M and removed size/S labels Dec 8, 2024
@wxiaoguang
Copy link
Contributor

Besides the problem above, I do no think it could work. I have explained in the issue, the "filter" problem is one of the root problems.
image

Maybe js dropdown on refresh and template fix could be combined. I understand, dropdown js fix is a "crutch" of a sort

This could be fixed by css

.filtered + .divider {
    display: none;
}

It can't be right either.

<div>label a</div>
---- <div class="divider"></div>
<div data-scope="xxx">label b-1</div>
<div data-scope="xxx" class="filtered">label b-2</div>
---- <div class="divider"></div> here, you shouldn't hide it.
<div>label c</div>

@stuzer05
Copy link
Contributor Author

stuzer05 commented Dec 8, 2024

It can't be right either.

<div>label a</div>
---- <div class="divider"></div>
<div data-scope="xxx">label b-1</div>
<div data-scope="xxx" class="filtered">label b-2</div>
---- <div class="divider"></div> here, you shouldn't hide it.
<div>label c</div>

image
Visually it works, can you share a screenshot?

@wxiaoguang
Copy link
Contributor

Visually it works, can you share a screenshot?

Before "visual work", we should make it "theoretically right", right?

I have finished my " Fix duplicate dropdown dividers #32760 ", and added enough tests, I think it should work for all cases. The behavior is 100% controlled and we could always fine-tune it by what we need.

@stuzer05
Copy link
Contributor Author

stuzer05 commented Dec 8, 2024

Visually it works, can you share a screenshot?

Before "visual work", we should make it "theoretically right", right?

I have finished my " Fix duplicate dropdown dividers #32760 ", and added enough tests, I think it should work for all cases. The behavior is 100% controlled and we could always fine-tune it by what we need.

Can we merge repo and org labels as well? I don't see any reason why we should separate them, because now:

  1. repo and org labels aren't visually separated, only by position (not obvious)
  2. should user really think about where labels are refined? - it's more an administrative work
  3. grouping makes more sense and make find labels easy instead of confusing users

scope_a/1
scope_a/2
label_3
scope_a/3

looks "broken" in UX

@wxiaoguang
Copy link
Contributor

Can we merge repo and org labels as well? I don't see any reason why we should separate them, because now:

I have no idea, I do not use these features, maybe you could open a proposal and invite the initial contributors to provide feedbacks.

@stuzer05 stuzer05 closed this Dec 8, 2024
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Mar 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants