Review Request: Do not randomly change the state of Konqueror's Bookmark toolbar when users modify toolbar settings.

[David Faure]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103580/#review9430
-----------------------------------------------------------

Ship it!


Assuming this doesn't hide the bar when it shouldn't (on startup, or when the user asks to see it), this looks ok.

- David Faure


On Dec. 30, 2011, 6:47 a.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103580/
> -----------------------------------------------------------
> 
> (Updated Dec. 30, 2011, 6:47 a.m.)
> 
> 
> Review request for KDE Base Apps.
> 
> 
> Description
> -------
> 
> Right now if you configure Konqueror's toolbars and press Apply or OK, the Bookmark toolbar is made visible regardless of its previous state. Even if you choose to hide the toolbar, it will still be shown after the toolbar settings modification. This patch fixes that by preserving the visibility state of the previous bookmark toolbar.
> 
> 
> Diffs
> -----
> 
>   konqueror/src/konqmainwindow.cpp 3b6e418 
> 
> Diff: http://git.reviewboard.kde.org/r/103580/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dawit Alemayehu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120102/f9797ea1/attachment.htm>