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

Textarea/hc bug #24701

Merged
merged 2 commits into from
Sep 8, 2022
Merged

Conversation

sopranopillow
Copy link
Contributor

Current Behavior

image

New Behavior

image

Note: when hovering and focused the border color is still blue, it's only when there's no state that it has that color.

Related Issue(s)

Fixes #24621

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 8, 2022

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

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

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-textarea
Textarea
23.732 kB
7.86 kB
23.946 kB
7.992 kB
214 B
132 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
189.372 kB
52.027 kB
react-components
react-components: FluentProvider & webLightTheme
33.317 kB
10.983 kB
🤖 This report was generated against 4c31b27421360e80d8435235092a634a67b0418a

@size-auditor
Copy link

size-auditor bot commented Sep 8, 2022

Asset size changes

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

Baseline commit: 4c31b27421360e80d8435235092a634a67b0418a (build)

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1581 1557 5000
Button mount 1144 1140 5000
FluentProvider mount 1902 1908 5000
FluentProviderWithTheme mount 765 749 10
FluentProviderWithTheme virtual-rerender 715 724 10
FluentProviderWithTheme virtual-rerender-with-unmount 737 744 10
MakeStyles mount 2236 2248 50000
SpinButton mount 2998 3068 5000

@sopranopillow sopranopillow merged commit 38bd1c1 into microsoft:master Sep 8, 2022
@sopranopillow sopranopillow deleted the textarea/hc-bug branch September 8, 2022 19:37
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Sep 9, 2022
* master: (63 commits)
  feat: add helper types to assist DOM element handling (microsoft#24722)
  applying package updates
  Textarea/hc bug (microsoft#24701)
  Fix null ref in use slider (microsoft#24728)
  Add Field unit tests, and remove snapshot tests (microsoft#24706)
  Stress Test: add build commands (microsoft#24575)
  Coachmark - useOffsetHeight should cause re-render on each set state to match v7 functionality (microsoft#24702)
  Implement screener tests for Field components (microsoft#24684)
  Update Field types to clean up react-field.api.md (microsoft#24703)
  fix(Popup): remove rotate(360deg) from PopupContent content styles (microsoft#24432)
  fix(FocusZone): should reset tabindex when focus is outside the zone with prop `shouldResetActiveElementWhenTabFromZone` (microsoft#24463)
  Fix greyed out legend key contrast ratio (microsoft#24714)
  fix: Portal compat should apply `focus-visible` ponyfill (microsoft#24712)
  Fix artifact error (microsoft#24717)
  chore(react-dialog): remove localShorthands in favor of griffel shorthands (microsoft#24715)
  Skip screener checks for draft PRs with exception of appropriately la… (microsoft#24694)
  fix: Remove provider classname from focus styles (microsoft#24710)
  feat: autocontrolled `useTable` hook (microsoft#24688)
  feat: add dialog properties to getNativeElementProps (microsoft#24698)
  Using migrate rather than upgrade term (microsoft#24695)
  ...
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.

[Bug]: Textarea Teams High Contrast theme shows hover outline color at rest
3 participants