Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Display badge on blocked & blocking issues#35004

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
Naxdy wants to merge1 commit intogo-gitea:main
base:main
Choose a base branch
Loading
fromNaxdy:work/mark-blocked-issues

Conversation

Naxdy
Copy link
Contributor

@NaxdyNaxdy commentedJul 8, 2025
edited
Loading

This PR supersedes#29117 by offering a more simplistic approach to the problem. Any issue / PR that is blocked by another issue / PR, or is blocking one or more issues / PRs now has a badge displayed next to it indicating as such, right before the labels.

Both cases are represented by different icons, so it is easy to see at a glance which issues / PRs require attention more urgently than others (look for the ones that areblocking the most).

Only open issues / PRs are considered for determining blocking items, and closed issues / PRs cannot "block" anything anymore, but can still be "blocked by" something, in case an issue was closed due to being cancelled as opposed to implemented.

In the future, we might even think about adding a tooltip that lists the individual issues that are blocking / blocked.


Closes#29018

Screenshots

Issue List (Repo)

image

PR List (Repo)

image

Issue List (Global)

image

Unresolved questions

  • Isstyle="background: none;" fine? It feels hacky, and like there should be a better way without adding a whole new class 🤔

@GiteaBotGiteaBot added the lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labelJul 8, 2025
@github-actionsgithub-actionsbot added modifies/translation modifies/goPull requests that update Go code modifies/templatesThis PR modifies the template files labelsJul 8, 2025
@NaxdyNaxdyforce-pushed thework/mark-blocked-issues branch 2 times, most recently from5c109c0 todbaba71CompareJuly 9, 2025 06:21
@anbraten
Copy link
Contributor

You could usetw-hidden from tailwind instead ofstyle="background: none;"

@Naxdy
Copy link
ContributorAuthor

Unfortunatelytw-hidden makes the entire element disappear. Thebackground: none is only to clear the background color, as otherwise the badge appears like this, which imo kinda clashes with the red color:

image

@NaxdyNaxdyforce-pushed thework/mark-blocked-issues branch fromdbaba71 to84b5ab0CompareJuly 9, 2025 06:46
@anbraten
Copy link
Contributor

Ahh sorry,bg-transparent might be it

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
lgtm/need 2This PR needs two approvals by maintainers to be considered for merging.modifies/frontendmodifies/goPull requests that update Go codemodifies/templatesThis PR modifies the template filesmodifies/translation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

In the PR list page show if a PR is blocked by another issue/PR.
3 participants
@Naxdy@anbraten@GiteaBot

[8]ページ先頭

©2009-2025 Movatter.jp