Skip to content

fix: after a refresh cheatsheet title anchor not working#1011

Merged
sabbirshawon merged 2 commits intodevsonket:developfrom
Jisan-mia:fix-refresh-prob
Oct 7, 2023
Merged

fix: after a refresh cheatsheet title anchor not working#1011
sabbirshawon merged 2 commits intodevsonket:developfrom
Jisan-mia:fix-refresh-prob

Conversation

@Jisan-mia
Copy link
Copy Markdown
Contributor

@Jisan-mia Jisan-mia commented Oct 6, 2023

fixes this issue #987

if you can find the problem following the mentioned issue
and compare with this pr deployed URL, you'll see the difference

as sheet component is rendered for every cheatsheet page.. the page has already a path like `https://devsonket.com/bootstrap5` here `/bootstrap5` is the path which is the id of current viewing cheatsheet..
sheet component has a title with an anchor tag for every title,
in the anchor tag href `${slug}#{title}` is being used...
that's why after a refresh /slug#title is added to the URL  like this `https://devsonket.com/bootstrap5/bootstrap5#title`
@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 6, 2023

Deploy Preview for devsonket ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 320c39f
🔍 Latest deploy log https://app.netlify.com/sites/devsonket/deploys/65204c5f67b7f9000965a9ce
😎 Deploy Preview https://deploy-preview-1011--devsonket.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@sabbirshawon sabbirshawon merged commit afac864 into devsonket:develop Oct 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants