update-favicons #482
Closed
update-favicons #482
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
have updated favicons and js file for picking random favicons
🎉 All Checks Passed!
✅ Completed Workflows
🚀 This PR is ready for review and can be safely merged to Great work! Your code meets all quality standards. 👏 |
🎉 All Checks Passed!
✅ Completed Workflows
🚀 This PR is ready for review and can be safely merged to Great work! Your code meets all quality standards. 👏 |
Contributor
Author
|
@sa-fw-an please check this |
Member
|
Please tell me one reason, y would you delete the present favicons and make your own, the issue is to add new similar favicons and update the list. the process is right, but do not delete the present favicons. |
Contributor
Author
|
Yea will do it correctly now again |
Member
Great, that shld help. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Description
Updated favicon images in public/favicons and added/modified JavaScript in public/frandomfavicons to enable random favicon selection on page load.
This improves branding and adds a dynamic element to the site’s appearance.
🔗 Related Issue
Fixes #258
🔄 Type of Change
🎨 UI/UX Update (visual changes, styling improvements)
🧹 Code Refactoring
📷 Visual Changes
Screenshots / GIFs
Before: Standard favicon displayed on all pages.After: Favicon changes randomly from a set on each page load.
Firefox (Version: 118)
Safari (Version: 16)
Edge (Version: 117)
Mobile Chrome (Device: Android)
Mobile Safari (Device: iPhone)
🖥️ Responsive Design
Desktop (1200px+)
Tablet (768px - 1199px)
Mobile (320px - 767px)
✅ Test Cases
Favicon changes randomly on each page load.
No broken links or missing favicon images.
JS runs without console errors.
♿ Accessibility
Proper heading hierarchy maintained
ARIA labels added where needed
Color contrast requirements met
Keyboard navigation works correctly
Screen reader testing performed
📋 PR Checklist
My code follows the project's coding style guidelines
I have tested these changes locally
I have updated the documentation accordingly
My changes generate no new warnings or console errors
I have added tests that prove my fix/feature works
All existing tests pass successfully
I have checked for and resolved any merge conflicts
I have optimized images/assets (if applicable)
I have validated all links are working correctly
💭 Additional Notes
Please review the JS logic for random favicon selection for efficiency.
All favicon images are optimized for web use.
📚 Reviewer Resources
Contributing Guide
Style Guide
Community Chat
Thank you for contributing to the Sugar Labs website! 🎉