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

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

Merged
merged 5 commits into from
May 27, 2024

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
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

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
17 of 19 checks passed
@@ -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)
  ...
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