Skip to content

add styles for non-linkable sidebar entries #2883

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

Merged
merged 3 commits into from
Dec 24, 2021

Conversation

Simek
Copy link
Collaborator

@Simek Simek commented Dec 16, 2021

Refs: #2882

So far we were not using non-linkable grouping in the sidebar (collapsible: false), but this feature could be leverage for the The New Architecture guide and future website content. This also enables us to rethink the current sidebar organization, which also can benefit for those non-linkable sections.

At this moment titles of the groups looks like the regular entry, which might confuse the users, This PR adds the proper styling for those sidebar headers and also fixes the missing indentation and unintended font size overwrite at the sidebar top-level after one of the latest Docusaurs updates.

Preview

You can only see the indent and font size changes in here. If you want to see the non-linkable check out the preview in #2882.

Screenshot 2021-12-16 at 13 53 56

@netlify
Copy link

netlify bot commented Dec 16, 2021

✔️ Deploy Preview for react-native ready!

🔨 Explore the source changes: 861665d

🔍 Inspect the deploy log: https://app.netlify.com/sites/react-native/deploys/61bb574558db780008379919

😎 Browse the preview: https://deploy-preview-2883--react-native.netlify.app

Simek added a commit to Simek/react-native-website that referenced this pull request Dec 21, 2021
@lunaleaps lunaleaps self-requested a review December 24, 2021 00:05
@lunaleaps
Copy link
Contributor

+1 to the indentation and style updates! I'll leave it to you when you want to merge! Thanks @Simek!

@Simek Simek merged commit 02f480f into facebook:main Dec 24, 2021
@Simek Simek deleted the style-non-linkable-nested-sidebar-entries branch December 24, 2021 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants