Bug 172324 - [REQ] Bigger Textboxes in Decoding Table
Summary: [REQ] Bigger Textboxes in Decoding Table
Status: RESOLVED FIXED
Alias: None
Product: okteta
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR wishlist
Target Milestone: ---
Assignee: Friedrich W. H. Kossebau
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-07 11:12 UTC by Benjamin Hessenauer
Modified: 2008-10-14 16:48 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
okteta application window (121.82 KB, image/png)
2008-10-13 13:24 UTC, Benjamin Hessenauer
Details
okteta application window (121.98 KB, image/png)
2008-10-13 13:26 UTC, Benjamin Hessenauer
Details

Note You need to log in before you can comment on or make changes to this bug.
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