mirror of
https://github.com/juce-framework/JUCE.git
synced 2026-01-10 23:44:24 +00:00
ad5a755b10
For some transforms, the program could get stuck in the following loop: - The content component emits a resized/moved notification, leading to the initial call to Viewport::updateVisibleArea. - New positions are computed for the viewport scrollbars, and scrollbar listeners are notified synchronously that the scrollbars have been updated. - The viewport itself listens to the scrollbars, so it receives a notification and updates the position of the content component. - The scrollbar position (quantised to an integer) resolves to a component position (also quantised to an integer) that differs from the existing position, so the new position is applied. - The viewport now attempts to set the scrollbars to the correct position in response, and notifies listeners that the scrollbars have moved... Normally, the recursion would exit at the point where the component position is set to its current position. If we're unlucky, though, converting from view pos to scrollbar pos, then scrollbar pos back to view pos may result in a view pos that differs from the original value. This fix adds a new exit condition from the recursion. On receiving a scrollbar move notification, we check whether the scrollbar position computed from the current view position matches the incoming scrollbar position. If it does, there's no need to compute and apply a new view position from the incoming scrollbar position. |
||
|---|---|---|
| .. | ||
| accessibility | ||
| application | ||
| buttons | ||
| commands | ||
| components | ||
| desktop | ||
| detail | ||
| drawables | ||
| filebrowser | ||
| keyboard | ||
| layout | ||
| lookandfeel | ||
| menus | ||
| misc | ||
| mouse | ||
| native | ||
| positioning | ||
| properties | ||
| widgets | ||
| windows | ||
| juce_gui_basics.cpp | ||
| juce_gui_basics.h | ||
| juce_gui_basics.mm | ||