Skip to content

[OUDS] feat(docs): added browserslistrc callouts - #3714

Open
duvalale wants to merge 6 commits into
mainfrom
ouds/main-ad-browsers-list-notice
Open

[OUDS] feat(docs): added browserslistrc callouts#3714
duvalale wants to merge 6 commits into
mainfrom
ouds/main-ad-browsers-list-notice

Conversation

@duvalale

@duvalale duvalale commented Jul 23, 2026

Copy link
Copy Markdown
Member

Related issues

Closes #3594

Description

Added callout to suggest using OUDS .browserslistrc file to introduction and browsers and devices page, and all components pages using floating labels.

Checklists

Progression (for Core Team only)

Live previews

@netlify

netlify Bot commented Jul 23, 2026

Copy link
Copy Markdown

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit 99e6198
🔍 Latest deploy log https://app.netlify.com/projects/boosted/deploys/6a6b62d676f8ca000812f68c
😎 Deploy Preview https://deploy-preview-3714--boosted.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates OUDS Web documentation to better communicate the recommended target browser list (via the repo’s .browserslistrc) and to warn users about floating-label compatibility issues when using default Browserslist/Autoprefixer settings.

Changes:

  • Added a “next steps” link to the Browsers & devices page from the Introduction.
  • Added an informational callout on Browsers & devices recommending adoption of the project’s .browserslistrc.
  • Added warning callouts to all floating-label components (text input, text area, select input, password input) linking back to Browsers & devices.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
site/src/content/docs/getting-started/introduction.mdx Adds a navigation link to browser/device support docs.
site/src/content/docs/getting-started/browsers-devices.mdx Adds a callout recommending use of the repo .browserslistrc for SCSS projects.
site/src/content/docs/components/text-input.mdx Adds a floating-label compatibility warning pointing to Browsers & devices.
site/src/content/docs/components/text-area.mdx Adds a floating-label compatibility warning pointing to Browsers & devices.
site/src/content/docs/components/select-input.mdx Adds a floating-label compatibility warning pointing to Browsers & devices.
site/src/content/docs/components/password-input.mdx Adds a floating-label compatibility warning pointing to Browsers & devices.

Comment thread site/src/content/docs/getting-started/introduction.mdx Outdated
Comment thread site/src/content/docs/getting-started/browsers-devices.mdx Outdated
Comment thread site/src/content/docs/components/text-input.mdx Outdated
Comment thread site/src/content/docs/components/text-area.mdx Outdated
Comment thread site/src/content/docs/components/select-input.mdx Outdated
Comment thread site/src/content/docs/components/password-input.mdx Outdated
@duvalale
duvalale force-pushed the ouds/main-ad-browsers-list-notice branch from 0f10427 to e20e376 Compare July 24, 2026 08:06

@MaxLardenois MaxLardenois left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice work, I suggest a toned-down wording ;)

Comment thread site/src/content/docs/components/password-input.mdx Outdated
Comment thread site/src/content/docs/components/password-input.mdx Outdated
Comment thread site/src/content/docs/components/select-input.mdx Outdated
Comment thread site/src/content/docs/getting-started/browsers-devices.mdx Outdated
@boosted-bot boosted-bot moved this from Need Dev Review to Dev Review In Progress in 🟣 [Orange-Boosted-Bootstrap] PRs Board Jul 24, 2026
duvalale and others added 4 commits July 24, 2026 11:12
@duvalale
duvalale force-pushed the ouds/main-ad-browsers-list-notice branch from b5cac8e to a0d330e Compare July 24, 2026 09:15

@MaxLardenois MaxLardenois left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Actually I would put the inputs callouts after the examples, because as it stands we have a lot of callout one after another and nothing stands out.
wdyt @louismaximepiton

@louismaximepiton

louismaximepiton commented Jul 24, 2026

Copy link
Copy Markdown
Member

I fear that having it after the example will be forgotten by many, including us, by I agree that nothing stands out right now. I'll think about a better solution if I have time

@boosted-bot boosted-bot moved this from Dev Review In Progress to Need Lead Dev Review in 🟣 [Orange-Boosted-Bootstrap] PRs Board Jul 30, 2026
@vprothais

Copy link
Copy Markdown
Collaborator

Actually I would put the inputs callouts after the examples, because as it stands we have a lot of callout one after another and nothing stands out. wdyt @louismaximepiton
I agree with you, we often have multiple callout one after another and it doesn't helps for readability. I think having it in the overview here is the good place. I don't know how to enhance this situation globaly. Let's merge like that as the problem is more global than just this callout.

@MaxLardenois

Copy link
Copy Markdown
Collaborator

Actually I would put the inputs callouts after the examples, because as it stands we have a lot of callout one after another and nothing stands out. wdyt @louismaximepiton
I agree with you, we often have multiple callout one after another and it doesn't helps for readability. I think having it in the overview here is the good place. I don't know how to enhance this situation globaly. Let's merge like that as the problem is more global than just this callout.

Approving as is, I will open an issue, maybe we need some designer help on this

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

Labels

None yet

Projects

Status: Need Lead Dev Review

Development

Successfully merging this pull request may close these issues.

[OUDS] Add information on target browser lists

6 participants