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

feat: update text to use element internals for custom states #31770

Conversation

chrisdholt
Copy link
Member

New Behavior

This PR updates the Text component to use Element Internals custom states for presentational attributes and provides an attribute fallback where not supported.

Related Issue(s)

  • Fixes #

@chrisdholt chrisdholt requested a review from a team as a code owner June 19, 2024 19:42
@chrisdholt chrisdholt force-pushed the users/chhol/update-text-with-element-internals branch from a4edf65 to 1352d8b Compare June 19, 2024 19:43
@fabricteam
Copy link
Collaborator

fabricteam commented Jun 19, 2024

📊 Bundle size report

✅ No changes found

@chrisdholt chrisdholt force-pushed the users/chhol/update-text-with-element-internals branch from 1352d8b to 7f6857f Compare June 19, 2024 22:53
@chrisdholt chrisdholt force-pushed the users/chhol/update-text-with-element-internals branch 2 times, most recently from 3af03a1 to 670d999 Compare June 24, 2024 23:04
@chrisdholt chrisdholt force-pushed the users/chhol/update-text-with-element-internals branch from 670d999 to bbb0a64 Compare June 24, 2024 23:09
packages/web-components/src/text/text.ts Outdated Show resolved Hide resolved
packages/web-components/src/text/text.ts Show resolved Hide resolved
@chrisdholt chrisdholt merged commit a651436 into microsoft:master Jun 25, 2024
18 checks passed
@chrisdholt chrisdholt deleted the users/chhol/update-text-with-element-internals branch June 25, 2024 00:25
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jun 26, 2024
* master: (90 commits)
  fix(react-drawer): do not create global header/footer roles (microsoft#31799)
  release: applying package updates - react v8
  release: applying package updates - web-components
  Add 'allowParentArrowNavigation' prop to `ComboBox`. (microsoft#31805)
  Add drawer web component (microsoft#31521)
  fix: MenuButton is styled correctly when aria-expanded is a string (microsoft#31809)
  chore: Updating version of `@fluentui/react-icons` to `^2.0.245` (microsoft#31802)
  chore:(docs) Migrate e2e Cypress testing doc (microsoft#31796)
  chore:(docs) Update API Extractor file (microsoft#31651)
  chore: migrate to nx 18 (microsoft#31746)
  release: applying package updates - react v8
  release: applying package updates - web-components
  feat: update text to use element internals for custom states (microsoft#31770)
  chore(web-components): Remove define imports from story modules (microsoft#31622)
  fix(`merge-styles`): Correctly processing selectors when value is a class name and they are not wrapped in `selectors` wrapper (microsoft#31798)
  style(react-combobox): turn off react-compiler lints for non compliant code (microsoft#31795)
  docs: Add AriaLiveAnnouncer to all examples (microsoft#31794)
  fix(public-docsite-v9): disable storybook keyboard shortcuts (microsoft#31762)
  feat: V0 Attachment compact component (microsoft#31634)
  release: applying package updates - react v8
  ...
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.

4 participants