Skip to content

Conversation

@ValentinaKozlova
Copy link
Contributor

We won't have non-interactive item anymore. Because of that we don't need slash divider because it was only for small non-interactive item.

Previous Behavior

  • Breadcrumb could have non-interactive items
  • Breadcrumb could have dividerType slash.

New Behavior

  • Removed non-interactive item. Text component should be used for file path
  • Removed dividerType because currently we have only chevron type

@codesandbox-ci
Copy link

codesandbox-ci bot commented Oct 24, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 8db0f8a:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@fabricteam
Copy link
Collaborator

fabricteam commented Oct 24, 2023

📊 Bundle size report

🤖 This report was generated against 5e65d8013cd09faf4b3777202f251d5c3d67c138

@size-auditor
Copy link

size-auditor bot commented Oct 24, 2023

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 5e65d8013cd09faf4b3777202f251d5c3d67c138 (build)

@ValentinaKozlova ValentinaKozlova marked this pull request as ready for review October 24, 2023 14:35
@ValentinaKozlova ValentinaKozlova requested review from a team as code owners October 24, 2023 14:35
@marcosmoura marcosmoura changed the title feat: (breadcrumb) Removed non-interactive item feat(react-breadcrumb): removed non-interactive item Oct 25, 2023
ValentinaKozlova and others added 2 commits October 25, 2023 15:25
…nts/Breadcrumb/useBreadcrumbContextValue.ts

Co-authored-by: Oleksandr Fediashov <[email protected]>
@ValentinaKozlova ValentinaKozlova merged commit d518c28 into microsoft:master Oct 25, 2023
@ValentinaKozlova ValentinaKozlova deleted the feat/breadcrumb-remove-non-interactive-item branch October 25, 2023 15:16
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 26, 2023
* master:
  chore: migrate to nx 16.10 (microsoft#29617)
  fix(react-drawer): accessibility tweaks (microsoft#29627)
  Fixing few legend colors not visible in dark mode (microsoft#29652)
  applying package updates
  Fixing dark mode bug for xAxis for Vertical Stacked bar chart (microsoft#29427)
  chore: Refactor RadioGroup stories to use Field, and remove individual storybook page for Radio (microsoft#29268)
  docs: fix description of SelectInitialValue story (microsoft#26493)
  chore: fix specs location (microsoft#29642)
  feat(scripts-task): implement initial verify-packaging task (microsoft#29595)
  feat(tools): replace npmignore with package.json#files for robust control over what we ship to npm (microsoft#29616)
  fix(workspace-plugin): fix storybook and vr-test issues wihtin prepare-initial-release generator (microsoft#29577)
  feat(react-breadcrumb): removed non-interactive item  (microsoft#29623)
  fix(react-motion): prevent from breaking the rule of hooks (microsoft#29628)
  fix(react-toast): invoke useCustomStyleHook_unstable() (microsoft#29632)
  feat: Deprecate Alert component (microsoft#29615)
  chore(react-infobutton, react-components): Deprecate InfoButton and InfoLabel from react-infobutton (microsoft#29605)
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