--On July 28, 2010 12:56:04 PM -0400 Derek Atkins <warl...@mit.edu> wrote:

Mike Alexander <m...@code.gnucash.org> writes:

+
+        /* If we're on a resize line and the column to the right is
zero +           width, resize that one. */
+        if (on_line)
+            resize_col = find_resize_col (header, ptr_col);
+        else
+            resize_col = ptr_col;
+
+        if (resize_col > -1)

I don't quite understand this patch (and definitely don't understand
the comment.  Are you saying that it will resize the zero-width
column?  Or are you saying that it will ignore the zero-width column?
The 0-width column to the right of the balance (the rate cell) is
0-width for a reason and should not automatically get resized.

-derek

I had a feeling I might get a message about this change. :-) It took me a while to figure out what I think the original author had in mind.

With this change, if you double click exactly on (actually within 1 pixel of) a dividing line and there is a zero width column to the right of the line (essentially there are multiple dividing lines in the same spot in this case) it will attempt to resize the zero width column to the right of the dividing line. If you double click anywhere else on a header it will resize the column you clicked on. You can demonstrate this by dragging one of the columns to be zero width then double click on the line where it is hiding.

The problem before was that it was applying the rule about zero width columns to the right even if you clicked in the middle of the column. That's why a double click in the middle of the balance column tried to resize the zero width column to the right of it. I fixed this so the zero width column rule only applies if you double click on a dividing line. In any case it won't resize the zero width column to the right of the balance column since, as you say, it's fixed at zero width.

Does this make it more clear? I think the change is definitely an improvement, although it might need to be tweaked some.

         Mike

_______________________________________________
gnucash-devel mailing list
gnucash-devel@gnucash.org
https://lists.gnucash.org/mailman/listinfo/gnucash-devel

Reply via email to