Closed Manancode closed 2 weeks ago
A new React functional component named FullscreenToggle
has been added to the project. This component manages fullscreen functionality and is integrated into the DocsLayout
component, which now wraps the DocsContext.Provider
with the FullscreenToggle
. The FullscreenToggle
allows users to enter or exit fullscreen mode, updating its state accordingly. The overall structure of the DocsLayout
remains intact, preserving existing features such as error handling and conditional rendering.
File | Change Summary |
---|---|
components/FullScreenToggle.tsx | Added FullscreenToggle component with props for children and className , and fullscreen logic. |
components/layout/DocsLayout.tsx | Integrated FullscreenToggle into DocsLayout , wrapping DocsContext.Provider . |
In a world of screens so wide,
A toggle now, we take in stride.
With a click, the view expands,
A rabbit's joy in pixel lands!
Fullscreen dreams, so bright and clear,
Hop along, the magic's here! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Built without sensitive environment variables
Name | Link |
---|---|
Latest commit | 66a3b5a0b8a67800a590934ab85992c341bc2299 |
Latest deploy log | https://app.netlify.com/sites/asyncapi-website/deploys/672b371f24069f00086970cf |
Deploy Preview | https://deploy-preview-3365--asyncapi-website.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 49.14%. Comparing base (
dfd08dd
) to head (66a3b5a
). Report is 14 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
⚡️ Lighthouse report for the changes in this PR: | Category | Score |
---|---|---|
🔴 Performance | 45 | |
🟢 Accessibility | 98 | |
🟢 Best practices | 92 | |
🟢 SEO | 100 | |
🔴 PWA | 33 |
Lighthouse ran on https://deploy-preview-3365--asyncapi-website.netlify.app/
@Manancode The button doesn't look very intuitive. An end user can go on a full screen mode via native browser features
Sir even i didnt felt any need for it but since no one reviewed the issue so just made a PR incase needed
Closing the linked issue since there doesn't seems to be a need for this feature at the moment. Other popular documentation frameworks like Docusaurus doesn't offer such a functionality
Added a fullscreen toggle feature to enhance the reading experience of documentation pages. The toggle will only appear on the navbar on /docs page
Tested locally ( works fine )
closes issue
Summary by CodeRabbit
New Features
Bug Fixes
Documentation