Skip to content

fix(Timepicker-compat): clearIcon not working in freeform#31324

Merged
YuanboXue-Amber merged 5 commits intomicrosoft:masterfrom
YuanboXue-Amber:timepicker-clear
May 27, 2024
Merged

fix(Timepicker-compat): clearIcon not working in freeform#31324
YuanboXue-Amber merged 5 commits intomicrosoft:masterfrom
YuanboXue-Amber:timepicker-clear

Conversation

@YuanboXue-Amber
Copy link
Contributor

fix #31123

title says all.

@YuanboXue-Amber YuanboXue-Amber requested a review from a team as a code owner May 9, 2024 12:58
@codesandbox-ci
Copy link

codesandbox-ci bot commented May 9, 2024

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.

@fabricteam
Copy link
Collaborator

fabricteam commented May 9, 2024

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-timepicker-compat
TimePicker
103.294 kB
34.527 kB
103.487 kB
34.632 kB
193 B
105 B

🤖 This report was generated against 3be221900e3c3898dc34570c5f4d14a224bc2d01

@YuanboXue-Amber
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@YuanboXue-Amber YuanboXue-Amber enabled auto-merge (squash) May 27, 2024 14:20
@YuanboXue-Amber YuanboXue-Amber merged commit a96863b into microsoft:master May 27, 2024
@@ -0,0 +1,7 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

🕵 fluentuiv9 Open the Visual Regressions report to inspect the affected screenshots

Avatar Converged 1 screenshots
Image Name Diff(in Pixels) Image Type
Avatar Converged.badgeMask.normal.chromium.png 2 Changed
Card Converged - Selectable 8 screenshots
Image Name Diff(in Pixels) Image Type
Card Converged - Selectable.appearance focusable + selectable.click.chromium.png 0 Added
Card Converged - Selectable.appearance focusable + selectable.hover.chromium.png 0 Added
Card Converged - Selectable.appearance focusable + selectable.normal.chromium.png 0 Added
Card Converged - Selectable.appearance focusable + selectable.selected.chromium.png 0 Added
Card Converged - Selectable.appearance selectable - Filled.click.chromium.png 0 Added
Card Converged - Selectable.appearance selectable - Filled.hover.chromium.png 0 Added
Card Converged - Selectable.appearance selectable - Filled.normal.chromium.png 0 Added
Card Converged - Selectable.appearance selectable - Filled.selected.chromium.png 0 Added

marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request May 28, 2024
* master: (49 commits)
  Update focus order in sankey chart for vertical navigation (microsoft#31469)
  chore: use new performant 'type-check' for v9 libaries (microsoft#31454)
  applying package updates
  fix(Timepicker-compat): clearIcon not working in freeform (microsoft#31324)
  chore: re-enable lint rule (microsoft#31459)
  feat(react-tag-picker): adds text property to TagPickerOption (microsoft#31474)
  feat(recipes): create package with initial implementation (moved from /apps) (microsoft#31386)
  applying package updates
  applying package updates
  chore: revert globals changes (microsoft#31470)
  (web-components) Use `ElementInternals` for TextInput elements (microsoft#31201)
  chore:(docs) Update and migrate component implementation guide (microsoft#31398)
  disallow all globals in Fluent v9 (microsoft#30967)
  chore:(react-nav-preview) Recomposing more components and some pixel pushing (microsoft#31387)
  fix(pr-deploy-site): explicitly set types to not include whole @types/* globals which are causing issues with addition of @types/web (microsoft#31465)
  fix(recipes-react-components): explicitly set types to not include whole @types/* globals which are cauising issues with addition of @types/web (microsoft#31463)
  applying package updates
  applying package updates
  applying package updates
  Update IconDirectionContextProvider import to import from specific path (microsoft#31006)
  ...
miroslavstastny pushed a commit to miroslavstastny/fluentui that referenced this pull request Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: TimePicker - clear button doesn't work when freeform and clearable are enabled

3 participants