-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
CSVImport: Add hint about right-clicking the header rows #7127
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
base: master
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #7127 +/- ##
==========================================
- Coverage 88.77% 88.77% -0.01%
==========================================
Files 334 334
Lines 73671 73678 +7
==========================================
+ Hits 65402 65406 +4
- Misses 8269 8272 +3 🚀 New features to boost your workflow:
|
| cb_hint = QHBoxLayout() | ||
| cb_hint.addWidget(self.column_type_edit_cb) | ||
| hint = QLabel( | ||
| "Hint: right click on the row or column header for additional options.") |
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.
right click --> right-click
| separator: false | ||
| Ignore: Izpusti | ||
| The column will not be loaded: Stolpca ni mogoče naložiti | ||
| 'Hint: right click on the row or column header for additional options.': Namig: desni klik na glavo vrstice ali stolpca pokaže dodatne možnosti. |
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.
right click --> right-click
|
Frankly, I think the placement of the hint is somewhat strange. It also makes the widget window wider than it was. Perhaps on hover? |
|
It is not wider for me, but I agree that the placement is strange. There are two problems with tooltips, though. The first is that they do not show immediatelly. They are useful in contexts where the user wants more info and expects to get it by hovering over some element, but not for exposing unexpected functionality. The secon problem is - hover over what? |
Issue
Fixes #6788.
Description of changes
Adds a hint. If we like it. Otherwise, close the PR and the issue.
Includes