Re: Review Request 129785: Stop debug spam in release builds

2017-03-12 Thread Miklós Máté
/ Testing --- Thanks, Miklós Máté

Re: Review Request 129286: Fix warnings about unhandled enumeration value in switch

2017-03-12 Thread Miklós Máté
ui/presentationwidget.cpp e6ecdc5 Diff: https://git.reviewboard.kde.org/r/129286/diff/ Testing --- Thanks, Miklós Máté

Re: Review Request 129785: Stop debug spam in release builds

2017-03-11 Thread Miklós Máté
> On Jan. 7, 2017, 2:24 p.m., Luigi Toscano wrote: > > Does it mean that you can't override the setting using kdebugsettings? If > > it's the case, it's probably a no-go. > > Miklós Máté wrote: > Oh, sorry I didn't realize how important this is: > > &q

Re: Review Request 129286: Fix warnings about unhandled enumeration value in switch

2017-03-11 Thread Miklós Máté
- Miklós --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129286/#review102792 --- On Oct. 30, 2016, 3:29 p.m., Miklós M

Re: Review Request 129287: Set viewport only once

2017-03-11 Thread Miklós Máté
marked as submitted. Review request for Okular. Changes --- Submitted with commit 59dd04766c46193baca9e16f78e0a539229d3257 by Albert Astals Cid on behalf of Miklós Máté to branch master. Repository: okular Description --- There are too many viewport setters already. Diffs

Re: Review Request 129785: Stop debug spam in release builds

2017-01-15 Thread Miklós Máté
> On Jan. 7, 2017, 2:24 p.m., Luigi Toscano wrote: > > Does it mean that you can't override the setting using kdebugsettings? If > > it's the case, it's probably a no-go. > > Miklós Máté wrote: > Oh, sorry I didn't realize how important this is: > > &q

Re: Review Request 129832: Only show warning messages by default

2017-01-14 Thread Miklós Máté
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129832/#review102029 --- Interesting hack. - Miklós Máté On Jan. 14, 2017, 7:23

Re: Review Request 129785: Stop debug spam in release builds

2017-01-11 Thread Miklós Máté
> On Jan. 7, 2017, 2:24 p.m., Luigi Toscano wrote: > > Does it mean that you can't override the setting using kdebugsettings? If > > it's the case, it's probably a no-go. > > Miklós Máté wrote: > Oh, sorry I didn't realize how important this is: > > &q

Review Request 129785: Stop debug spam in release builds

2017-01-07 Thread Miklós Máté
--- Okular core prints lots of debug messages, stop that in release builds. Diffs - OkularConfigureChecks.cmake 70fe8768 ui/formwidgets.cpp 7e68b5c7 Diff: https://git.reviewboard.kde.org/r/129785/diff/ Testing --- Thanks, Miklós Máté

Re: Review Request 129288: Fix unused computation result warning in plucker

2016-12-14 Thread Miklós Máté
- generators/plucker/unpluck/qunpluck.cpp 8363dd3 Diff: https://git.reviewboard.kde.org/r/129288/diff/ Testing --- Thanks, Miklós Máté

Re: Review Request 129584: fix loading eps

2016-12-11 Thread Miklós Máté
generators/txt/libokularGenerator_txt.json 97325db Diff: https://git.reviewboard.kde.org/r/129584/diff/ Testing --- Thanks, Miklós Máté

Re: Review Request 129584: fix loading eps

2016-12-09 Thread Miklós Máté
ts.git/tree/src/imageformats > > Okular from git master displays eps files fine here using the kimgio > backend. > Though spectre would still be the better choice of course. > > Miklós Máté wrote: > On my system (Debian Unstable) kimgio says unsupported file f

Re: Review Request 129610: Check all supported mime types the generator supports

2016-12-08 Thread Miklós Máté
marked as submitted. Review request for Okular. Changes --- Submitted with commit e81fb3d639400df03d47749e4c5532b32394b45c by Albert Astals Cid on behalf of Miklós Máté to branch Applications/16.12. Repository: okular Description --- Stopping on the first inherit match can prevent

Re: Review Request 129584: fix loading eps

