[Okular-devel] Review Request 127013: Properly show marker for current section in TOC

[Miklós Máté]

> On Feb. 18, 2016, 12:16 a.m., Miklós Máté wrote:
> > ui/tocmodel.cpp, line 152
> > <https://git.reviewboard.kde.org/r/127013/diff/1/?file=443812#file443812line152>
> >
> >     No. This function marks all the TOC items that contain the current page, but later in the data() method they are filtered, and in the end only one triangle marker will be shown (depending on how the tree is expanded in the view). If there are multiple TOC items on the same level that point to the same page, this algorithm only marks the last one.
> 
> Albert Astals Cid wrote:
>     I'm not convinced marking the last one is the best, could we mark "the correct" one? After all we know the viewports for the two items so we know what's more correct than the other, no? If you need such a document i can send it to you.

TOCModel::setCurrentViewport() is only called when the page number changes. What do you mean "the correct one"?


- Miklós


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127013/#review92516
-----------------------------------------------------------


On Feb. 8, 2016, 5:49 p.m., Miklós Máté wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127013/
> -----------------------------------------------------------
> 
> (Updated Feb. 8, 2016, 5:49 p.m.)
> 
> 
> Review request for Okular.
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> fixes bug #342076
> 
> 
> Diffs
> -----
> 
>   ui/toc.cpp 09625af 
>   ui/tocmodel.cpp ce93366 
> 
> Diff: https://git.reviewboard.kde.org/r/127013/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Miklós Máté
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20160219/7d6f66cc/attachment.html>