Change argument of copy_name to be the paths itself

This way we can remove the --mark flag again. This however, breaks
backward compatibility.

Author: jcjgraf

docs/changelog.rst

@@ -17,15 +17,16 @@ Added:
   * ``6`` or ``PyQt6``: Use PyQt6.
   * ``PySide6``: Use PySide6 (Qt for Python). This is highly experimental and should be
     used with care.
-* The ``--mark`` flag to ``:copy-name`` command. It allows for copying the names of all
-  marked images instead of the currently selected one.
 
 Changed:
 ^^^^^^^^
 
 * The JPEG image header check was simplified to have a false negative rate of 0, while
   maintaining a decently low false positive rate.
 * Support for PyQt5 < 5.15 was dropped.
+* Command ``:copy-name`` takes the paths to copy as an argument. This allows to copy
+  e.g. all marked images, using ``:copy-name %m``. Keybinding for this new functionality
+  have been added.
 
 Fixed:
 ^^^^^^

tests/end2end/features/misc/clipboard.feature

@@ -2,35 +2,35 @@ Feature: Interaction with the system clipboard.
 
     Scenario: Copy basename from library path to clipboard.
         Given I open a directory with 1 paths
-        When I run copy-name
+        When I run copy-name %
         Then the clipboard should contain 'child_01'
 
     Scenario: Copy basename from library path to primary.
         Given I open a directory with 1 paths
-        When I run copy-name --primary
+        When I run copy-name % --primary
         Then the primary selection should contain 'child_01'
 
     Scenario: Copy abspath from library path to clipboard.
         Given I open a directory with 1 paths
-        When I run copy-name --abspath
+        When I run copy-name % --abspath
         # /tmp from the directory in which tests are run
         Then the absolute path of child_01 should be saved in the clipboard
 
     Scenario: Copy basename from image path to clipboard.
         Given I open any image
-        When I run copy-name
+        When I run copy-name %
         Then the clipboard should contain 'image.jpg'
 
     Scenario: Copy and paste basename from library
         Given I open a directory with 1 paths
-        When I run copy-name
+        When I run copy-name %
         And I run paste-name
         Then the working directory should be child_01
 
     Scenario: Copy basename from marked images to clipboard.
         Given I open 5 images
         When I run mark image_01.jpg image_02.jpg
-        And I run copy-name --mark
+        And I run copy-name %m
         Then the clipboard should contain 'image_01.jpg image_02.jpg'
 
     Scenario: Copy image to clipboard.

vimiv/api/_modules.py

@@ -11,7 +11,7 @@
 """
 
 import os
-from typing import List
+from typing import List, Iterable
 
 from vimiv.qt.core import QDateTime
 from vimiv.qt.gui import QGuiApplication, QClipboard
@@ -68,30 +68,33 @@ def toggle(mode: str) -> None:
     api.modes.get_by_name(mode).toggle()
 
 
-@api.keybindings.register("yA", "copy-name --abspath --primary")
-@api.keybindings.register("yY", "copy-name --primary")
-@api.keybindings.register("ya", "copy-name --abspath")
-@api.keybindings.register("yy", "copy-name")
-@api.keybindings.register("ymA", "copy-name --abspath --primary --mark")
-@api.keybindings.register("ymY", "copy-name --primary --mark")
-@api.keybindings.register("yma", "copy-name --abspath --mark")
-@api.keybindings.register("ymy", "copy-name --mark")
+@api.keybindings.register("yA", "copy-name % --abspath --primary")
+@api.keybindings.register("yY", "copy-name % --primary")
+@api.keybindings.register("ya", "copy-name % --abspath")
+@api.keybindings.register("yy", "copy-name %")
+@api.keybindings.register("ymA", "copy-name %m --abspath --primary")
+@api.keybindings.register("ymY", "copy-name %m --primary")
+@api.keybindings.register("yma", "copy-name %m --abspath")
+@api.keybindings.register("ymy", "copy-name %m")
 @api.commands.register()
-def copy_name(abspath: bool = False, primary: bool = False, mark: bool = False) -> None:
-    """Copy name of current path or of marked images to system clipboard.
+def copy_name(
+    paths: Iterable[str], abspath: bool = False, primary: bool = False
+) -> None:
+    """Copy file name or full path of provided paths(s) to system clipboard.
+
+    **syntax:** ``:copy-name path [path ...] [--abspath] [--primary]``
 
-    **syntax:** ``:copy-name [--abspath] [--primary] [--mark]``
+    positional arguments:
+        * ``paths``: The path(s) to copy.
 
     optional arguments:
         * ``--abspath``: Copy absolute path instead of basename.
         * ``--primary``: Copy to primary selection.
-        * ``--mark``: Copy all marked images instead of current path.
     """
     clipboard = QGuiApplication.clipboard()
     mode = QClipboard.Mode.Selection if primary else QClipboard.Mode.Clipboard
-    paths = api.mark.paths if mark else [api.current_path()]
-    names = [path if abspath else os.path.basename(path) for path in paths]
-    clipboard.setText(" ".join(names), mode=mode)
+    text = " ".join(path if abspath else os.path.basename(path) for path in paths)
+    clipboard.setText(text, mode=mode)
 
 
 @api.keybindings.register("yi", "copy-image")