fredvs / ideU

The ide. Universal.
GNU General Public License v2.0
63 stars 9 forks source link

Delete word at cursor interferes with clipboard #23

Closed JonnyTech closed 4 years ago

JonnyTech commented 4 years ago

Just tried the new delete word at cursor feature but found that it cleared the contents of my clipboard and replaced it with the deleted word. Is this by design? If so can you add an option to delete the word without touching the clipboard buffer please?

fredvs commented 4 years ago

Ha, ok, I did miss that, ok I will check this. Thanks to note it.

fredvs commented 4 years ago

Hello. Should be fixed in last commit 502e8d6.

Thanks to note it.

JonnyTech commented 4 years ago

Thanks @fredvs, shall check tomorrow.

Does lines 1280- in actionsmodule.pas also need changing?:

procedure tactionsmo.deletewordatcursor(const sender: tcustomaction);
begin
sourcefo.activepage.copywordatcursor();
sourcefo.activepage.edit.cutselection;

end;
fredvs commented 4 years ago

Thanks @fredvs, shall check tomorrow.

Does lines 1280- in actionsmodule.pas also need changing?:

procedure tactionsmo.deletewordatcursor(const sender: tcustomaction);
begin
sourcefo.activepage.copywordatcursor();
sourcefo.activepage.edit.cutselection;

end;

Huh, it is exactly what is changed in commit 502e8d6....

procedure tactionsmo.ondeleteword(const sender: TObject);
begin
  sourcefo.activepage.selectwordatcursor();
 sourcefo.activepage.edit.deleteselection;
end;
JonnyTech commented 4 years ago

Oh, ok, because they are two procedures at different places in the same file: deletewordatcursor and ondeleteword.

fredvs commented 4 years ago

Oh, ok, because they are two procedures at different places in the same file: deletewordatcursor and ondeleteword.

Yes before commit 502e8d6, now it is fixed and there is only ondeleteword

Note that selectwordatcursor was added in 502e8d6 commit.

fredvs commented 4 years ago

@JonnyTech Ho, I apologize, you are totally right, sorry, deletewordatcursor should be replaced by selectword.

Many thank to note it, I will fix it now.

fredvs commented 4 years ago

Done in commit 122f70b. @JonnyTech : Well seen!

JonnyTech commented 4 years ago

Thanks Fred... don't you ever sleep??

fredvs commented 4 years ago

No, sorry, I dont have time for that.

;-)