Bug 344644 - Table View or Table Widget do not indicate that a cell is current
Summary: Table View or Table Widget do not indicate that a cell is current
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: QStyle (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Hugo Pereira Da Costa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-27 23:49 UTC by Jarosław Staniek
Modified: 2019-09-23 02:06 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
oxygen style (5.68 KB, image/png)
2015-02-27 23:50 UTC, Jarosław Staniek
Details
breeze style (5.10 KB, image/png)
2015-02-27 23:50 UTC, Jarosław Staniek
Details
table widget ui for test (1.35 KB, application/x-designer)
2015-02-27 23:51 UTC, Jarosław Staniek
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jarosław Staniek 2015-02-27 23:49:15 UTC
Table View or Table Widget with selectionMode == NoSelection do not indicate that a cell is current. In Oxygen and other styles current cell was clearly indicated.



Reproducible: Always



Expected Results:  
"Current" indicated.

Qt 4 style 5.2.90. See the attachments for oxygen and breeze. Also attached a test .ui file.
Comment 1 Jarosław Staniek 2015-02-27 23:50:04 UTC
Created attachment 91334 [details]
oxygen style
Comment 2 Jarosław Staniek 2015-02-27 23:50:28 UTC
Created attachment 91335 [details]
breeze style
Comment 3 Jarosław Staniek 2015-02-27 23:51:09 UTC
Created attachment 91336 [details]
table widget ui for test
Comment 4 Hugo Pereira Da Costa 2015-02-28 16:18:28 UTC
on it.
I know exactly what's missing :)
Thanks for reporting.
Comment 5 Jarosław Staniek 2015-04-27 22:28:02 UTC
Hello Hugo. What's new with this? 

Still the same issue https://bugsfiles.kde.org/attachment.cgi?id=91335
Comment 6 Jarosław Staniek 2015-05-02 14:39:42 UTC
Git commit ac5d0edf054abd153669ffbd52e0218f5493a650 by Jaroslaw Staniek.
Committed on 02/05/2015 at 14:32.
Pushed by staniek into branch 'calligra/2.9'.

Usability improvements in Kexi CSV import

- Enable single selection of table cells ("current" cell indicator is not visible in Breeze style)
- Display font of edited column names in bold also during editing
- Allow editing only first row (column names) but also when "First row contains column names" is OFF; this let's the user to fully customize column names before importing
- Use "Column <n>" column caption and "column_<n>" column name for empty names.

REVIEW:123549

M  +15   -18   kexi/plugins/importexport/csv/KexiCSVImportDialogModel.cpp
M  +2    -2    kexi/plugins/importexport/csv/KexiCSVImportDialogModel.h
M  +59   -14   kexi/plugins/importexport/csv/kexicsvimportdialog.cpp
M  +3    -1    kexi/plugins/importexport/csv/kexicsvimportdialog.h

http://commits.kde.org/calligra/ac5d0edf054abd153669ffbd52e0218f5493a650
Comment 7 Noah Davis 2019-09-23 02:06:24 UTC
This seems to be fixed in the current version of Breeze.