Bug 342217

Summary:	Hide scrollbars option resets after closing krita
Product:	[Applications] krita	Reporter:	truekikim
Component:	General	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	dimula73, pcchou, rishabh.saxena2112, sven.langkamp
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	http://commits.kde.org/calligra/6a08b4b038372488080d9f49d0e18ca41d0827a3	Version Fixed In:
Sentry Crash Report:

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