Skip to content

Conversation

@khmakoto
Copy link
Member

@khmakoto khmakoto commented Jan 26, 2021

Pull request checklist

Description of changes

Port of #16625 and #16721

This PR adds an underline prop that provider underline styling to the Link component. This is done to add a styling difference in places where color is not enough contrast to differentiate the Link (i.e. cases where the Link is alongside other text).

@khmakoto khmakoto removed the needs cherry-pick Temporary label for PRs which may need to be cherry-picked to master label Jan 26, 2021
@fabricteam
Copy link
Collaborator

fabricteam commented Jan 26, 2021

Perf Analysis

No significant results to display.

All results

Scenario Render type 7.0 Ticks PR Ticks Iterations Status
BaseButton mount 892 869 5000
Breadcrumb mount 42978 42798 5000
Checkbox mount 1544 1522 5000
CheckboxBase mount 1274 1292 5000
ChoiceGroup mount 4892 4934 5000
ComboBox mount 935 913 1000
CommandBar mount 7803 7773 1000
ContextualMenu mount 14773 15193 1000
DefaultButton mount 1085 1115 5000
DetailsRow mount 3540 3556 5000
DetailsRowFast mount 3598 3542 5000
DetailsRowNoStyles mount 3344 3412 5000
Dialog mount 1656 1681 1000
DocumentCardTitle mount 1824 1821 1000
Dropdown mount 2527 2522 5000
FocusTrapZone mount 1693 1721 5000
FocusZone mount 1805 1770 5000
IconButton mount 1767 1763 5000
Label mount 334 329 5000
Layer mount 1933 1930 5000
Link mount 434 455 5000
MenuButton mount 1451 1446 5000
MessageBar mount 2080 2063 5000
Nav mount 3223 3260 1000
OverflowSet mount 1416 1415 5000
Panel mount 1445 1454 1000
Persona mount 781 774 1000
Pivot mount 1411 1405 1000
PrimaryButton mount 1295 1268 5000
Rating mount 7535 7473 5000
SearchBox mount 1261 1282 5000
Shimmer mount 2511 2496 5000
Slider mount 1466 1467 5000
SpinButton mount 4987 4945 5000
Spinner mount 406 418 5000
SplitButton mount 3231 3155 5000
Stack mount 495 483 5000
StackWithIntrinsicChildren mount 1546 1553 5000
StackWithTextChildren mount 4603 4656 5000
SwatchColorPicker mount 10150 10217 5000
TagPicker mount 2782 2788 5000
TeachingBubble mount 51321 51161 5000
Text mount 420 422 5000
TextField mount 1339 1363 5000
Toggle mount 806 797 5000
button mount 112 99 5000

@size-auditor
Copy link

size-auditor bot commented Jan 26, 2021

Asset size changes

Project Bundle Baseline Size New Size Difference
office-ui-fabric-react office-ui-fabric-react-DetailsList 215.821 kB 215.905 kB ExceedsBaseline     84 bytes
office-ui-fabric-react office-ui-fabric-react-Breadcrumb 193.931 kB 194.015 kB ExceedsBaseline     84 bytes
office-ui-fabric-react office-ui-fabric-react-ShimmeredDetailsList 226.279 kB 226.363 kB ExceedsBaseline     84 bytes
office-ui-fabric-react office-ui-fabric-react-GroupedList 122.067 kB 122.151 kB ExceedsBaseline     84 bytes
office-ui-fabric-react office-ui-fabric-react-DocumentCard 209.969 kB 210.053 kB ExceedsBaseline     84 bytes
office-ui-fabric-react office-ui-fabric-react-Link 43.45 kB 43.534 kB ExceedsBaseline     84 bytes

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

Baseline commit: 04e14bd18f4ba6f8daac631217643362c9dedd44 (build)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 26, 2021

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 02f4156:

Sandbox Source
Fluent UI Button Configuration
codesandbox-react-template Configuration

@msft-fluent-ui-bot
Copy link
Collaborator

Hello @khmakoto!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msft-fluent-ui-bot) and give me an instruction to get started! Learn more here.

@msft-fluent-ui-bot msft-fluent-ui-bot merged commit 49d501e into microsoft:7.0 Feb 18, 2021
@khmakoto khmakoto deleted the linkUnderline7.0 branch February 18, 2021 20:43
@msft-fluent-ui-bot
Copy link
Collaborator

🎉@uifabric/fabric-website@v7.16.30 has been released which incorporates this pull request.:tada:

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

🎉@uifabric/example-app-base@v7.17.0 has been released which incorporates this pull request.:tada:

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

🎉@fluentui/react-examples@v0.11.29 has been released which incorporates this pull request.:tada:

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

🎉office-ui-fabric-react@v7.161.0 has been released which incorporates this pull request.:tada:

Handy links:

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.

5 participants