Skip to content

Conversation

silverwind
Copy link
Member

Previous code passed a jQuery collection which initTooltip couldn't handle. Instead, iterate the individial matched elements and add a dollar to the variable name to make it clear it's jQuery.

Fixes: #20809

Previous code passed a jQuery collection which initTooltip couldn't
handle. Instead, iterate the individial matched elements and add a
dollar to the variable name to make it clear it's jQuery.

Fixes: go-gitea#20809
@silverwind silverwind added type/bug topic/ui Change the appearance of the Gitea UI labels Aug 16, 2022
@silverwind silverwind added this to the 1.18.0 milestone Aug 16, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Aug 16, 2022
@silverwind
Copy link
Member Author

I also renamed form to $form for clarity, which then made eslint hit on jquery/no-serialize, so I replaced $form.serialize with the modern vanilla js equivalent.

@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 Aug 16, 2022
@lafriks lafriks merged commit 1e0fcd1 into go-gitea:main Aug 16, 2022
@silverwind silverwind deleted the fix-tooltip-init branch August 16, 2022 22:18
@lunny lunny added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Aug 17, 2022
zjjhot added a commit to zjjhot/gitea that referenced this pull request Aug 17, 2022
* giteaofficial/main:
  Add proxy host into allow list (go-gitea#20798)
  Fix tooltip init after review (go-gitea#20814)
vsysoev pushed a commit to IntegraSDL/gitea that referenced this pull request Aug 28, 2022
* Fix tooltip init after review

Previous code passed a jQuery collection which initTooltip couldn't
handle. Instead, iterate the individial matched elements and add a
dollar to the variable name to make it clear it's jQuery.

Fixes: go-gitea#20809
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. topic/ui Change the appearance of the Gitea UI type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Post a review comment on pull request files view frontend error.
5 participants