Skip to content

Conversation

@mehmetf
Copy link
Contributor

@mehmetf mehmetf commented Aug 18, 2020

Reverts #63599

This was reverted because #63599, the ":" separator became vertically offset from the hours and minutes because of the addition of a nested Row widget that didn't specify the crossAxisAlignment appropriately. This ended up being caught by internal screenshot tests and so was reverted.

@flutter-dashboard flutter-dashboard bot added a: internationalization Supporting other languages or locales. (aka i18n) f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. labels Aug 18, 2020
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

@mehmetf mehmetf merged commit 7f122c7 into master Aug 18, 2020
@mehmetf mehmetf deleted the revert-63599-time-picker-input-rtl branch August 18, 2020 21:15
@goderbauer
Copy link
Member

Can you please include some information regarding why a PR was reverted in a revert?

@rami-a
Copy link
Contributor

rami-a commented Aug 19, 2020

Added a comment about why it was reverted in the description

@goderbauer
Copy link
Member

Thanks!

smadey pushed a commit to smadey/flutter that referenced this pull request Aug 27, 2020
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

a: internationalization Supporting other languages or locales. (aka i18n) 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.

4 participants