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

Méven <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/syst |https://invent.kde.org/syst
                   |em/dolphin/-/commit/28ed413 |em/dolphin/-/commit/7ca6dae
                   |99aab91a040ce9fbf3510df5893 |a69a38c006a001bb423b9398b63
                   |777dd6                      |9d027f

--- Comment #19 from Méven <[email protected]> ---
Git commit 7ca6daea69a38c006a001bb423b9398b639d027f by Méven Car.
Committed on 19/07/2026 at 09:43.
Pushed by meven into branch 'release/26.08'.

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).
(cherry picked from commit 28ed41399aab91a040ce9fbf3510df5893777dd6)

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

https://invent.kde.org/system/dolphin/-/commit/7ca6daea69a38c006a001bb423b9398b639d027f

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

Reply via email to