2016-12-04 Thread Miklós Máté
ts.git/tree/src/imageformats > > Okular from git master displays eps files fine here using the kimgio > backend. > Though spectre would still be the better choice of course. > > Miklós Máté wrote: > On my system (Debian Unstable) kimgio says unsupported file f

Re: Review Request 129584: fix loading eps

2016-12-03 Thread Miklós Máté
itted a new review request that can replace the last hunk. - Miklós --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129584/#review101206 -------

Review Request 129610: Check all supported mime types the generator supports

2016-12-03 Thread Miklós Máté
/ Testing --- Thanks, Miklós Máté

Review Request 129584: fix loading eps

2016-11-29 Thread Miklós Máté
://git.reviewboard.kde.org/r/129584/diff/ Testing --- Thanks, Miklós Máté

Re: Review Request 129286: Fix warnings about unhandled enumeration value in switch

2016-11-21 Thread Miklós Máté
ialized warning. - Miklós --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129286/#review100919 --- On Oct. 30, 2016, 3

Re: Review Request 129288: Fix unused computation result warning in plucker

2016-11-20 Thread Miklós Máté
le is used to track whether to > > start or end a formatted string span (represented by tags in the > > explode tool). > > Martin Tobias Holmedahl Sandsmark wrote: > > https://cgit.kde.org/okular.git/commit/?id=d9a31fa02ad92c663b3c5bd0fc5be38b09bc761f > for mor

Re: Review Request 129288: Fix unused computation result warning in plucker

2016-11-20 Thread Miklós Máté
my repository doesn't contain that commit. - Miklós --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129288/#review100941 ------

Re: Review Request 129285: Fix comparision between signed and unsigned

2016-11-17 Thread Miklós Máté
marked as submitted. Review request for Okular. Changes --- Submitted with commit 4520ac7192f5c505e8fed5a14f66df85214f6620 by Albert Astals Cid on behalf of Miklós Máté to branch Applications/16.12. Repository: okular Description --- Fewer warnings is good. Diffs

Review Request 129287: Set viewport only once

2016-10-30 Thread Miklós Máté
--- There are too many viewport setters already. Diffs - ui/thumbnaillist.cpp 8a6b12d Diff: https://git.reviewboard.kde.org/r/129287/diff/ Testing --- Thanks, Miklós Máté

Review Request 129288: Fix unused computation result warning in plucker

2016-10-30 Thread Miklós Máté
://git.reviewboard.kde.org/r/129288/diff/ Testing --- Thanks, Miklós Máté

Review Request 129284: Don't jump up 1/2 page on file reload if scrolled to page (with left-right cursor)

2016-10-30 Thread Miklós Máté
w.cpp bb540cb Diff: https://git.reviewboard.kde.org/r/129284/diff/ Testing --- Thanks, Miklós Máté

Review Request 129285: Fix comparision between signed and unsigned

2016-10-30 Thread Miklós Máté
--- Fewer warnings is good. Diffs - generators/dvi/TeXFont_PFB.cpp 6b27093 Diff: https://git.reviewboard.kde.org/r/129285/diff/ Testing --- Thanks, Miklós Máté

Review Request 129286: Fix warnings about unhandled enumeration value in switch

2016-10-30 Thread Miklós Máté
/diff/ Testing --- Thanks, Miklós Máté

Re: [Okular-devel] Review Request 127526: Set viewport to the position clicked on the thumbnail list

2016-04-05 Thread Miklós Máté
marked as submitted. Review request for Okular. Changes --- Submitted with commit c183dd8bd300b39e55e3309b6b279b6e7b8e19c2 by Albert Astals Cid on behalf of Miklós Máté to branch Applications/16.04. Repository: okular Description --- https://bugs.kde.org/show_bug.cgi?id=328686

[Okular-devel] Review Request 127526: Set viewport to the position clicked on the thumbnail list

2016-03-29 Thread Miklós Máté
--- https://bugs.kde.org/show_bug.cgi?id=328686 Also fix the mouse cursor shape (could be a separate patch, but I don't know how to post a series here) Diffs - ui/thumbnaillist.cpp 8704bfa Diff: https://git.reviewboard.kde.org/r/127526/diff/ Testing --- Thanks, Miklós Máté

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

