Skip to content

Conversation

janezd
Copy link
Contributor

@janezd janezd commented Jul 13, 2025

Issue

Fixes #6788.

Description of changes

Adds a hint. If we like it. Otherwise, close the PR and the issue.

Includes
  • Code changes

Copy link

codecov bot commented Jul 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.77%. Comparing base (5eb97b8) to head (f78e039).
⚠️ Report is 14 commits behind head on master.

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:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.")
Copy link
Contributor

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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

right click --> right-click

@ajdapretnar
Copy link
Contributor

Frankly, I think the placement of the hint is somewhat strange. It also makes the widget window wider than it was. Perhaps on hover?

@janezd janezd added the needs discussion Core developers need to discuss the issue label Jul 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs discussion Core developers need to discuss the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File widget: read file without header
2 participants