Skip to content

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

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

[OUDS] feat(docs): added browserslistrc callouts#3714
duvalale wants to merge 5 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 b316fd7
🔍 Latest deploy log https://app.netlify.com/projects/boosted/deploys/6a633537a799d90008526806
😎 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
<Code lang="plaintext" filePath=".browserslistrc" />

<Callout type="info">
We strongly suggest using our <code>.browserslistrc</code> file in projects using OUDS SCSS. This helps avoid potential browser-specific issues that can arise from differing target browser lists.

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.

Suggested change
We strongly suggest using our <code>.browserslistrc</code> file in projects using OUDS SCSS. This helps avoid potential browser-specific issues that can arise from differing target browser lists.
We strongly suggest using our <code>.browserslistrc</code> file in projects using OUDS Web SCSS. This helps avoid potential browser-specific issues that can arise from differing target browser lists.

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.

You forgot to apply this ;)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Oops, it got overwritten, done

@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 satnd out right now. I'll think about a better solution if I have time

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

Labels

None yet

Projects

Status: Dev Review In Progress

Development

Successfully merging this pull request may close these issues.

[OUDS] Add information on target browser lists

5 participants