Skip to content

update Docusaurus and other deps, fix broken styles#3059

Merged
Simek merged 5 commits intofacebook:mainfrom
Simek:update-docuaurus-bump-deps
Apr 11, 2022
Merged

update Docusaurus and other deps, fix broken styles#3059
Simek merged 5 commits intofacebook:mainfrom
Simek:update-docuaurus-bump-deps

Conversation

@Simek
Copy link
Collaborator

@Simek Simek commented Apr 6, 2022

Why & how

Superseeds:

This PR updates Docusaurus to latest beta (beta.18) and most of the other dependencies.

Due to changes in Docusaurus code, I had to adjust the config and add a bunch of fixes to our custom styles, which become broken after the update. I also had to duplicate the GA move in the config, otherwise the website won't start.

Test plan

I have run the updated version of the website on localhost and I used the currently shipped website as a 1:1 comparison, so the style issues was easier to spot, however, there still might be small display problems which I was not able to catch. If you see any regression in preview deploy, please let me know!

@netlify
Copy link

netlify bot commented Apr 6, 2022

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 9267ca6
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/624d7c7f8db58d0009f5aa93
😎 Deploy Preview https://deploy-preview-3059--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Apr 6, 2022

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 72ee66e
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/624f31a692c9b700097f5a15
😎 Deploy Preview https://deploy-preview-3059--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@slorber slorber mentioned this pull request Apr 7, 2022

<p>
Android API >= 23 Requires an additional step to check for, and request
Android API &gt;= 23 Requires an additional step to check for, and request
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this needed?

@lunaleaps
Copy link
Contributor

I think this looks good! Are there particular places we should look for breakages? I guess, also should we confirm that GA tracking is working (tbh not sure how to do that)

@Simek
Copy link
Collaborator Author

Simek commented Apr 7, 2022

Are there particular places we should look for breakages?

Generally every layout have at least one issue, sidebars too, only home page looks correct after upgrade, so it's hard to point on the specific thing. I usually test few pages, with focus on the origin diversity:

  • homage (custom JS)
  • versioned doc page (docs plugin)
  • blog post list and blog post (blog plugin)
  • community page (custom MD)

I guess, also should we confirm that GA tracking is working (tbh not sure how to do that)

The fix has been deployed yesterday in separate PR, so you probably should see a data spike on the GA dashboard now, if everything is working correctly.

@lunaleaps
Copy link
Contributor

Oh actually I do think I found an issue -- I thought it was a general issue but it's not reproing off prod: #3061

@Simek Simek linked an issue Apr 7, 2022 that may be closed by this pull request
@Simek
Copy link
Collaborator Author

Simek commented Apr 7, 2022

I have fixed the mobile menu hover issue and the deploy preview should be updated. I have set the hover color, to one which we use for highlighting the active page in the menu.

Please let me know, if now it looks fine for you.

@Simek Simek merged commit fadaea5 into facebook:main Apr 11, 2022
@Simek Simek deleted the update-docuaurus-bump-deps branch April 11, 2022 17:13
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.

Bad hover background color for menu on mobile web

4 participants