Skip to content

fix(theme-classic): add caret for dropdown on mobile#7048

Merged
lex111 merged 2 commits intomainfrom
lex111-patch-1
Mar 28, 2022
Merged

fix(theme-classic): add caret for dropdown on mobile#7048
lex111 merged 2 commits intomainfrom
lex111-patch-1

Conversation

@lex111
Copy link
Contributor

@lex111 lex111 commented Mar 28, 2022

Motivation

After merging of #6983, the caret icon on collapsible dropdown is no longer displayed on mobiles

image

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

image

Related PRs

@lex111 lex111 added the pr: bug fix This PR fixes a bug in a past release. label Mar 28, 2022
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Mar 28, 2022
@netlify
Copy link

netlify bot commented Mar 28, 2022

[V2]

Name Link
🔨 Latest commit 4f31766
🔍 Latest deploy log https://app.netlify.com/sites/docusaurus-2/deploys/624234ee00ab440008a0caa5
😎 Deploy Preview https://deploy-preview-7048--docusaurus-2.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.

@github-actions
Copy link

github-actions bot commented Mar 28, 2022

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 58
🟢 Accessibility 100
🟢 Best practices 92
🟢 SEO 100
🟢 PWA 90

Lighthouse ran on https://deploy-preview-7048--docusaurus-2.netlify.app/

@github-actions
Copy link

github-actions bot commented Mar 28, 2022

Size Change: +26 B (0%)

Total Size: 807 kB

ℹ️ View Unchanged
Filename Size Change
website/.docusaurus/globalData.json 49.9 kB 0 B
website/build/assets/css/styles.********.css 105 kB 0 B
website/build/assets/js/main.********.js 613 kB +26 B (0%)
website/build/index.html 38.4 kB 0 B

compressed-size-action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Signed Facebook CLA pr: bug fix This PR fixes a bug in a past release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants