1. 02 Apr, 2020 1 commit
  2. 29 Mar, 2020 1 commit
  3. 28 Mar, 2020 6 commits
  4. 27 Mar, 2020 1 commit
  5. 26 Mar, 2020 3 commits
  6. 16 Mar, 2020 1 commit
  7. 15 Mar, 2020 4 commits
  8. 14 Mar, 2020 1 commit
  9. 12 Mar, 2020 2 commits
    • Albert Astals Cid's avatar
      Don't use QDir::separator · d1568efd
      Albert Astals Cid authored
      If we're building paths for Qt, always use /
      d1568efd
    • David Hurka's avatar
      Move Continuous option to View Mode submenu · 493ba07e
      David Hurka authored
      Summary:
      This moves the Continuous option from the View menu to the
      View Mode submenu. This makes the View Mode menu (a KActionMenu)
      more useful in the main toolbar. Additionally, “Continuous” is explained by the context “View Mode”.
      
      The primary intention was to give this View Mode a similar usage pattern like
      the Change Colors menu (D21195), if both are added to the toolbar. See my
      suggestion in Bugs 407217 and 407326.
      
      FEATURE: 407326
      
      Screenshot before:
      {F6821917}
      
      Screenshot now:
      {F6821920}
      and in the toolbar:
      {F6821921}
      
      Test Plan:
      * Look into View menu and test entries
      * Add View Mode menu to toolbar and test entries
      
      Reviewers: #okular, #vdg, aacid
      
      Spies: aacid, ngraham, okular-devel, kde-doc-english
      
      Tags: #okular, #documentation
      
      Differential Revision: https://phabricator.kde.org/D21196
      493ba07e
  10. 10 Mar, 2020 1 commit
  11. 08 Mar, 2020 3 commits
  12. 06 Mar, 2020 2 commits
  13. 04 Mar, 2020 2 commits
  14. 02 Mar, 2020 2 commits
    • Albert Astals Cid's avatar
      4964e7e4
    • Albert Astals Cid's avatar
      Fix re-rendering of images when using partial updates · 81c00571
      Albert Astals Cid authored
      Partial updates trigger when the page is taking "too much" to render (>
      500 ms).
      
      When this happens we store a pixmap for the page, this meant that
      Page::hasPixmap returned true, so when moving the viewport around we
      would think that that page was already rendered, and thus needed no
      rendering so we didn't add it to the list of requested pixmaps.
      
      Then on document when seeing the new list of requested pixmaps we would
      go and check and say "oh there's one request going on that we don't want
      anymore, let's cancel it", so we would cancel the page that we actually
      wanted (and clear the partial pixmap)
      
      Then on the next scroll we would realize we did not have that page
      pixmap and then request it again which since it's the current page would
      immediately stop all other renders and start this one.
      
      Then we would get a temporary pixmap and the loop of cancellations and
      requesting again would keep happening as long as the user moved the
      viewport.
      
      We fix that my making hasPixmap return false if the pixmap we have is a
      partial one, because that's what the function meant "forever" until we
      recently introduced partial updates so all the calls to hasPixmap
      actually mean "hasNonPartialPixmap"
      
      BUGS: 418086
      81c00571
  15. 01 Mar, 2020 4 commits
  16. 29 Feb, 2020 5 commits
  17. 24 Feb, 2020 1 commit