-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
Feature/add back to top button #281
base: main
Are you sure you want to change the base?
Feature/add back to top button #281
Conversation
✅ Deploy Preview for opensauced-landing ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
The button is on the wrong page. I'd expect to see it while reading a long article https://opensauced.pizza/blog/how-we-saved-thousands-of-dollars-deploying-low-cost-open-source-ai-technologies |
@bdougie, oh yeah, that's true. Sorry, I placed it on the main blog page (http://localhost:3001/blog). Should I put the button only on the specific blog post page (http://localhost:3001/blog/how-we-saved-thousands-of-dollars-deploying-low-cost-open-source-ai-technologies), or should I include it on both the main blog page and the specific blog post page? |
Yes. Only on blog post page.
|
@bdougie all done sir 🙇 |
I won't be able to view this from anything but mobile for awhile, but placement of the button is not working. I'd suggest centering it. I'd also suggest providing screen shots and a description to the PR. Please read this blog post. https://dev.to/opensauced/the-secret-recipe-to-getting-your-pull-requests-reviewed-and-merged-faster-1fag |
Hey, @bdougie However, I have rectified the issue. Here is the corrected and well-placed "Back to Top" button for mobile screen sizes and larger ones: large-screenmobile-screen |
Description
Related Tickets & Documents
Mobile & Desktop Screenshots/Recordings
Steps to QA
Tier (staff will fill in)
[optional] What gif best describes this PR or how it makes you feel?