-
Notifications
You must be signed in to change notification settings - Fork 35
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
Feature/sidebar sub items #3181
base: staging
Are you sure you want to change the base?
Feature/sidebar sub items #3181
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## staging #3181 +/- ##
===========================================
- Coverage 95.42% 95.29% -0.14%
===========================================
Files 80 78 -2
Lines 13066 13060 -6
Branches 444 445 +1
===========================================
- Hits 12468 12445 -23
- Misses 580 598 +18
+ Partials 18 17 -1 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
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.
LGTM, it's ready to merge into Staging as soon we merge the #3238 into Production
This pull request includes several changes to improve the sidebar functionality and update test selectors. The most important changes include modifications to the sidebar component to support expanded nodes, updates to test selectors, and improvements to the tree structure and node components.
Sidebar functionality improvements:
site/gatsby-site/src/components/sidebar/index.js
: Added support for expanded nodes with localStorage persistence and toggle functions. [1] [2] [3] [4] [5]site/gatsby-site/src/components/sidebar/tree.js
: Refactored tree structure to handle expanded nodes and added PropTypes validation.site/gatsby-site/src/components/sidebar/treeNode.js
: Updated tree node component to handle expanded nodes and added keyboard accessibility. [1] [2] [3]Test selector updates:
site/gatsby-site/playwright/e2e/landingPage.spec.ts
: Updated test selectors for sidebar elements.site/gatsby-site/playwright/e2e/navigation.spec.ts
: Modified test selectors to ensure accurate element targeting.Other changes:
site/gatsby-site/tailwind.config.js
: Added dynamic class safelisting for TailwindCSS.