Skip to content

Remove private OutlinedButton default outline geometry class#70975

Merged
HansMuller merged 1 commit intoflutter:masterfrom
HansMuller:remove_private_outline_from_outlined_button
Nov 23, 2020
Merged

Remove private OutlinedButton default outline geometry class#70975
HansMuller merged 1 commit intoflutter:masterfrom
HansMuller:remove_private_outline_from_outlined_button

Conversation

@HansMuller
Copy link
Contributor

@HansMuller HansMuller commented Nov 21, 2020

Remove OutlinedButton's private _Outline RoundedRectangleBorder subclass.

It was added in #70393 to satisfy a backwards incompatibility (a one pixel difference) in the way the default outline was rendered. This is no longer necessary as of (internal Google change) cl/339303032.

@flutter-dashboard flutter-dashboard bot added f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. labels Nov 21, 2020
@google-cla google-cla bot added the cla: yes label Nov 21, 2020
@HansMuller HansMuller requested a review from rami-a November 22, 2020 01:19
Copy link
Contributor

@rami-a rami-a left a comment

Choose a reason for hiding this comment

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

LGTM

@HansMuller
Copy link
Contributor Author

The problems reported by "Google Testing" were fixed in (internal Google change) cl/343891652.

@HansMuller HansMuller merged commit 89a7bf7 into flutter:master Nov 23, 2020
@HansMuller HansMuller deleted the remove_private_outline_from_outlined_button branch November 23, 2020 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants