Skip to content

Conversation

silverwind
Copy link
Member

@silverwind silverwind commented Nov 24, 2022

The hashchange event did not fire on re-click of a active anchor. Instead, use the click event which always fires.

Fixes: #21680

The hashchange event did not fire on re-click of a active anchor.
Instead, use the click event.

Fixes: go-gitea#21680
@silverwind silverwind added this to the 1.19.0 milestone Nov 24, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Nov 24, 2022
Copy link
Member

@delvh delvh left a comment

Choose a reason for hiding this comment

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

I still rather disagree about the external anchors, but I can accept the current state.

@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 Nov 25, 2022
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 25, 2022
@silverwind
Copy link
Member Author

I pushed a fix so that it only aborts when :target exists on page load, not on subsequent clicks.

@silverwind silverwind added the outdated/backport/v1.18 This PR should be backported to Gitea 1.18 label Nov 25, 2022
@lunny
Copy link
Member

lunny commented Nov 26, 2022

make L-G-T-M work

@lunny lunny merged commit 5f38acd into go-gitea:main Nov 26, 2022
@lunny lunny removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 26, 2022
@silverwind silverwind deleted the hashclick branch November 26, 2022 15:39
silverwind added a commit to silverwind/gitea that referenced this pull request Nov 26, 2022
The hashchange event did not fire on re-click of a active anchor.
Instead, use the click event which always fires.

Fixes: go-gitea#21680

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Co-authored-by: zeripath <art27@cantab.net>
lunny added a commit that referenced this pull request Nov 26, 2022
Backport #21931. The hashchange event did not fire on re-click of a
active anchor. Instead, use the click event which always fires.

Fixes: #21680

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Co-authored-by: zeripath <art27@cantab.net>
zjjhot added a commit to zjjhot/gitea that referenced this pull request Nov 29, 2022
* giteaofficial/main:
  Use random bytes to generate access token (go-gitea#21959)
  Replace fmt.Sprintf with hex.EncodeToString (go-gitea#21960)
  Implement FSFE REUSE for golang files (go-gitea#21840)
  Correct the fallbacks for mailer configuration (go-gitea#21945)
  [skip ci] Updated licenses and gitignores
  Fix typos (go-gitea#21947)
  Fix markdown anchor re-clicking (go-gitea#21931)
  Prevent NPE if trying to restore an already restored deleted branch (go-gitea#21940)
  add rel="me" to org.website link (go-gitea#21941)
@zeripath zeripath added the backport/done All backports for this PR have been created label Jan 13, 2023
@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
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. outdated/backport/v1.18 This PR should be backported to Gitea 1.18 type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Anchors break after clicked once
5 participants