Implement scroll-to-top feature#165
Conversation
✅ Deploy Preview for vocabulary-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| @@ -1 +1,3 @@ | |||
| .DS_Store No newline at end of file | |||
There was a problem hiding this comment.
changes to this file should be removed they're not a part of this repo
|
Hi! @Daodu111 This is a lot of good work, and the start of a very good direction! The visual styling of this button is really nice, however we don't utilizing drop shadows in other instances and the use of it here is a bit of a UX collision. Additionally the choice of color is too attention-seeking the neutral grey would be a better direction. It's placement is too far to the right, and too close to the bottom of the window. It should likely appear once the page has reached a great height, and well after any sidebar area of content might occur; and likely be positioned somewhere closer to the middle of the sidebar area's location. The button also needs to be an element that could occur within the HTML of every page, but only become visible on longer pages. Please see the original Issue on the mention of pseudo classes to obscure the element.
(you'll also need to resolve the current conflicts with the PR as well) |
Fixes
Description
I created a scroll-to-top feature that uses only HTML and CSS and follows the instructions of using pseudeoelements to solve the issue.
Technical details
It uses only HTML and CSS
Tests
Screenshots
Checklist
Update index.md).mainormaster).visible errors.
Developer Certificate of Origin
For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."
Developer Certificate of Origin