mirror/aseprite
1
0
Fork 0
mirror of https://github.com/aseprite/aseprite.git synced 2025-02-06 21:39:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix mouse capture being lost when scrolling list of fonts

Browse Source 
As font thumbnails are generated on-demand, each time a new thumbnail
is generated, the viewport will be updated (updateView() called). This
viewport update will remove the scroll bars temporarily which will
remove the mouse capture from them, but we'd like to preserve that
capture so the mouse can continue dragging the mouse/scrolling.
This commit is contained in:
David Capello 2024-09-27 19:01:51 -03:00
parent ec3f8617f0
commit 08d9101b62
1 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/ui/view.cpp
View File

@ -14,6 +14,7 @@
#include "gfx/size.h"
#include "ui/display.h"
#include "ui/intern.h"
#include "ui/manager.h"
#include "ui/message.h"
#include "ui/move_region.h"
#include "ui/resize_event.h"
@ -158,6 +159,15 @@ void View::updateView(const bool restoreScrollPos)
Widget* vw = UI_FIRST_WIDGET(m_viewport.children());
Point scroll = viewScroll();
// Get current mouse capture just in case if one of the scroll bars
// have the mouse captured, which means that while we were scrolling
// this updateView() was called/the viewport area changed (e.g. if
// an item thumbnail was generated when it was displayed and the
// viewport now is bigger).
Manager* man = manager();
Widget* mouseCapture = (man ? man->getCapture(): nullptr);
ASSERT(man);
// Set minimum (remove scroll-bars)
setScrollableSize(Size(0, 0), false);
@ -186,6 +196,11 @@ void View::updateView(const bool restoreScrollPos)
}
invalidate();
// Restore the mouse capture if it changed, which means that a
// scroll bar (when it was temporarily removed) lost the capture.
if (man && man->getCapture() != mouseCapture && mouseCapture->isVisible())
man->setCapture(mouseCapture);
}
Viewport* View::viewport()