[gwenview] [Bug 432075] Zoom 100% and back to Fit window broken since spin box commit

2021-01-27 Thread Antonio Prcela
https://bugs.kde.org/show_bug.cgi?id=432075

Antonio Prcela  changed:

   What|Removed |Added

  Latest Commit||https://invent.kde.org/grap
   ||hics/gwenview/commit/dcf34b
   ||852e58d0ba1a32d2d94889d7392
   ||e731918
 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #6 from Antonio Prcela  ---
Git commit dcf34b852e58d0ba1a32d2d94889d7392e731918 by Antonio Prcela.
Committed on 27/01/2021 at 17:49.
Pushed by ngraham into branch 'master'.

Fix zoom with F or middle mouse click caused by the introduction of the spinbox
in 2cdaca95

With the introduction of the spin box, the F button / middle mouse click don't
work properly anymore.
Problem caused because the spinbox did an emitZoomChanged even tho the zoom was
changed by something else than the spinbox. In the case from the bug report it
is 'Fit to window'.
Spin box emits now only after it had focus - either by mouse scroll change or
manually typing a value into it.

M  +3-1lib/zoomwidget.cpp

https://invent.kde.org/graphics/gwenview/commit/dcf34b852e58d0ba1a32d2d94889d7392e731918

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

[gwenview] [Bug 432075] Zoom 100% and back to Fit window broken since spin box commit

2021-01-26 Thread Leszek Lesner
https://bugs.kde.org/show_bug.cgi?id=432075

--- Comment #5 from Leszek Lesner  ---
This seems to fix it for me aswell. Thank you for the quick fix.

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

[gwenview] [Bug 432075] Zoom 100% and back to Fit window broken since spin box commit

2021-01-26 Thread Antonio Prcela
https://bugs.kde.org/show_bug.cgi?id=432075

--- Comment #4 from Antonio Prcela  ---
(In reply to Bug Janitor Service from comment #3)
> A possibly relevant merge request was started @
> https://invent.kde.org/graphics/gwenview/-/merge_requests/32

This should do the trick. I've tested and it works as it used to. Can you also
test?

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

[gwenview] [Bug 432075] Zoom 100% and back to Fit window broken since spin box commit

2021-01-26 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=432075

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #3 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/graphics/gwenview/-/merge_requests/32

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

[gwenview] [Bug 432075] Zoom 100% and back to Fit window broken since spin box commit

2021-01-26 Thread Antonio Prcela
https://bugs.kde.org/show_bug.cgi?id=432075

--- Comment #2 from Antonio Prcela  ---
(In reply to Nate Graham from comment #1)
> Antonio, can you investigate? Thanks!

Yes, i hope to fix it until friday.

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

[gwenview] [Bug 432075] Zoom 100% and back to Fit window broken since spin box commit

2021-01-26 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=432075

Nate Graham  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|REPORTED|CONFIRMED
   Keywords||regression
   Priority|NOR |HI

--- Comment #1 from Nate Graham  ---
Antonio, can you investigate? Thanks!

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

[gwenview] [Bug 432075] Zoom 100% and back to Fit window broken since spin box commit

2021-01-25 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=432075

Nate Graham  changed:

   What|Removed |Added

 CC||antonio.prc...@gmail.com,
   ||n...@kde.org

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