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(Dropdown): Freeform search should be case insensitive #24879

Merged

Conversation

jurokapsiar
Copy link
Contributor

Freeform search should not care about case sensitivity.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 21, 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 55309a9:

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

@size-auditor
Copy link

size-auditor bot commented Sep 21, 2022

Asset size changes

Project Bundle Baseline Size New Size Difference
office-ui-fabric-react fluentui-react-northstar-Dropdown 204.246 kB 204.462 kB ExceedsBaseline     216 bytes

ExceedsTolerance Over Tolerance (1024 B) ExceedsBaseline Over Baseline BelowBaseline Below Baseline New New Deleted  Removed 1 kB = 1000 B

Baseline commit: d6837b7a5af8838945316aff35870c525f0c3753 (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 21, 2022

📊 Bundle size report

🤖 This report was generated against d6837b7a5af8838945316aff35870c525f0c3753

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 21, 2022

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
ChatWithPopoverPerf.default 314 262 1.2:1
ButtonMinimalPerf.default 138 117 1.18:1
TreeWith60ListItems.default 136 116 1.17:1
GridMinimalPerf.default 276 238 1.16:1
TableMinimalPerf.default 331 285 1.16:1
CardMinimalPerf.default 444 389 1.14:1
ListMinimalPerf.default 420 368 1.14:1
StatusMinimalPerf.default 545 483 1.13:1
TableManyItemsPerf.default 1549 1372 1.13:1
RefMinimalPerf.default 167 153 1.09:1
ChatDuplicateMessagesPerf.default 231 213 1.08:1
CarouselMinimalPerf.default 379 353 1.07:1
BoxMinimalPerf.default 283 267 1.06:1
DropdownManyItemsPerf.default 550 518 1.06:1
PopupMinimalPerf.default 528 497 1.06:1
PortalMinimalPerf.default 146 138 1.06:1
AlertMinimalPerf.default 219 208 1.05:1
ButtonOverridesMissPerf.default 1030 984 1.05:1
LoaderMinimalPerf.default 560 531 1.05:1
SegmentMinimalPerf.default 282 268 1.05:1
CheckboxMinimalPerf.default 1714 1649 1.04:1
RosterPerf.default 1749 1679 1.04:1
ReactionMinimalPerf.default 309 298 1.04:1
TreeMinimalPerf.default 670 643 1.04:1
VideoMinimalPerf.default 605 584 1.04:1
InputMinimalPerf.default 926 901 1.03:1
ListWith60ListItems.default 511 503 1.02:1
SliderMinimalPerf.default 1310 1280 1.02:1
SplitButtonMinimalPerf.default 3505 3444 1.02:1
TooltipMinimalPerf.default 1937 1902 1.02:1
AvatarMinimalPerf.default 153 152 1.01:1
MenuMinimalPerf.default 697 692 1.01:1
RadioGroupMinimalPerf.default 365 361 1.01:1
AccordionMinimalPerf.default 112 112 1:1
ButtonSlotsPerf.default 451 451 1:1
DropdownMinimalPerf.default 2061 2059 1:1
HeaderSlotsPerf.default 604 605 1:1
ProviderMergeThemesPerf.default 941 942 1:1
AttachmentMinimalPerf.default 120 121 0.99:1
AttachmentSlotsPerf.default 878 891 0.99:1
DatepickerMinimalPerf.default 4585 4624 0.99:1
MenuButtonMinimalPerf.default 1383 1399 0.99:1
TextAreaMinimalPerf.default 391 393 0.99:1
HeaderMinimalPerf.default 288 294 0.98:1
ProviderMinimalPerf.default 333 339 0.98:1
ToolbarMinimalPerf.default 748 762 0.98:1
ChatMinimalPerf.default 577 597 0.97:1
FormMinimalPerf.default 264 271 0.97:1
TextMinimalPerf.default 265 274 0.97:1
EmbedMinimalPerf.default 2799 2921 0.96:1
ImageMinimalPerf.default 291 304 0.96:1
IconMinimalPerf.default 525 554 0.95:1
DialogMinimalPerf.default 581 624 0.93:1
ItemLayoutMinimalPerf.default 866 929 0.93:1
ListCommonPerf.default 470 505 0.93:1
CustomToolbarPrototype.default 2066 2222 0.93:1
AnimationMinimalPerf.default 405 439 0.92:1
ListNestedPerf.default 412 450 0.92:1
DividerMinimalPerf.default 257 281 0.91:1
LabelMinimalPerf.default 273 300 0.91:1
FlexMinimalPerf.default 210 234 0.9:1
SkeletonMinimalPerf.default 245 278 0.88:1
LayoutMinimalPerf.default 251 291 0.86:1

@jurokapsiar jurokapsiar merged commit 9dd5a3b into microsoft:master Oct 24, 2022
@jurokapsiar jurokapsiar deleted the v0-dropdown-freeform-insensitive branch October 24, 2022 12:26
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 25, 2022
* master: (106 commits)
  fix: PopoverTriggerChildProps should be exported (microsoft#25159)
  feat: replace ToolbarRadio implementation by usage of toggle button as Radio (microsoft#25343)
  docs: improve Toolbar docs examples (microsoft#25269)
  feat(tools): add unstable API setup updates (microsoft#25355)
  applying package updates
  Fix wrong narration when legend selected (microsoft#24903)
  applying package updates
  chore(react-persona): Update beachball settings and change file's type (microsoft#25363)
  chore: Refactor Field VR tests to have individual tests per component (microsoft#25263)
  chore(react-persona, react-components, vr-tests-v9): Reverting react-persona's version to beta   (microsoft#25357)
  Publishing migration package (microsoft#25354)
  fix: Detailslist is still tabbable when isHeaderVisible=false (microsoft#25342)
  fix: list even/odd off-by-one issue (microsoft#25358)
  feat: add Dropdown a11y spec (microsoft#24917)
  spinbutton: update internal padding for small size (microsoft#25286)
  chore(global-context): migrate to new package structure (microsoft#25341)
  feat: Add validationState to Progress, to make the bar red or green (microsoft#25253)
  feat: Add accessibility scenarios for Fluent UI v9 components #3 (microsoft#23334)
  feat(Dropdown): Freeform search should be case insensitive (microsoft#24879)
  feat(what-input): Limit keyboard detection in inputs (microsoft#25087)
  ...
NotWoods pushed a commit to NotWoods/fluentui that referenced this pull request Nov 18, 2022
…#24879)

* feat(Dropdown): Freeform search should be case insensitive

* fix problem with last letter

* changelog

Co-authored-by: Juraj Kapsiar <[email protected]>
@khmakoto khmakoto added Fluent UI react-northstar (v0) Work related to Fluent UI V0 and removed Fluent UI react-northstar labels Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fluent UI react-northstar (v0) Work related to Fluent UI V0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants