-
Notifications
You must be signed in to change notification settings - Fork 428
fix(gnoweb): UI batch #4462
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
fix(gnoweb): UI batch #4462
Conversation
🛠 PR Checks SummaryAll Automated Checks passed. ✅ Manual Checks (for Reviewers):
Read More🤖 This bot helps streamline PR reviews by verifying automated checks and providing guidance for contributors and reviewers. ✅ Automated Checks (for Contributors):🟢 Maintainers must be able to edit this pull request (more info) ☑️ Contributor Actions:
☑️ Reviewer Actions:
📚 Resources:Debug
|
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR addresses various UI/UX issues and improves the overall user experience across different platforms and browsers. ## Changes ### Visual Improvements - **Responsive header**: Enhanced mobile experience, particularly useful for home page on mobile devices - **Image link tooltip positioning**: Fixed tooltip positioning on Safari browser - **Margin adjustments**: Improved spacing for realm-view content and alert components ### Bug Fixes - **Fast command copy**: Removed unnecessary comments from fast command code copy functionality - **Enable `p` and `r` to be clickable/browsable**: Make breadcrumb navigation more interactive on breadcrumb for better UX - **Source page logging**: Cleaned up logging from source page - **Source page padding**: Added proper top padding to source pages - **Windows link emojis**: Removed emoji display from links on Windows platform - **iPad summary markers**: Fixed summary marker display issues on iPad devices ### Platform-Specific Fixes - **Safari compatibility**: Fixed image link tooltip positioning issues - **Windows compatibility**: Removed emoji characters from links to prevent display issues - **iPad compatibility**: Resolved summary marker rendering problems ### CSS Improvements - **Realm-view margins**: Optimized margin handling for paragraphs containing only image links - **Alert margins**: Adjusted markdown alert component margins for better visual hierarchy - **Responsive design**: Enhanced header responsiveness for better mobile experience - **On this page**: Remove ToC title when the ToC is empty. ## Impact These changes improve the overall user experience by: - Providing better mobile responsiveness - Ensuring consistent behavior across different browsers and platforms - Fixing visual inconsistencies and spacing issues - Cleaning up unnecessary code and logging
This PR addresses various UI/UX issues and improves the overall user experience across different platforms and browsers.
Changes
Visual Improvements
Bug Fixes
p
andr
to be clickable/browsable: Make breadcrumb navigation more interactive on breadcrumb for better UXPlatform-Specific Fixes
CSS Improvements
Impact
These changes improve the overall user experience by: