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

Fix Fomantic UI dropdown icon bug when there is a search input in menu #27225

Merged
merged 1 commit into from
Sep 24, 2023

Conversation

wxiaoguang
Copy link
Contributor

Fix #27224

And add the case to the devtest page.

@wxiaoguang wxiaoguang added the backport/v1.21 This PR should be backported to Gitea 1.21 label Sep 24, 2023
@wxiaoguang wxiaoguang added this to the 1.22.0 milestone Sep 24, 2023
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 24, 2023
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 24, 2023
@github-actions github-actions bot added the topic/ui Change the appearance of the Gitea UI label Sep 24, 2023
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Sep 24, 2023
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Sep 24, 2023
@denyskon denyskon added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Sep 24, 2023
@wxiaoguang wxiaoguang merged commit efd5176 into go-gitea:main Sep 24, 2023
25 checks passed
@wxiaoguang wxiaoguang deleted the fix-dropdown-icon branch September 24, 2023 13:08
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Sep 24, 2023
@GiteaBot GiteaBot added backport/done All backports for this PR have been created and removed reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. labels Sep 24, 2023
lunny pushed a commit that referenced this pull request Sep 24, 2023
#27225) (#27228)

Backport #27225 by @wxiaoguang

Fix #27224

And add the case to the devtest page.

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@silverwind
Copy link
Member

Could maybe also be fixed with pointer-event: none on the triangle, so that click events "go through" it.

@wxiaoguang
Copy link
Contributor Author

Could maybe also be fixed with pointer-event: none on the triangle, so that click events "go through" it.

No, it can't. The "dropdown icon" has a special event listener.

@silverwind
Copy link
Member

Huh, seems pointless 😆.

zjjhot added a commit to zjjhot/gitea that referenced this pull request Sep 25, 2023
* giteaofficial/main:
  Add missing public user visibility in user details page (go-gitea#27246)
  Use mask-based fade-out effect for `.new-menu` (go-gitea#27181)
  [skip ci] Updated translations via Crowdin
  Fix z-index on markdown completion (go-gitea#27237)
  Update database-preparation and add note re: MariaDB (go-gitea#27232)
  cleanup locale function usage (go-gitea#27227)
  Fix EOL handling in web editor (go-gitea#27141)
  Fix PushEvent NullPointerException jenkinsci/github-plugin (go-gitea#27203)
  fix issues on action runners page (go-gitea#27226)
  Fix Fomantic UI dropdown icon bug when there is a search input in menu (go-gitea#27225)
  Update go-enry to 2.8.5 (go-gitea#27215)
  Update nodejs installation method in release container (go-gitea#27207)
  Quote table `release` in sql queries (go-gitea#27205)
  Fix push mirror, wrong timestamp format (go-gitea#27153)
  Allow copying issue comment link on archived repos and when not logged in (go-gitea#27193)
  fix: text decorator on issue sidebar menu label (go-gitea#27206)
  Update JS and Poetry dependencies and eslint (go-gitea#27200)
  Remove some dead code (go-gitea#27196)

# Conflicts:
#	templates/repo/issue/view_content/context_menu.tmpl
lunny pushed a commit that referenced this pull request Oct 9, 2023
)

Follow #27225

The change in #27225 is not ideal, this should be the complete fix:
support the layout which Fomantic doesn't support.
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Oct 9, 2023
…gitea#27526)

Follow go-gitea#27225

The change in go-gitea#27225 is not ideal, this should be the complete fix:
support the layout which Fomantic doesn't support.
wxiaoguang added a commit that referenced this pull request Oct 9, 2023
) (#27534)

Backport #27526 by @wxiaoguang

Follow #27225

The change in #27225 is not ideal, this should be the complete fix:
support the layout which Fomantic doesn't support.

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Dec 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created backport/v1.21 This PR should be backported to Gitea 1.21 lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dropdown doesn't work when clicking on an arrow
6 participants