https://bugs.kde.org/show_bug.cgi?id=395375
Andrey <a.kama...@icloud.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Latest Commit|https://commits.kde.org/kri |https://commits.kde.org/kri |ta/befdf29e08828b85c2e30e30 |ta/0d88587df4606973743ae75c |fae6d188a9d0805c |6afc57a35f34c154 --- Comment #10 from Andrey <a.kama...@icloud.com> --- Git commit 0d88587df4606973743ae75c6afc57a35f34c154 by Andrey Kamakin, on behalf of Jonathan Liu. Committed on 26/06/2018 at 14:18. Pushed by akamakin into branch 'akamakin/T8628-multithreading-optimization'. Do not scroll pop-up palette when setting document offset Summary: QWidget::scroll(int dx, int dy) will scroll the widget including its children. Since KisPopupPalette is a child of KisQPainterCanvas when using the QPainter-based canvas we want to scroll only the widget and not its children by calling QWidget::scroll(int dx, int dy, const QRect &r) instead. Test Plan: 1. Click Settings > Configure Krita > Display 2. Uncheck "Canvas Graphics Acceleration" and click OK 3. File > New to create new document 4. Click "Freehand Brush Tool" 5. Right-click on canvas, rotate using wheel. The popup should not move. 6. Right-click on canvas, reset rotation by clicking at top circle of wheel. The popup should not move. 7. Right-click on canvas, zoom using zoom slider. The popup should not move. 8. Right-click on canvas, reset zoom to 100% by clicking "100%" button. The popup should not move,. Reviewers: #krita, dkazakov Reviewed By: #krita, dkazakov Subscribers: dkazakov, #krita Tags: #krita Differential Revision: https://phabricator.kde.org/D13703 M +1 -1 libs/flake/KoCanvasControllerWidget.cpp M +1 -1 libs/image/generator/kis_generator_layer.cpp https://commits.kde.org/krita/0d88587df4606973743ae75c6afc57a35f34c154 -- You are receiving this mail because: You are watching all bug changes.