Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

chore(List): move to hooks #2267

Merged
merged 20 commits into from
Jan 28, 2020
Merged

chore(List): move to hooks #2267

merged 20 commits into from
Jan 28, 2020

Conversation

layershifter
Copy link
Member

@layershifter layershifter commented Jan 24, 2020

Extracted from #2207.

BREAKING CHANGES

Only limited amount of props are passed to component's styles functions.

List
debug
horizontal
isListTag
ListItem
debug
important
navigable
selectable
selected
truncateContent
truncateHeader
hasContent
hasContentMedia
hasHeader
hasHeaderMedia

@DustyTheBot
Copy link
Collaborator

DustyTheBot commented Jan 24, 2020

Perf comparison

Status Scenario Fluent TPI Fabric TPI Ratio Iterations Ticks
🔧 Avatar.Fluent 0.54 0.37 1.46:1 2000 1071
🔧 Button.Fluent 1.19 0.17 7:1 1000 1188
🔧 Checkbox.Fluent 1.41 0.29 4.86:1 1000 1412
🔧 Dialog.Fluent 0.32 0.17 1.88:1 5000 1587
🔧 Dropdown.Fluent 3.71 0.36 10.31:1 1000 3713
🔧 Icon.Fluent 0.23 0.04 5.75:1 5000 1158
🔧 Image.Fluent 0.1 0.07 1.43:1 5000 499
🔧 Slider.Fluent 1.95 0.3 6.5:1 1000 1950
🦄 Text.Fluent 0.06 0.17 0.35:1 5000 320
🦄 Tooltip.Fluent 0.34 18.03 0.02:1 5000 1719

🔧 Needs work     🎯 On target     🦄 Amazing

Generated by 🚫 dangerJS

@layershifter layershifter changed the title [WIP] chore(List): move to hooks chore(List): move to hooks Jan 27, 2020
Copy link
Contributor

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

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

Love it <3

@layershifter layershifter merged commit c488504 into master Jan 28, 2020
@layershifter layershifter deleted the chore/list-to-hooks branch January 28, 2020 16:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants