1. 04 Jul, 2021 1 commit
  2. 03 Jul, 2021 1 commit
  3. 22 Jun, 2021 2 commits
    • Felix Ernst's avatar
      Merge branch 'release/21.04' addendum · 823e99fe
      Felix Ernst authored
      This should have belonged to my previous merge
      480b3067 but somehow I managed to
      miss this one line.
      
      It was written by Derek Christ as part of
      dc3beae3.
      823e99fe
    • Derek Christ's avatar
      Fix shift-action modifier in context menu · dc3beae3
      Derek Christ authored and Méven Car's avatar Méven Car committed
      Before this patch, the shift-action modifier in context menus did not
      work when a sub-context menu is open, that does not have the main
      context menu as its parent.
      The new fix installs an event filter on QApplication whenever a new
      context menu is opened to make the context menu aware of shift-presses
      even when a sub-context menu is in focus.
      
      BUG: 425997
      FIXED-IN: 21.04
      dc3beae3
  4. 18 May, 2021 1 commit
    • Nate Graham's avatar
      Use more appropriate icon for "Create New" action · 7b112c7b
      Nate Graham authored
      Right now this action is using the "document-new" icon, which depicts
      creating a new file. This is inaccurate, as the menu provides options for
      creating more than just files: it has one option to create folders and
      three for creating different types of links. In fact, depending on the
      set of installed software, there may not be any options to create new
      documents at all!
      
      To fix this, the icon would need to be made either more specific or
      more generic.
      
      There is no such icon that depicts being able to create a folder, a
      document, or a link; such a thing is too specific to be able to
      represent with a small icon. So our only real option is to use a more
      generic icon.
      
      This commit does that.
      7b112c7b
  5. 13 May, 2021 1 commit
    • Felix Ernst's avatar
      Re-arrange the contents of the hamburger menu · 53a778c1
      Felix Ernst authored and Nate Graham's avatar Nate Graham committed
      To improve usability, the entries in the hamburger menu are
      changed. Maybe the biggest fault of the previous menu contents were
      that there were too many actions.
      
      The new menu contents are composed of all the actions which are
      necessary to use Dolphin and those which are very useful and should
      be of interest for most users. Some menu contents change depending
      on the state of the application. We can be more bold in only
      showing what really seems necessary because this commit activates
      the special sub-menu of KHamburgerMenu that helps users discover
      all further features of Dolphin.
      
      The hamburger menu is from now on also added to the context menus
      in the view when both the menu bar and toolbar are hidden. This
      allows users to hide both of them and still use all features of
      Dolphin.
      53a778c1
  6. 10 May, 2021 1 commit
  7. 02 Jan, 2021 1 commit
  8. 28 Dec, 2020 1 commit
    • Chau Duong's avatar
      Add options to hide some context menu entries · a512176b
      Chau Duong authored and Elvis Angelaccio's avatar Elvis Angelaccio committed
      This commit add options to hide the following context menu entries:
      - Add to Places
      - Copy Location
      - Duplicate Here
      - Open in New Tab and Open in New Tabs
      - Open in New Window
      - Sort By
      - View Mode
      
      The Services settings page is renamed to Context Menu
      
      ShowCopyMoveMenu option is moved from GeneralSettings to ContextMenuSettings
      
      BUG: 314594
      a512176b
  9. 19 Dec, 2020 1 commit
  10. 15 Dec, 2020 1 commit
  11. 07 Dec, 2020 1 commit
  12. 30 Nov, 2020 1 commit
  13. 29 Nov, 2020 1 commit
  14. 23 Nov, 2020 2 commits
  15. 21 Nov, 2020 1 commit
  16. 17 Nov, 2020 1 commit
    • Nate Graham's avatar
      Separate Delete/Move To Trash menu items in context menu · b76cc3f6
      Nate Graham authored
      Right now these actions are in a visual group with "Rename" and "Add to
      Places" which is silly as those actions have no logical connection
      whatsoever to deletion.
      
      This commits moves those items into the section above, putting them in a
      more logical place and leaving the Delete/Move To Trash item in its own
      section.
      
      BUG: 429256
      FIXED-IN: 21.04
      b76cc3f6
  17. 14 Nov, 2020 1 commit
  18. 23 Oct, 2020 2 commits
  19. 13 Oct, 2020 1 commit
  20. 26 Aug, 2020 1 commit
  21. 25 Aug, 2020 1 commit
  22. 29 Jun, 2020 1 commit
    • Yann Holme-Nielsen's avatar
      ## Summary · 86e3b82e
      Yann Holme-Nielsen authored and Elvis Angelaccio's avatar Elvis Angelaccio committed
      * Adds a "Copy location" item, after the "Copy" Context item and Edit Menu, which will attempt to copy the path of the fist item into clipboard.
      
      ## Reasoning
      Most File Managers have this option through one or another way.
      Also using the default Copy option often results in different behaviour depending on the target software, Konsole will take the path. Other Programs will use the URI. Which ultimately could lead to non optimal User Experience.
      
      ## Notes
      * Should the target file **not** be on a local hard drive, this fallback to using the remote URL, feedback is wanted on that matter.
      
      FEATURE: 407004
      86e3b82e
  23. 15 Mar, 2020 2 commits
    • Nate Graham's avatar
      Add Duplicate feature · 158d12ac
      Nate Graham authored and Nate Graham's avatar Nate Graham committed
      Summary: Adds a Duplicate feature to Dolphin, showing up as a menu item in the File menu that appears when one or more items are selected and the directory is writable. Duplicated items receive the names of the original files with " copy" appended before the file extension, if any.
      
      Test Plan:
      
      {F5201386} {F5201393}
      
      Test cases:
      
      - Try to duplicate when nothing is selected: **PASS**: menu item is grayed out
      - Try to duplicate anything on a read-only local volume: **PASS**:  menu item is grayed out
      - Try to duplicate anything on a read-only samba share: **PASS**: menu item is grayed out
      - Duplicate single local file on R/W volume: **PASS**: item is duplicated and named correctly
      - Duplicate multiple local files on R/W volume: **PASS**: 3 items are duplicated, named correctly, and selected
      - Duplicate single local directory on  R/W volume: **PASS**: item is duplicated and named correctly, but a rename operation is not initiated
      - Duplicate multiple local directories on R/W volume: **PASS**: 3 items are duplicated, named correctly, and selected
      - Duplicate single file on R/W samba share: **PASS**: item is duplicated and correctly
      - Duplicate multiple files on R/W samba share: **PASS**: 3 items are duplicated, named correctly, and selected
      - Duplicate single directory on R/W samba share: **PASS**: item is duplicated and named correctly
      - Duplicate multiple directory on R/W samba share: **PASS**: 3 items are duplicated, named correctly, and selected
      - Try to undo a successful duplication: **PASS**: operation is undone
      
      This is my first attempt at a big change like this and I'm sure it's full of issues. I will accept any and all suggestions for improvement. :)
      
      Reviewers: #dolphin, #kde_applications, elvisangelaccio, dfaure, broulik, davidedmundson
      
      Subscribers: kfm-devel, meven, markg, fazevedo, cfeck, #dolphin
      
      Tags: #dolphin
      
      Differential Revision: https://phabricator.kde.org/D8208
      158d12ac
    • Nate Graham's avatar
      Add Duplicate feature · 405dd624
      Nate Graham authored and Nate Graham's avatar Nate Graham committed
      Summary: Adds a Duplicate feature to Dolphin, showing up as a menu item in the File menu that appears when one or more items are selected and the directory is writable. Duplicated items receive the names of the original files with " copy" appended before the file extension, if any.
      
      Test Plan:
      
      {F5201386} {F5201393}
      
      Test cases:
      
      - Try to duplicate when nothing is selected: **PASS**: menu item is grayed out
      - Try to duplicate anything on a read-only local volume: **PASS**:  menu item is grayed out
      - Try to duplicate anything on a read-only samba share: **PASS**: menu item is grayed out
      - Duplicate single local file on R/W volume: **PASS**: item is duplicated and named correctly
      - Duplicate multiple local files on R/W volume: **PASS**: 3 items are duplicated, named correctly, and selected
      - Duplicate single local directory on  R/W volume: **PASS**: item is duplicated and named correctly, but a rename operation is not initiated
      - Duplicate multiple local directories on R/W volume: **PASS**: 3 items are duplicated, named correctly, and selected
      - Duplicate single file on R/W samba share: **PASS**: item is duplicated and correctly
      - Duplicate multiple files on R/W samba share: **PASS**: 3 items are duplicated, named correctly, and selected
      - Duplicate single directory on R/W samba share: **PASS**: item is duplicated and named correctly
      - Duplicate multiple directory on R/W samba share: **PASS**: 3 items are duplicated, named correctly, and selected
      - Try to undo a successful duplication: **PASS**: operation is undone
      
      This is my first attempt at a big change like this and I'm sure it's full of issues. I will accept any and all suggestions for improvement. :)
      
      Reviewers: #dolphin, #kde_applications, elvisangelaccio, dfaure, broulik, davidedmundson
      
      Subscribers: kfm-devel, meven, markg, fazevedo, cfeck, #dolphin
      
      Tags: #dolphin
      
      Differential Revision: https://phabricator.kde.org/D8208
      405dd624
  24. 14 Feb, 2020 1 commit
    • Nate Graham's avatar
      Restore former position of "Create New" menu item in viewport context menu · 4c923a53
      Nate Graham authored
      Summary:
      I've seem various complaints from users about the fact that the {nav Create New...} menu
      item in the viewport context menu is now below {nav Open With} item. All the other
      changes to menus seem to have been well-received but this one has garnered criticism.
      Looking through D23757 and D11884, I can't actually see that this was intentional, so
      it might even be a bug.
      
      Either way, this patch restores the old position of the {nav Create New} menu items at
      the top of the Viewport context menu.
      
      BUG: 417640
      FIXED-IN: 20.04.0
      
      Test Plan: {F8093605}
      
      Reviewers: #vdg, #dolphin, elvisangelaccio, ndavis
      
      Reviewed By: #vdg, ndavis
      
      Subscribers: asturmlechner, markuss, kfm-devel
      
      Tags: #dolphin
      
      Differential Revision: https://phabricator.kde.org/D27267
      4c923a53
  25. 12 Feb, 2020 1 commit
    • Ilya Bizyaev's avatar
      Swap "Open in New Window" and "Open in New Tab" in context menus · 402f7f30
      Ilya Bizyaev authored
      Summary:
      Since Dolphin's default behaviour became opening in new tabs rather than in new windows
      (a very positive change, in my opinion), I always find myself accidentally opening folders
      in new windows from the context menu. This is because in most browsers with tab workflow
      (e.g. Firefox, Chromium, Falkon) the top context menu action is opening in a new tab, and
      my muscle memory plays a bad trick :) I'm aware of middle-clicking, but I find it
      inconvenient with my touchpad.
      
      I suspect I'm probably not alone in that, so I think it makes sense to swap these actions
      in Dolphin to match widely used apps.
      
      Test Plan:
      Open Dolphin, right click folders in the main view and in the Places panel, ensure the
      actions are indeed swapped
      
      Reviewers: #vdg, ndavis, #dolphin, ngraham
      
      Reviewed By: #vdg, ndavis, #dolphin, ngraham
      
      Subscribers: ngraham, ndavis, kfm-devel
      
      Tags: #dolphin
      
      Differential Revision: https://phabricator.kde.org/D27318
      402f7f30
  26. 23 Dec, 2019 1 commit
  27. 06 Dec, 2019 1 commit
  28. 02 Dec, 2019 1 commit
  29. 01 Dec, 2019 1 commit
  30. 23 Nov, 2019 1 commit
  31. 15 Sep, 2019 1 commit
    • Nate Graham's avatar
      Clean up hamburger menu and viewport and single-folder context menus · 0303df09
      Nate Graham authored
      Summary:
      Dolphin's hamburger and context menus have grown organically over time,
      becoming a bit messy and somewhat visually overwhelming. This makes them
      harder to parse and more intimidating to use.
      
      This patch cleans up the hamburger menu and viewport and single-folder context
      menus to group items more logically, and remove items that aren't actually relevant
      to the context.
      
      The hamburger menu part of the patch is fairly significant, and draws from the
      principle of only showing actions with a global scope that are not already accessible
      from another visible method (e.g. via the default toolbar). In the end, it manages to be
      shorter than the current hamburger menu with expose actions that are more relevant.
      
      A visible method to display context-specific actions should be explored separately
      (see https://bugs.kde.org/show_bug.cgi?id=411500).
      
      Depends on D23945
      
      Test Plan:
      Before, hamburger menu: {F7334178}
      After, hamburger menu: {F7350958}
      
      Before, viewport: {F7324802}
      After, viewport: {F7330109}
      
      Before, one folder selected: {F7324798}
      After, one folder selected: {F7341163}
      
      No change for the context menus shown when selecting a single item, multiple items, or multiple folders
      
      Reviewers: #vdg, #dolphin, meven, elvisangelaccio, GB_2
      
      Reviewed By: #vdg, #dolphin, meven, elvisangelaccio, GB_2
      
      Subscribers: GB_2, mmustac, elvisangelaccio, meven, ndavis, kfm-devel
      
      Tags: #dolphin
      
      Differential Revision: https://phabricator.kde.org/D23757
      0303df09
  32. 08 Sep, 2019 1 commit
  33. 04 Sep, 2019 1 commit
    • Luca Beltrame's avatar
      Unbreak context menu showing when right clicking on an empty area · d23fba32
      Luca Beltrame authored
      In D22149, a request was made to remove a variable assignment ("action",
      on line 389 in src/dolphincontextmenu.cpp. However the change went too
      far, and it actually removed the exec() call, not just the assignment
      which should have stayed.
      
      This resulted in the context menu not being shown at all.
      
      Adding back exec(m_pos) at least allows for the context menu to show.
      
      CCMAIL: nate@kde.org
      d23fba32
  34. 01 Sep, 2019 1 commit
    • Nate Graham's avatar
      Add "Add to Places" action to file menu · 7cad80b2
      Nate Graham authored
      Summary:
      It's recommended that actions available in context menus be available in the main menu
      as well for discoverability's sake. This patch does so for the "Add to Places" action.
      
      The action is moved over to the main window, and accessed in the context menu via the
      actionCollection it lives in.
      
      BUG: 390757
      FIXED-IN: 19.08.0
      
      Test Plan:
      - Action still works
      - Action still appears in context menu when relevant
      - Action in the File menu only becomes enabled when only a single directory is selected or nothing is selected
      
      {F7143876}
      
      {F7143877}
      
      {F7143878}
      
      {F7143879}
      
      Reviewers: #dolphin, elvisangelaccio
      
      Reviewed By: #dolphin, elvisangelaccio
      
      Subscribers: elvisangelaccio, kfm-devel
      
      Tags: #dolphin
      
      Differential Revision: https://phabricator.kde.org/D22149
      7cad80b2
  35. 07 Jun, 2019 1 commit
  36. 10 Apr, 2019 1 commit
    • Nate Graham's avatar
      Don't show "Open With" menu items for empty directories · 14f0cd52
      Nate Graham authored
      Summary:
      Any app that registers itself as able to open directories generally can't do anything
      useful with an empty directory. So, don't show the {nav Open With} menu items for them.
      
      Test Plan:
      Before: {F6759793}
      
      After: {F6759794}
      
      The {nav Open With} items still appear for non-empty directories.
      
      Reviewers: #dolphin, elvisangelaccio
      
      Reviewed By: #dolphin, elvisangelaccio
      
      Subscribers: trmdi, kfm-devel
      
      Tags: #dolphin
      
      Differential Revision: https://phabricator.kde.org/D20396
      14f0cd52