[Okular-devel] Review Request: Unfolded toc remains unfolded on reload unless the file is changed

[Albert Astals Cid]

> On Jan. 2, 2013, 9:46 p.m., Albert Astals Cid wrote:
> > part.cpp, line 1622
> > <http://git.reviewboard.kde.org/r/107994/diff/5/?file=103405#file103405line1622>
> >
> >     Do we really need this collapse here?
> 
> Jaydeep Solanki wrote:
>     yes I added that because suppose an "Open" tag attribute is there in the toc on a specific node, so okular will expand that node, but if the user had closed that node before reload then this will keep it closed after reload, otherwise along with m_oldTocExpandedIndexes, that node will also get expanded.

But have you tried that really works? Because the "Open" tag attribute will get executed after this code, so adding the collapse all won't do anything, no?
Which file you used to test that?


- Albert


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


On Dec. 31, 2012, 2 p.m., Jaydeep Solanki wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107994/
> -----------------------------------------------------------
> 
> (Updated Dec. 31, 2012, 2 p.m.)
> 
> 
> Review request for Okular.
> 
> 
> Description
> -------
> 
> This fix will preserve the state of the Table Of Content (toc) on document reload, but if the toc is edited/changed, then it will be restored to default (i.e. all nodes will be folded)
> 
> 
> This addresses bug 312138.
>     http://bugs.kde.org/show_bug.cgi?id=312138
> 
> 
> Diffs
> -----
> 
>   part.h 0c57560 
>   part.cpp 1922128 
>   ui/toc.h eeeff98 
>   ui/toc.cpp 4c84b62 
>   ui/tocmodel.h a857dc0 
>   ui/tocmodel.cpp 39add80 
> 
> Diff: http://git.reviewboard.kde.org/r/107994/diff/
> 
> 
> Testing
> -------
> 
> Checked. Works fine with me.
> 
> 
> Thanks,
> 
> Jaydeep Solanki
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20130102/368b2c38/attachment.html>