https://bugs.kde.org/show_bug.cgi?id=514401

Méven <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/syst
                   |                            |em/dolphin/-/commit/28ed413
                   |                            |99aab91a040ce9fbf3510df5893
                   |                            |777dd6
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #18 from Méven <[email protected]> ---
Git commit 28ed41399aab91a040ce9fbf3510df5893777dd6 by Méven Car.
Committed on 18/07/2026 at 12:02.
Pushed by meven into branch 'master'.

Make inline-rename re-triggering robust and add a regression test

Triggering inline renaming again on the item already being renamed (for
example pressing F2 twice) must not disturb the running edit. Two follow-ups:

- DolphinView::renameSelectedItems() connects roleEditingFinished to
  slotRoleEditingFinished inside a deferred scrollingStopped lambda. A second
  trigger runs that lambda again and used to add a duplicate connection. Pass
  Qt::UniqueConnection so the second trigger is a no-op.

- Add KItemListControllerTest::testRestartingInlineRenameKeepsEditor(),
  asserting that a second editRole() on the item already being edited does not
  cancel the ongoing edit (which would silently drop the pending rename).

M  +26   -1    src/tests/kitemlistcontrollertest.cpp
M  +1    -1    src/views/dolphinview.cpp

https://invent.kde.org/system/dolphin/-/commit/28ed41399aab91a040ce9fbf3510df5893777dd6

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to