Bug 172324

Summary: [REQ] Bigger Textboxes in Decoding Table
Product: [Applications] okteta Reporter: Benjamin Hessenauer <hessenauer>
Component: generalAssignee: Friedrich W. H. Kossebau <kossebau>
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: okteta application window
okteta application window

Description Benjamin Hessenauer 2008-10-07 11:12:50 UTC
Version:           0.1.1 (using KDE 4.1.2)
OS:                Linux
Installed from:    Ubuntu Packages

I would like to have bigger Textboxes in the Decoding Table.

Right now, you have to click in the textbox and use the arrow keys to see the whole number, especially 32-bit float and 64-bit float. That is very annoying if you want to check a lot of numbers.

Maybe you could expand the textboxes depending on the width of the okteta application window?
Comment 1 Friedrich W. H. Kossebau 2008-10-12 01:35:18 UTC
Hi, thanks for the report. Should not have happened, seems guessing the size needed with a placeholder XXXXXXXXXXXX fails for you. Will increase the placeholder then for 4.1.3.

For KDE 4.2 Okteta is receiving a new version of the decoding table, a simple two-column table (type|value), where the width of the values depends on the width of the toolview. This allows to support more datatypes, for now I added 64bit ints. Any suggestions for other useful types?
Comment 2 Friedrich W. H. Kossebau 2008-10-12 01:43:25 UTC
Benjamin, could you perhaps tell how many chars you need to scroll? What font are you using? The letter X should be as wide as the digits usually... Little screenshot of the decoder table would be useful, too :)
Comment 3 Benjamin Hessenauer 2008-10-13 13:24:16 UTC
Created attachment 27849 [details]
okteta application window
Comment 4 Benjamin Hessenauer 2008-10-13 13:51:03 UTC
I am using the standard Kubuntu font "DejaVU Sans", font size 7!

Scrolling:

32 and 64bit float:       4 chars
unsigned 32 bit integer:  1 char
signed 32 bit integer:    2 chars  (Only if the integers are big, of course)

Comment 5 Benjamin Hessenauer 2008-10-13 13:52:51 UTC
I dont know how to delete attachments, but the second one actually shows a meaningful 32 bit float.
Comment 6 Benjamin Hessenauer 2008-10-13 13:55:58 UTC
Sorry,

DejaVU Sans Mono for fixed-width font
Comment 7 Friedrich W. H. Kossebau 2008-10-14 16:48:38 UTC
SVN commit 871326 by kossebau:

fixed: calculation of the width of the value fields was too primitive

QLineEdit should get a function "QSize sizeFor( const QString & ) const" or similar.
And having to set QStyleOptionFrameV2 manually oneself sucks a little, too.
First time in years I am unsatisfied with Qt, oh dear. I still guess I overlooked something ;)

CCBUG: 172324




 M  +39 -7     kprimitivetypesview.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=871326