Skip to content

style: make on-this-page sticky#184

Merged
mansona merged 3 commits intoember-learn:mainfrom
IgnaceMaes:sticky-sidebars
Feb 22, 2024
Merged

style: make on-this-page sticky#184
mansona merged 3 commits intoember-learn:mainfrom
IgnaceMaes:sticky-sidebars

Conversation

@IgnaceMaes
Copy link
Copy Markdown
Member

@IgnaceMaes IgnaceMaes commented Feb 21, 2024

Implements part of #182

Makes the "On This Page" section sticky

Test: https://deploy-preview-184--guidemaker-ember-template.netlify.app/release/guide-test

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 21, 2024

Deploy Preview for guidemaker-ember-template ready!

Name Link
🔨 Latest commit 923f049
🔍 Latest deploy log https://app.netlify.com/sites/guidemaker-ember-template/deploys/65d73476ee46f200089703b9
😎 Deploy Preview https://deploy-preview-184--guidemaker-ember-template.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.

@mansona
Copy link
Copy Markdown
Member

mansona commented Feb 22, 2024

This looks great! I think you need to just update tests since you added a new page 👍

@IgnaceMaes
Copy link
Copy Markdown
Member Author

Ah good catch, updated!

Copy link
Copy Markdown
Member

@mansona mansona left a comment

Choose a reason for hiding this comment

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

Looks great 🎉 :shipit:

@mansona mansona merged commit 4b55862 into ember-learn:main Feb 22, 2024
@github-actions github-actions bot mentioned this pull request Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants