amount/total column is not always large enough

Bug #324435 reported by Michael Rooney
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
wxBanker
Fix Released
Low
Unassigned

Bug Description

The amount column (and I assume total as well) doesn't properly auto-size so large amounts get cut off. Perhaps it should be as wide as the largest item?

Related branches

Michael Rooney (mrooney)
Changed in wxbanker:
importance: Undecided → Low
status: New → Confirmed
Michael Rooney (mrooney)
Changed in wxbanker:
milestone: none → 0.9
Revision history for this message
Michael Rooney (mrooney) wrote :

Yes, it should be simple enough to make this as wide as the longest entry. Getting text extents of thousands of transactions could be slow, so we should make a guess and sort transactions by highest amount, then use the text extend of the largest value. The negative sign also takes up room in every currency that I know of, so perhaps we should compare based on highest value, but multiple negative numbers by -10 to make up for the extra character.

We can do this on account setting, as well as currency change events and recalculate the extent here as well. It might be ideal to keep a "largestValue" attribute, so when transactions are added they can be compared against this and grow if necessary, without comparing everything on every addition.

Changed in wxbanker:
status: Confirmed → Triaged
summary: - [0.5] amount column is not always large enough
+ amount/total column is not always large enough
Revision history for this message
Michael Rooney (mrooney) wrote :

Okay, the Amount and Total columns now fit to the widest value on changing accounts or currencies. Handling wider added transactions seems like a nice to have but not as crucial, so I'll implement that if it is requested.

Changed in wxbanker:
milestone: 0.9 → 0.8.1
status: Triaged → Fix Committed
Michael Rooney (mrooney)
Changed in wxbanker:
status: Fix Committed → Fix Released
Revision history for this message
Toddandmargo-n (toddandmargo-n) wrote :

A workaround I came u with. Exit wxbanker and restart it. The columns will resize

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.