fix: allow inline code to wrap on mobile viewports#13252
fix: allow inline code to wrap on mobile viewports#13252blaqshyd wants to merge 2 commits intoflutter:mainfrom
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
There was a problem hiding this comment.
Code Review
This pull request updates the SCSS for code components, primarily changing the white-space and word-break properties for inline code to allow for better text wrapping. It also includes various formatting adjustments, such as removing spaces around child combinators and cleaning up trailing whitespace. The review feedback identifies unnecessary empty lines that should be removed to maintain consistency with the existing file style.
Description of what this PR is changing or adding, and why:
This PR resolves the horizontal overflow issue reported in #13251.
By removing the rigid white-space: nowrap rule for main code and main kbd elements, the site now utilizes a fluid wrapping model. This ensures that long API identifiers (common in breaking changes documentation) wrap gracefully on mobile devices
or on desktop viewports with an active sidebar.
Issues fixed by this PR (if any): Fixes #13251
PRs or commits this PR depends on (if any):
VIsual Verification
Presubmit checklist
of 80 characters or fewer.