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 for positioning logic land on fractional values #15469

Closed
wants to merge 1 commit into from
Closed

fix for positioning logic land on fractional values #15469

wants to merge 1 commit into from

Conversation

dev-a11y
Copy link

@dev-a11y dev-a11y commented Oct 12, 2020

fractional values in this place cause hard to debug subpixel rendering problems

Pull request checklist

Description of changes

see discussion in related issue

fractional values in this place cause hard to debug subpixel rendering problems
@msft-github-bot msft-github-bot added Component: ContextualMenu needs cherry-pick Temporary label for PRs which may need to be cherry-picked to master labels Oct 12, 2020
@codesandbox-ci
Copy link

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

Sandbox Source
Fluent UI Button Configuration
microsoft/fluentui: codesandbox-react-template Configuration
microsoft/fluentui: codesandbox-react-next-template Configuration
microsoft/fluentui: codesandbox-react-northstar-template Configuration

@msft-github-bot
Copy link
Contributor

msft-github-bot commented Oct 12, 2020

Perf Analysis

No significant results to display.

All results

Scenario Render type 7.0 Ticks PR Ticks Iterations Status
BaseButton mount 923 930 5000
Breadcrumb mount 39595 39408 5000
Checkbox mount 1589 1566 5000
CheckboxBase mount 1315 1310 5000
ChoiceGroup mount 5144 5015 5000
ComboBox mount 930 887 1000
CommandBar mount 7399 7403 1000
ContextualMenu mount 12895 12644 1000
DefaultButton mount 1187 1173 5000
DetailsRow mount 3762 3704 5000
DetailsRowFast mount 3602 3582 5000
DetailsRowNoStyles mount 3397 3418 5000
Dialog mount 1440 1518 1000
DocumentCardTitle mount 1779 1749 1000
Dropdown mount 2579 2598 5000
FocusTrapZone mount 1679 1672 5000
FocusZone mount 1749 1767 5000
IconButton mount 1756 1769 5000
Label mount 324 327 5000
Layer mount 1907 1973 5000
Link mount 437 433 5000
MenuButton mount 1483 1502 5000
MessageBar mount 2026 2023 5000
Nav mount 3230 3257 1000
OverflowSet mount 1382 1420 5000
Panel mount 1433 1432 1000
Persona mount 843 835 1000
Pivot mount 1411 1423 1000
PrimaryButton mount 1259 1270 5000
Rating mount 7840 7818 5000
SearchBox mount 1306 1292 5000
Shimmer mount 2687 2588 5000
Slider mount 1492 1479 5000
SpinButton mount 5002 5012 5000
Spinner mount 407 402 5000
SplitButton mount 3167 3191 5000
Stack mount 509 522 5000
StackWithIntrinsicChildren mount 1593 1594 5000
StackWithTextChildren mount 4865 4845 5000
SwatchColorPicker mount 10340 10293 5000
TagPicker mount 2960 2930 5000
TeachingBubble mount 50528 50632 5000
Text mount 440 434 5000
TextField mount 1405 1472 5000
Toggle mount 879 834 5000
button mount 108 104 5000

@size-auditor
Copy link

size-auditor bot commented Oct 12, 2020

Asset size changes

Project Bundle Baseline Size New Size Difference
office-ui-fabric-react office-ui-fabric-react-HoverCard 96.845 kB 96.905 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-ComboBox 239.426 kB 239.486 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-ContextualMenu 152.641 kB 152.701 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-DatePicker 204.749 kB 204.809 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Grid 175.165 kB 175.225 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Positioning 19.217 kB 19.277 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Pivot 180.368 kB 180.428 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Pickers 277.233 kB 277.293 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-PersonaCoin 111.843 kB 111.903 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Persona 111.843 kB 111.903 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Panel 194.98 kB 195.04 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Dialog 204.63 kB 204.69 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Nav 182.67 kB 182.73 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-DocumentCard 209.353 kB 209.413 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-MessageBar 183.568 kB 183.628 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Dropdown 226.696 kB 226.756 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Facepile 204.152 kB 204.212 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Keytips 102.181 kB 102.241 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-KeytipLayer 99.07 kB 99.13 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-FloatingPicker 233.934 kB 233.994 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Keytip 78.693 kB 78.753 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-CommandBar 195.809 kB 195.869 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-PositioningContainer 71.176 kB 71.236 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-SearchBox 181.481 kB 181.541 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-SpinButton 187.502 kB 187.562 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Tooltip 84.011 kB 84.071 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Breadcrumb 193.468 kB 193.528 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Button 188.101 kB 188.161 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-TeachingBubble 199.651 kB 199.711 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-SwatchColorPicker 185.551 kB 185.611 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-ButtonGrid 175.165 kB 175.225 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-SelectedItemsList 223.282 kB 223.342 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Coachmark 92.708 kB 92.768 kB ExceedsBaseline     60 bytes
office-ui-fabric-react office-ui-fabric-react-Callout 84.209 kB 84.267 kB ExceedsBaseline     58 bytes

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

Baseline commit: 25a4e0e7783de5d8f733ca85a762c2308823438e (build)

@ecraig12345 ecraig12345 requested a review from joschect October 12, 2020 18:43
@paulgildea paulgildea added the Contribution Candidate This issue is a good opportunity to contribute to Fluent UI label Oct 21, 2020
@khmakoto
Copy link
Member

Now that we're publishing beta versions of 8.0 please also port this to the master branch once it merges.

@joschect joschect closed this Dec 17, 2020
@joschect joschect reopened this Dec 17, 2020
@layershifter layershifter reopened this Jan 5, 2021
@layershifter
Copy link
Member

@dev-a11y can you please generate a changelog entry via yarn change command? 🐱

@dev-a11y
Copy link
Author

dev-a11y commented Jan 7, 2021

@layershifter, sorry - had some difficulties setting up env on my machine, latest build error was sth with electron package installation and I gave it a rest; please reject this pull request and if still valid that code adjustment can be pushed by someone else; I'll remove my branch/fork soon

@dev-a11y dev-a11y closed this Jan 7, 2021
@dev-a11y dev-a11y deleted the dev-a11y-patch-positioning-fractional-values-v7 branch January 7, 2021 10:16
@ecraig12345 ecraig12345 removed the needs cherry-pick Temporary label for PRs which may need to be cherry-picked to master label Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: ContextualMenu Contribution Candidate This issue is a good opportunity to contribute to Fluent UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants