-
Notifications
You must be signed in to change notification settings - Fork 226
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
Pipboy #187
base: main
Are you sure you want to change the base?
Pipboy #187
Conversation
👋 Thanks for participating! I will review as soon as possible, usually within a few hours. Watch for notifications as I may request some small changes to make sure this meets the guidelines. In a few minutes, the Netlify process at the end of this PR will make a preview available. Look for the message "Deploy preview ready!" then click the "Details" link next to it to launch a preview of your styles. Make changes if needed, and comment if you do so I can re-deploy the preview. Note that the "sponsor footer" is not available to fully style, but you may fix any issues it causes to your layout. Thanks again, I hope you learned something while making your stylesheet! 💫 |
Hey @carterfromsl - I really enjoyed the unique style here! The biggest issue to resolve is lack of distinguishable focus styles. To test, please tab around your design and fix any place you don't see a visible focus (adding back outlines is the easiest). Let me know when you've updated your styles, and I'll rebuild the preview for another review. |
Thanks so much, @5t3ph ! I've updated the CodePen to add a more distinguishable focus with a black and amber box shadow and a thin inset outline to form a striped border. Let me know if there's anything else I can do to improve it! |
Hey again, @5t3ph ! |
New Stylesheet Submission:
title
,author
, andstylesheet
?stylesheet
a full URL that is publicly accessible and renders an unminified, compiled CSS file?@media (prefers-reduced-motion: reduce)
to remove animations on*,*::before,*::after
?.skip-link
?"Inspired by the iconic amber terminal display of Fallout: New Vegas, this design evokes nostalgic vibes of navigating the Mojave Desert."
Apologies for the lack of comments and the stream-of-consciousness vibe to the stylesheet. If you'd prefer a more structured approach, let me know and I can reorganize it to be more tidy. Otherwise, I've got this covered and can maintain as is.