-
-
Notifications
You must be signed in to change notification settings - Fork 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
Adds missing tooltips #5773
Adds missing tooltips #5773
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.
Nice to have this tooltips!
@@ -1798,6 +1831,9 @@ | |||
</item> | |||
<item row="1" column="0"> | |||
<widget class="QToolButton" name="mAddLayerRestrictionButton"> | |||
<property name="toolTip"> | |||
<string>Add layer to exclude</string> |
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.
I don't know the context, but maybe clearer Add layer to the exclude list
or ... to the list of excluded layers
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.
You are right. I think it was a mistake.
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.
Can you update the PR so we can merge?
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.
I will. But I can only do it on Monday.
this is the context for those tool tips. Do you think they still need changes? "Add layers to exclude" and "Add composer to exclude" |
@SrNetoChan no, probably it's fine as it is |
Description
This PR adds some missing toolTips in some dialog's buttons.
Tooltips are useful for users to understand what the button will do, but also for documentation. It's easier to expalin the user to "click the Add color button" than "click the button with the plus sign".
I have also found some wrong tool tips in the metadata widget, but I will try to address that in a different PR.
Checklist
fixes #11111
in the commit message next to the description[FEATURE]
in the commit message[needs-docs]
in the commit message and containt sufficient information in the commit message to be documentedscripts/prepare-commit.sh
script before each commit