1. 13 Feb, 2020 1 commit
  2. 11 Feb, 2020 1 commit
  3. 10 Feb, 2020 4 commits
  4. 09 Feb, 2020 1 commit
  5. 06 Feb, 2020 1 commit
  6. 04 Feb, 2020 1 commit
  7. 02 Feb, 2020 1 commit
    • David Hurka's avatar
      Replace ToolAction by ToggleActionMenu · c6d937ab
      David Hurka authored
      Summary:
      This replaces ToolAction by a near-drop-in replacement named ToggleActionMenu. The new annotation toolbar already uses this (D15580).
      
      Unlike ToolAction, ToggleActionMenu inherits from KActionMenu to be more flexible.
      * Menu can be set from outside, not hard coded.
      * Default action for toolbar button is controllable from outside. (Theoretically, the button could trigger //anything// now.)
      * KActionMenu instead of KSelectAction:
        - Pluggable in other menus, thus called “Menu”.
        - Doesn’t make the actions exclusive, so //any// actions can be added to the menu.
      * ImplicitDefaultAction mode can choose the default action of the toolbar buttons automatically, by looking for the first checked action in the menu.
      
      Toolbar buttons use the default action //of// this menu, not this menu itself as action.
      
      Because the default action is configurable now, D21622 and D21635 (where we tried to fine-tune ToolAction) become obsolete.
      
      Screenshot:
      Everything like before, here with mouse_selecttool added to Tools menu to show submenu capability.
      {F6884228}
      
      Test Plan:
      ToolAction replacement and ImplicitDefaultAction mode:
       * Open Okular and look at toolbar button -> has correct tool selected.
       * Open a document.
       * Look at toolbar button menu -> Correct menu entries (like before, with ToolAction).
       * Select some selection tools through shortcuts and toolbar button -> behaves correctly.
      Usage as submenu:
       * Add ToggleActionMenu ("mouse_selecttool") to menubar (..../kxmlgui5/okular/part.rc) -> Submenu looks correctly, has no checkbox attached and so on...
      Toolbar buttons:
       * Add diverse other actions to the menu -> still works as before.
       * Add actions when toolbar buttons are already created -> actions are added to existing buttons.
       * setDefaultAction() to some completely unrelated action. -> ToggleActionMenu does not get confused.
      
      Reviewers: simgunz
      
      Reviewed By: simgunz
      
      Subscribers: aacid, ngraham, simgunz, okular-devel
      
      Tags: #okular
      
      Differential Revision: https://phabricator.kde.org/D21971
      c6d937ab
  8. 01 Feb, 2020 1 commit
  9. 31 Jan, 2020 3 commits
  10. 30 Jan, 2020 2 commits
    • Tobias Deiminger's avatar
      Fix race condition in generator.cpp · 378e99d7
      Tobias Deiminger authored
      The check whether to allocate a new QMutex was unprotected.
      Two threads may check and allocate concurrently, but only one pointer gets
      remembered, the other one will leak. In worst case the returned
      mutex is different for two threads, so that two threads try to
      synchronize by using two different mutexes.
      378e99d7
    • Tobias Deiminger's avatar
      Fix render stop and high load due to timing issue · 593803b0
      Tobias Deiminger authored
      Text generation is connected to pixmap generation thread started signal.
      However the signal may have been emitted faster than the connect could took place,
      and because started is fired only once, the connected lambda never got executed.
      
      generatePixmap tried to sync up with that never happening text generation anyway
      by means of scheduling itself. This lead to a infinite loop via a no more sleeping QEventLoop.
      
      Fixed by moving the connect in front of starting the thread.
      
      BUG: 396137
      BUG: 396087
      CCBUG: 403643
      593803b0
  11. 29 Jan, 2020 1 commit
  12. 28 Jan, 2020 2 commits
  13. 26 Jan, 2020 2 commits
  14. 24 Jan, 2020 1 commit
  15. 19 Jan, 2020 1 commit
  16. 17 Jan, 2020 2 commits
  17. 16 Jan, 2020 1 commit
  18. 15 Jan, 2020 1 commit
    • Kai Uwe Broulik's avatar
      Port SearchLineWidget to use KBusyIndicatorWidget · 643a0804
      Kai Uwe Broulik authored
      It's a lot nicer than manually painting a KPixmapSequence and supports high dpi out of the box.
      
      The new code requires Frameworks >= 5.61, it is therefore only enabled
      if at least that Frameworks version is available.
      643a0804
  19. 11 Jan, 2020 3 commits
  20. 09 Jan, 2020 3 commits
  21. 08 Jan, 2020 1 commit
  22. 07 Jan, 2020 2 commits
    • Albert Astals Cid's avatar
      88e5775f
    • Albert Astals Cid's avatar
      "Fix" PartTest::testAdditionalActionTriggers · 45303be6
      Albert Astals Cid authored
      By waiting for the delayResizeEventTimer to trigger before starting the
      test.
      
      The test "randomly" failed before, which shows there's an actual bug in
      the code, but the actual bug is "if you are moving the mouse over a
      field that has a on focus action and at that very same time the
      delayResizeEventTimer triggers, the action won't be executed".
      
      That's almost impossible to reproduce in real life so i'm going to
      pretend the bug is not there for all purposes
      45303be6
  23. 06 Jan, 2020 2 commits
    • Bob Bai's avatar
      Enable closing a tab by middle click · 60f1fa79
      Bob Bai authored
      Some discussions at https://phabricator.kde.org/D25484.
      
      This MR adds:
      
      * mouse middle button event handler for closing a tab
      * an `undo close tab` action
      
      Test plan:
      
      1. Compile and install.
      2. Enable `Open new files in tabs`.
      3. Open two documents. They should be opened in two tabs in the same window.
      4. Middle click one of the tabs. It should be closed.
      5. Choose `File`-`Undo close tab`. The closed tab should be reopened.
      6. Middle click one of the tabs. It should be closed.
      7. Press <kbd>Ctrl</kbd>+<kbd>Shift</kbd>+<kbd>T</kbd>. The closed tab should be reopened.
      60f1fa79
    • Christoph Feck's avatar
      Update Appstream for new release · 475652a9
      Christoph Feck authored
      475652a9
  24. 05 Jan, 2020 1 commit
  25. 03 Jan, 2020 1 commit