Bug 342217 - Hide scrollbars option resets after closing krita
Summary: Hide scrollbars option resets after closing krita
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: General (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-26 10:10 UTC by truekikim
Modified: 2015-01-31 11:11 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description truekikim 2014-12-26 10:10:56 UTC
Checking the "hide scrollbars" box in Configure Krita>display works only for one session. After opening krita again the box is still checked but the scrollbars are not hidden.


Reproducible: Always

Steps to Reproduce:
1. go to Configure Kriuta > display
2. check hide scrollbars and click ok
3. close Krita
4. open Krita again (scrollbars are now visible)
Comment 1 pcchou 2014-12-26 14:41:46 UTC
Hello, I'm a Google code-in student who is trying to do bug-triaging task.
Can you provide which version of krita are you using?
Comment 2 truekikim 2014-12-26 14:51:40 UTC
2.9 beta 1

To see that the scrollbars are not hidden you have to open an image.
Comment 3 Dmitry Kazakov 2014-12-29 12:21:03 UTC
I can confirm. The problem seems to be somewhere in the loading code, that is the Setting is loaded correctly (the checkbox in checked), but the scrollbars are not hidden until you open and reapply the settings dialog.
Comment 4 Rishabh Saxena 2014-12-30 09:19:42 UTC
Is anyone else is currently working on fixing this bug or not?
Comment 5 Sven Langkamp 2015-01-01 15:06:57 UTC
I don't think anybody is working on this currently.
Comment 6 Halla Rempt 2015-01-31 11:11:01 UTC
Git commit 6a08b4b038372488080d9f49d0e18ca41d0827a3 by Boudewijn Rempt.
Committed on 31/01/2015 at 11:09.
Pushed by rempt into branch 'calligra/2.9'.

During startup/load, showHideScrollbars() is called too early, when
d->currentImageView is still null, missing the opportunity to disable
scroll bars. Let's try again when a subsequent setCurrentView is called.

CCMAIL:ademko@gmail.com
REVIEW:122335

Patch by Aleksander Demko. Thanks!

M  +2    -0    krita/ui/KisViewManager.cpp

http://commits.kde.org/calligra/6a08b4b038372488080d9f49d0e18ca41d0827a3