-
Notifications
You must be signed in to change notification settings - Fork 28.2k
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
Add documentation for the secondary button #164555
Add documentation for the secondary button #164555
Conversation
@TahaTesser can you please check the pr u.u I would love to make more improvements if needed. |
/// The secondary button is typically accessed using the right mouse | ||
/// button on desktop platforms such as Windows, macOS, and Linux. |
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.
/// The secondary button is typically accessed using the right mouse | |
/// button on desktop platforms such as Windows, macOS, and Linux. | |
/// The secondary button is typically accessed using the right mouse | |
/// button on desktop platforms such as Windows, macOS, and Linux. |
If it's a mouse button, it does't necessarily only for Windows, macOS, and Linux. You can connect mouse to other devices.
According to some Google search, the secondary button is the right mouse button for right-handed users and left mouse button for the left-handed users.
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.
Thanks for suggestion! I'm sending changes
/// | ||
/// The secondary button is a mouse button that can be configured | ||
/// based on the user's system settings. By default, it is the | ||
/// right button for right-handed users and the left button for | ||
/// left-handed users. |
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.
The secondary button is an abstraction not specific to mouse.
See https://api.flutter.dev/flutter/gestures/kSecondaryButton-constant.html which explains how it maps to stylus and mouse.
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.
Thanks for suggestion. To avoid own reinterpretation of the secondary button, I'm sending new documentation that references kSecondaryButton in a similar way to how the GestureDetector docs do.
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!
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
In this PR, I've added extra comments about what the Secondary Button is, and in what cases it can be activated. Issue #123049
Pre-launch Checklist
///
).