-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
GeneralPane: Add BalloonTips #12829
base: master
Are you sure you want to change the base?
GeneralPane: Add BalloonTips #12829
Conversation
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.
This looks so good to me, apart from the minor nitpick I left earlier. Gave it a try and didn't notice anything wrong design-wise, all tooltips showed as expected...
034d630
to
729ecbb
Compare
I've added the suggestions above. |
@MayImilae Do you want to review those description strings? |
I need too, I've just been busy. I'll leave this tab open and get to it tomorrow. EDIT: ...oh yea I have notes. I'm busy with the you know what but I'll write up a bunch of notes in the coming couple of days. |
I aimed to make the tooltips accurate and detailed, but encourage double-checking.