https://bugs.kde.org/show_bug.cgi?id=395375

Dmitry Kazakov <dimul...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|UNCONFIRMED                 |RESOLVED
      Latest Commit|                            |https://commits.kde.org/kri
                   |                            |ta/eadfec3dd43ecd72c3080a87
                   |                            |96711477089511b8

--- Comment #8 from Dmitry Kazakov <dimul...@gmail.com> ---
Git commit eadfec3dd43ecd72c3080a8796711477089511b8 by Dmitry Kazakov, on
behalf of Jonathan Liu.
Committed on 25/06/2018 at 07:57.
Pushed by dkazakov into branch 'master'.

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/eadfec3dd43ecd72c3080a8796711477089511b8

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to