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

[chore]: create base class for accordion item and remove style and layout specific api #32102

Merged
merged 3 commits into from
Jul 25, 2024

Conversation

eljefe223
Copy link
Contributor

Previous Behavior

Style and layout API was bundled into one AccordionItem class.

New Behavior

Abstracts out style and layout API and creates AccordionItem class.

Related Issue(s)

  • Fixes #

@eljefe223 eljefe223 requested a review from a team as a code owner July 24, 2024 20:46
@fabricteam
Copy link
Collaborator

fabricteam commented Jul 24, 2024

📊 Bundle size report

✅ No changes found

@fabricteam
Copy link
Collaborator

fabricteam commented Jul 24, 2024

🕵 fluentui-web-components-v3 No visual regressions between this PR and main

@eljefe223 eljefe223 force-pushed the users/jes/accordion-base-class branch from c24073a to e766283 Compare July 25, 2024 04:16
@eljefe223 eljefe223 enabled auto-merge (squash) July 25, 2024 04:17
@eljefe223 eljefe223 merged commit da667ca into microsoft:master Jul 25, 2024
16 of 18 checks passed
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jul 25, 2024
* master: (77 commits)
  chore(react-examples): replace storybook deprecated api with static stories (microsoft#32074)
  [chore]: create base class for accordion item and remove style and layout specific api (microsoft#32102)
  release: applying package updates - web-components
  chore(web-components): add test for complex focus management (microsoft#32009)
  [chore]: create base class for avatar and remove style and layout specific api (microsoft#32083)
  [chore]: create base class for text input and remove style and layout specific api (microsoft#32080)
  release: applying package updates - web-components
  chore:(react-nav-preview) Scaffold AppItem (microsoft#32088)
  docs(`react-teaching-popover`): Adding subcomponents' API to `TeachingPopover` documentation page (microsoft#32084)
  feat(web-components): relax setTheme() argument type to allow custom tokens (microsoft#32087)
  release: applying package updates - react-components
  [Chore]: Create Spinner base class to abstract out style and layout specific api. (microsoft#32067)
  [Chore]: Create Progress Bar base class to abstract out style and layout specific api. (microsoft#32066)
  fix(react-storybook-addon): transform decorator to function in withAriaLive() (microsoft#32011)
  fix(motion): improve Web Animations API detection in tests (microsoft#32029)
  chore(eslint-plugin): removes type dependency on @fluentui/react-utilities internals in ban-instanceof-html-element rule (microsoft#32072)
  release: applying package updates - react v8
  release: applying package updates - web-components
  Chore: Create Divider base class to abstract out style and layout specific api (microsoft#32065)
  fix(TimePicker): Clear text when date value changes to null (microsoft#31626)
  ...
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.

3 participants