-
Notifications
You must be signed in to change notification settings - Fork 36
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
DOP-5093: make tabs selectors interactive on offline builds #1325
Conversation
✅ Deploy Preview for docs-frontend-dotcomprd ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for docs-frontend-stg ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -4,7 +4,8 @@ const { siteMetadata } = require('../site-metadata'); | |||
|
|||
const GATSBY_IMAGE_EXTENSIONS = ['webp', 'png', 'avif']; | |||
|
|||
const needsImageOptimization = ['dotcomprd', 'dotcomstg'].includes(siteMetadata.snootyEnv); | |||
const needsImageOptimization = |
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.
minor change to prevent image optimization from offline builds. will put in a separate PR if preferred!
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
Stories/Links:
DOP-5093
Staging Links:
Link to download sample HTML
Another page on cloud docs with tabs selectors
The full bundle can be located in this S3 path . Each html file could be downloaded and inspected to see that tabs selectors are working
Regression check to verify tabs selectors have no regression without the
OFFLINE_DOCS
flagNotes:
This PR makes tab selectors work in offline docs, by making a custom menu dropdown (similar to the Select menu dropdown) and a script to enable the UI/UX
README updates