r23202 - gnucash/trunk/src/gnome - Bug #699450

Mike Evans mikee at code.gnucash.org
Tue Oct 1 08:22:05 EDT 2013

Author: mikee
Date: 2013-10-01 08:22:00 -0400 (Tue, 01 Oct 2013)
New Revision: 23202
Trac: http://svn.gnucash.org/trac/changeset/23202

Bug #699450

Reverses previous commit for this bug and applies patches for fix.

This is an amalgum of 7 patches.
Patch comments are reproduced below.  All patches supplied by R Ratcliff.

[PATCH 1/7] Fix budget view resizing issue

The fix here is to change the hardcoded width value from 86 to 60.
* Remove size-allocate signal bound to treeviewcolumn (columns
  don't emit signals)

[PATCH 2/7] Fix the way column width is determined

The budget tree view has a bunch of columns that have visible
set to false. These columns have a width of 0. Change
gbv_tree_view_resized_cb to ignore these columns.

[PATCH 3/7] Change column sizing method.

Not sure if this is crucial to the bug fix.

[PATCH 4/7] Fix regression for window horizontal resizing

This change does two things
* Set expand on totals_title_col to TRUE
* Not set the width of this column in gbv_treeview_resized_cb

Note: The second one is a bit tricky. It relies on the fact that
totals_title_col has a default sizing of GROW_ONLY. It only explicitly
sets the size on columns that have a sizing of FIXED

[PATCH 5/7] Fix horizontal scrolling

* Add a vbox inside the scrolled_window, and add the two treeviews to that.

This fixes horizontal scrolling. If the budget view is too wide for the
window, then a horizontal scrollbar will appear at the bottom, and scroll
both the main budget view and the totals view at the bottom.

However, it breaks vertical scrolling (vertical scrollbar will scroll the
entire view, instead of just the top view). I'll fix this in the next commit.

[PATCH 6/7] Fix vertical scrolling

Vertical scrolling will only apply to the top view (budget values) and not
to the bottom view (budget totals).

Implementation: add an inner scrolled window for the vertical scrollbar.
Set scrolled_window vertical scrollbar policy to NEVER.

Note: This design looks weird when both scrollbars are used because you
have to scroll horizontally to the right in order to see the vertical
scrollbar for the top section.

[PATCH 7/7] Clean up code before submitting patch

* Remove debugging and commented out code

More information about the gnucash-patches mailing list