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.

Reply via email to