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 markdown anchor re-clicking #21931

Merged
merged 9 commits into from
Nov 26, 2022
Merged

Fix markdown anchor re-clicking #21931

merged 9 commits into from
Nov 26, 2022

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
web_src/js/markup/anchors.js Outdated Show resolved Hide resolved
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