After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 675517 - Signal sidebar widgets about whether resize is on-going
Signal sidebar widgets about whether resize is on-going
Status: RESOLVED WONTFIX
Product: reinteract
Classification: Other
Component: general
unspecified
Other All
: Normal enhancement
: ---
Assigned To: reinteract-maint
reinteract-maint
gnome[unmaintained]
Depends on:
Blocks:
 
 
Reported: 2012-05-05 17:37 UTC by Robert Schroll
Modified: 2018-07-10 22:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Change set_sidebar_width to say if in an on-going resize (3.35 KB, patch)
2012-05-05 17:37 UTC, Robert Schroll
none Details | Review
Only resize replot Axes at end of sidebar resize (882 bytes, patch)
2012-05-05 17:37 UTC, Robert Schroll
none Details | Review

Description Robert Schroll 2012-05-05 17:37:26 UTC
When the sidebar is resized, the widgets have their set_sidebar_width 
methods called with the new widths.  This happens as often as possible, 
given the delays of those method calls.  Some widgets, however, may take 
a while to resize.  There should be a way for them to ignore resize 
events from an on-going resize and only resize when the user stops 
dragging the sidebar divider.

This patch adds a in_resize argument (suggestions for a better name?) to 
the set_sidebar_width method, so the widget can decide whether it should 
adjust its size or not.  The internal signaling is a bit hackish at the 
moment.  At the very least it needs to be documented, but maybe there's 
a better way.

The second patch changes replot to only resize at the end of drags.  
This is just for a demonstration; the current behavior hasn't bothered 
me at all.
Comment 1 Robert Schroll 2012-05-05 17:37:29 UTC
Created attachment 213502 [details] [review]
Change set_sidebar_width to say if in an on-going resize

A binary flag is added to as an argument to set_sidebar_width.  This
check is started in ViewSidebarLayout.do_motion_notify, which checks
whether it was called by a BUTTON_RELEASE event, which means that the
resize is over, or not.  This state is passed into sidebar_width via the
sign of the value, which is a bit hackish.
Comment 2 Robert Schroll 2012-05-05 17:37:33 UTC
Created attachment 213503 [details] [review]
Only resize replot Axes at end of sidebar resize

This is only for demonstration purposes; continuous resizing is probably
okay for replot Axes.
Comment 3 André Klapper 2018-07-10 22:05:38 UTC
Reinteract is not under active development anymore and had its last code changes
in early 2012: http://git.fishsoup.net/cgit/reinteract/log/

Closing this report as WONTFIX as part of Bugzilla Housekeeping to reflect
reality. Please feel free to reopen this ticket (or rather transfer the project
to GNOME Gitlab, as GNOME Bugzilla is deprecated) if anyone takes the
responsibility for active development again.