Skip to content

Update branch_dropdown.tmpl #34278

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 3 commits into from
Closed

Conversation

kerwin612
Copy link
Member

{{/* show dummy elements before Vue componment is mounted, this code must match the code in BranchTagSelector.vue */}}

before:
before

after:
after

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 24, 2025
@github-actions github-actions bot added the modifies/templates This PR modifies the template files label Apr 24, 2025
@silverwind
Copy link
Member

silverwind commented Apr 24, 2025

BTW we could likely render this "Compare" text into the placeholder so it does not flash empty on page load until the JS has loaded. Same technique should already be in use on branch dropdown on the repo page.

@kerwin612
Copy link
Member Author

BTW we could likely render this "Compare" text into the placeholder so it does not flash empty on page load until the JS has loaded. Same technique should already be in use on branch dropdown on the repo page.

after

done.

{{end}}
<strong class="tw-ml-2 tw-inline-block gt-ellipsis">{{Iif .DropdownFixedText .SelectedRefShortName}}</strong>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change is not ideal. The old bug was caused by incorrect Iif, it should be or

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, to match Vue, maybe the `if is better

@wxiaoguang
Copy link
Contributor

Merge the changes into Fix button alignments #34276

@wxiaoguang wxiaoguang closed this Apr 27, 2025
@kerwin612 kerwin612 deleted the patch-1 branch April 28, 2025 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/templates This PR modifies the template files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants