Bug 243564 - Resuming KDE session with Okular viewing a PDF in fs mode with nav panel hidden results in Contents pane shown
Summary: Resuming KDE session with Okular viewing a PDF in fs mode with nav panel hidd...
Status: RESOLVED FIXED
Alias: None
Product: okular
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Okular developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-04 11:05 UTC by Ivan Lezhnjov Jr.
Modified: 2010-07-21 00:06 UTC (History)
0 users

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


Attachments
A screenshot of Okular immediately after the KDE session was resumed (376.26 KB, image/png)
2010-07-04 11:05 UTC, Ivan Lezhnjov Jr.
Details
patch (456 bytes, patch)
2010-07-14 20:49 UTC, Mathieu Lornac
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ivan Lezhnjov Jr. 2010-07-04 11:05:58 UTC
Created attachment 48584 [details]
A screenshot of Okular immediately after the KDE session was resumed

Version:           unspecified
OS:                Linux

% okular --version
Qt: 4.6.3
KDE Development Platform: 4.4.5 (KDE 4.4.5)
Okular: 0.10.5

Reproducible: Always

Steps to Reproduce:
- Run Okular
- Open any PDF file
- Hide menu bar and navigation panel
- Go to full-screen mode
- Logout
- Login


Actual Results:  
OKular was opened with the PDF file in full-screen mode, menu bar hidden, part of navigation panel (Contents, Thumbnails, Reviews, Bookmarks buttons) hidden too but the Contents, that was hidden prior to logout was shown.

Expected Results:  
I expect that Okular should remember the settings, i.e. menu bar and navigation panel should be completely hidden.
Comment 1 Mathieu Lornac 2010-07-14 20:49:03 UTC
Created attachment 49152 [details]
patch

A test on the panel of contents selection was made instead of a test on the sidebar display
Comment 2 Albert Astals Cid 2010-07-21 00:06:16 UTC
SVN commit 1152384 by aacid:

Make sure that setting the toc to be the current index does not break sidebar visibility
Make Sidebar::isSidebarVisible symmetric with Sidebar::setSidebarVisibility
If showing the container make sure the list is also shown
BUGS: 243564


 M  +2 -0      part.cpp  
 M  +4 -1      ui/sidebar.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1152384