Bug 344644

Summary:	Table View or Table Widget do not indicate that a cell is current
Product:	[Plasma] Breeze	Reporter:	Jarosław Staniek <staniek>
Component:	QStyle	Assignee:	Hugo Pereira Da Costa <hugo.pereira.da.costa>
Status:	RESOLVED FIXED
Severity:	normal	CC:	hugo.pereira.da.costa, noahadvs
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	oxygen style breeze style table widget ui for test

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.