Ticket #2155 (closed enhancement: fixed)

Opened 9 years ago

Last modified 9 years ago

When editor_persistent_selections=0 pasting from clipboad deletes currently selected block (paste over current selection).

Reported by: nickk9 Owned by: angel_il
Priority: minor Milestone: 4.7.2
Component: mcedit Version: 4.7.1
Keywords: Cc: zaytsev
Blocked By: Blocking:
Branch state: Votes for changeset: zaytsev andrew_b

Description

It would be great to implement 'paste over current selection' feature when persistent selection is off. So, when one press paste key sequence, mcedit not only inserts clipboard content but also deletes currently selected block.

Attachments

edit.patch (556 bytes) - added by nickk9 9 years ago.

Change History

Changed 9 years ago by nickk9

comment:1 Changed 9 years ago by nickk9

  • Status changed from new to accepted
  • Owner set to nickk9

Here is proposed patch for src/editor/edit.c which implements desired feature.

comment:2 Changed 9 years ago by angel_il

  • Owner changed from nickk9 to angel_il
  • severity changed from no branch to on review

branch: 2155_remove_selection_before_insert
changeset: c0213c9f9cba83406c0ef157ce119224cbe6be10

comment:3 follow-up: ↓ 5 Changed 9 years ago by zaytsev

  • Votes for changeset set to zaytsev

Change the commit text to:

    Remove selected text before input or paste from clipboard.
    Thanks to nikk9 <dev.nul@…> for the original patch.

comment:4 Changed 9 years ago by zaytsev

  • Cc zaytsev added

comment:5 in reply to: ↑ 3 Changed 9 years ago by angel_il

Replying to zaytsev:

Change the commit text to:

Remove selected text before input or paste from clipboard.
Thanks to nikk9 <dev.nul@…> for the original patch.

done: 829373c04f674b87ad211a6037e18ee7e33ac760

comment:6 Changed 9 years ago by andrew_b

  • Votes for changeset changed from zaytsev to zaytsev andrew_b
  • severity changed from on review to approved

comment:7 Changed 9 years ago by andrew_b

  • Milestone changed from 4.7 to 4.7.2

comment:8 Changed 9 years ago by angel_il

  • Status changed from accepted to testing
  • Resolution set to fixed
  • severity changed from approved to merged

comment:9 Changed 9 years ago by angel_il

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.