2016-03-21 Thread Miklós Máté
marked as submitted. Review request for Okular. Changes --- Submitted with commit 1371ccfa28cf777dd04d3f67a1947e41b02dba0d by Albert Astals Cid on behalf of Miklós Máté to branch Applications/16.04. Repository: okular Description --- fixes bug #342076 Diffs - ui

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

2016-03-19 Thread Miklós Máté
--- v4: only show 1 triangle in mobile version Repository: okular Description --- fixes bug #342076 Diffs (updated) - ui/tocmodel.cpp ce93366 Diff: https://git.reviewboard.kde.org/r/127013/diff/ Testing --- Thanks, Miklós Máté

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

2016-03-18 Thread Miklós Máté
w (i.e. onyl the last one > > has the triangle)? > > Miklós Máté wrote: > Without information about the expanded nodes the best we can do is to put > triangles at every containing node. Using only the last node is not good, > because it may be hidden within a collapsed

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

2016-03-15 Thread Miklós Máté
w (i.e. onyl the last one > > has the triangle)? > > Miklós Máté wrote: > Without information about the expanded nodes the best we can do is to put > triangles at every containing node. Using only the last node is not good, > because it may be hidden within a collapsed

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

2016-03-12 Thread Miklós Máté
sed parent node. - Miklós --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127013/#review93378 --- On March 9, 2016, 8

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

2016-03-09 Thread Miklós Máté
--- v3: fix for the mobile version Repository: okular Description --- fixes bug #342076 Diffs (updated) - ui/tocmodel.cpp ce93366 Diff: https://git.reviewboard.kde.org/r/127013/diff/ Testing --- Thanks, Miklós Máté

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

2016-03-07 Thread Miklós Máté
) > * Introduce a new role named someting like PathToLeaf that returns the > arrow item for the whole path (except the leaf) (for that we do not need view > info) > * In pageitemdeleate (that is tied to the view) you can inspect the view > expanded/collapsed status + the ne

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

2016-03-07 Thread Miklós Máté
w about "dynamic_cast< QTreeView* > ( QObject::parent() );" and only filter the highlighted items when this returns not NULL? - Miklós --- This is an automatically generated e-mail. To reply, visit: https://g

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

2016-02-28 Thread Miklós Máté
-- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127013/#review92846 ------- On Feb. 27, 2016, 10:30 a.m., Miklós Máté wrote: > > --

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

2016-02-27 Thread Miklós Máté
-d973dbcb5b64__okular_marker_v2.diff Thanks, Miklós Máté ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel

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

2016-02-25 Thread 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 > &g

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

2016-02-19 Thread 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 > &g

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

2016-02-17 Thread Miklós Máté
items on the same level that point to the same page, this algorithm only marks the last one. - Miklós Máté On Feb. 8, 2016, 5:49 p.m., Miklós Máté wrote: > > --- > This is an automatically generated e-mail. To reply,

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

2016-02-17 Thread Miklós Máté
> On Feb. 11, 2016, 10:27 p.m., Miklós Máté wrote: > > ui/toc.cpp, line 44 > > <https://git.reviewboard.kde.org/r/127013/diff/1/?file=443811#file443811line44> > > > > The model has to know about the treeview, see line 209. This also makes > > sure

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

2016-02-16 Thread Miklós Máté
> On Feb. 11, 2016, 10:27 p.m., Miklós Máté wrote: > > ui/toc.cpp, line 44 > > <https://git.reviewboard.kde.org/r/127013/diff/1/?file=443811#file443811line44> > > > > The model has to know about the treeview, see line 209. This also makes > > sure

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

2016-02-11 Thread Miklós Máté
ment62944> Instead of marking the toc items that point to the current page I mark the toc items that *contain* the current page. - Miklós Máté On Feb. 8, 2016, 5:49 p.m., Miklós Máté wrote: > > --- > This is an automatic

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

2016-02-08 Thread Miklós Máté
--- 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é ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman