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

feat(react-textarea): Add shadow variant of filled appearance #24512

Merged

Conversation

sopranopillow
Copy link
Contributor

Current Behavior

There are no shadows applied to Textarea.

New Behavior

Textarea now has filled-shadow variants.

Note: to see the shadow, you might need to change the background to a lighter color.

image

@fabricteam
Copy link
Collaborator

fabricteam commented Aug 24, 2022

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-textarea
Textarea
23.988 kB
8.011 kB
25.013 kB
8.133 kB
1.025 kB
122 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
187.939 kB
52.05 kB
react-components
react-components: FluentProvider & webLightTheme
33.394 kB
11.007 kB
react-portal-compat
PortalCompatProvider
5.851 kB
1.964 kB
🤖 This report was generated against 5c78472d96502d05a10016d9502eceada4edfd8a

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 24, 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 d2b7cfa:

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

@size-auditor
Copy link

size-auditor bot commented Aug 24, 2022

Asset size changes

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

Baseline commit: 5c78472d96502d05a10016d9502eceada4edfd8a (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Aug 24, 2022

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1286 2233 5000 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1286 2233 5000 Possible regression
Button mount 957 957 5000
FluentProvider mount 1615 1619 5000
FluentProviderWithTheme mount 633 634 10
FluentProviderWithTheme virtual-rerender 601 600 10
FluentProviderWithTheme virtual-rerender-with-unmount 632 628 10
MakeStyles mount 1918 1906 50000
SpinButton mount 2603 2512 5000

Copy link
Contributor

@spmonahan spmonahan left a comment

Choose a reason for hiding this comment

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

Just blocking this so we can review the API with the larger team.

@sopranopillow sopranopillow merged commit 151520d into microsoft:master Sep 15, 2022
@sopranopillow sopranopillow deleted the input/add-shadows-textarea branch September 15, 2022 20:01
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Sep 19, 2022
* master: (28 commits)
  fix: use trigger prop for aria-haspopup (microsoft#24794)
  chore(react-dialog): scaffold DialogContent component (microsoft#24844)
  chore: Northstar screener should read from screenerStates.json (microsoft#24848)
  applying package updates
  (web components) Standardize focus treatment (microsoft#24771)
  Divider - allow default prop override (microsoft#24840)
  GroupedList: fix virtualization (unstable preview) (microsoft#24460)
  fix: Add explicit children prop to TeachingBubble to support React 18 (microsoft#24823)
  feat: Adds `visible` prop to `TableCellActions` (microsoft#24831)
  [Northstar][Dropdown] Fix styling mutation when merging themes (microsoft#24787)
  fix: export `tableCellActionsClassNames` from unstable (microsoft#24830)
  bugfix(react-dialog): Adds color style to DialogSurface (microsoft#24832)
  applying package updates
  Prevent group toggling from selecting the whole group (microsoft#24822)
  feat(react-textarea): Add shadow variant of filled appearance (microsoft#24512)
  applying package updates
  Adding lib-commonjs top-level entries to exports map (microsoft#24792)
  Created shim packages (microsoft#24780)
  feat(react-menu): replace keydown handlers by useARIAButtonShorthand on MenuItem (microsoft#24738)
  fix: update version mismatches triggered by v9 release (microsoft#24812)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants