-
Notifications
You must be signed in to change notification settings - Fork 27.3k
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
ButtonBar RTL docs clarification #24264
Conversation
@HansMuller Let me know if this is a good place for this line, or if there is anything else I can add to clarify. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some wordsmithing.
@@ -12,7 +12,8 @@ import 'raised_button.dart'; | |||
/// A horizontal arrangement of buttons. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An end-aligned row of buttons.
@@ -12,7 +12,8 @@ import 'raised_button.dart'; | |||
/// A horizontal arrangement of buttons. | |||
/// | |||
/// Places the buttons horizontally according to the padding in the current | |||
/// [ButtonTheme]. | |||
/// [ButtonTheme]. Alignment and order of children are reversed between RTL and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might be best to be more specific here (since in one sense the Row's "alignment" doesn't change, it's always MainAxisAlignment.end).
The children are laid out in a [Row] with [MainAxisAlignment.end]. When the [Directionality] is [TextDirection.ltr] the button bar's children are right justified and the last child becomes the rightmost child. When the [Directionality] [TextDirection.rtl] the children are left justified and the last child becomes the leftmost child.
28ce249
to
6222fb9
Compare
@HansMuller Updated with your comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Closes #23217