Skip to content
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

update Docusaurus and other deps, fix broken styles #3059

Merged
merged 5 commits into from
Apr 11, 2022

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
@@ -41,7 +41,7 @@ To request access to location, you need to add the following line to your app's
Android API >= 18 Positions will also contain a `mocked` boolean to indicate if position was created from a mock provider.

<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