qlog: Column width miscalculated on Vista

Bug #430502 reported by Greg
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
QBzr
Fix Released
High
Alexander Belchenko

Bug Description

The Log window does not appear to save the column width or they are miscalculated.
This can be a usability hit as the Date column must be manually resized each time.
It does appear to save the window size, though not the window position.

Please look at the attached screen shot to see this demonstrated.

System:
Windows 7 64
BZR Windows build v1.18.1
QBZR About says 'Version 0.13.1.final.0 (bzrlib 1.18.1)'

Related branches

Revision history for this message
Greg (gregspecialsource) wrote :
Revision history for this message
Alexander Belchenko (bialix) wrote :

Maybe we can save it.

Changed in qbzr:
importance: Undecided → Wishlist
status: New → Confirmed
Revision history for this message
Gary van der Merwe (garyvdm) wrote :

I think I have fixed the calculation of the width. Greg: if you can run from source, please will you test trunk, and verify if it is fixed.

Changed in qbzr:
assignee: nobody → Gary van der Merwe (garyvdm)
status: Confirmed → Fix Committed
Revision history for this message
Greg (gregspecialsource) wrote :

I grabbed (what I believe is the intended version) with 'bzr branch lp:qbzr' and rebooted.
qbzr is still listed as v0.15dev
I see no change in the column behavior. The date is still truncated and manual adjustments are not saved.

Revision history for this message
Greg (gregspecialsource) wrote :

Having said that, if the mouse drag behavior is accurate, the Date column is only 1 pixel short.

Changed in qbzr:
importance: Wishlist → Low
summary: - Log window Column Width not saved or miscalculated
+ qlog: Column width miscalculated on Vista
Changed in qbzr:
importance: Low → High
Changed in qbzr:
status: Fix Committed → Confirmed
Revision history for this message
Alexander Belchenko (bialix) wrote :

What's the status of this bug? Greg, can you test with latest qbzr trunk?

If we need just made it 1 pixel wider, let's just do it.

Revision history for this message
Greg (gregspecialsource) wrote :

Sorry, I have not updated qbzr recently. I'm also away at present and won't be able to provide any feedback until I return. If possible, please find another Windows user to confirm the status of this bug. Again, I'm sorry I can't assist for a while.

Revision history for this message
Greg (gregspecialsource) wrote :

qbzr v0.17.0 still exhibits this issue. Manually resizing the column by 1 pixel displays the full date and time without truncation.

Revision history for this message
André Bachmann (andrebachmann-dd) wrote :

I still can reproduce this problem with qbzr 0.21.0dev2 on Win7. The column of data is exactly one pixel too narrow. I need a little hint about where the column width is saved, because it isn't in qlog.py. Then I will try to fix this.

Revision history for this message
Alexander Belchenko (bialix) wrote :

logwidget.py: LogList: __init__ has the code:

        header.resizeSection(logmodel.COL_DATE,
                             fm.width("88-88-8888 88:88") + col_margin)

I believe that's the place where we calculate the width of date column.

Changed in qbzr:
assignee: Gary van der Merwe (garyvdm) → Alexander Belchenko (bialix)
status: Confirmed → In Progress
milestone: none → 0.23.1
Revision history for this message
Alexander Belchenko (bialix) wrote :

Increasing the second argument in the call above does help. I'm not sure what's wrong here.

Revision history for this message
Alexander Belchenko (bialix) wrote :

I've just added extra space to sample string for which fm.width is computed. Hopefully that won't be a big problem on other platforms. Will see.

Changed in qbzr:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.