Skip to content
Commit 1a2867c8 authored by Jonah Brüchert's avatar Jonah Brüchert 🌳 Committed by Tomaz Canabrava
Browse files

TerminalDisplay: Extract copying related functions

parent 8e8b9243
  • Hi, I am not sure that it's this commit which breaks my konsole but I rebuild it this morning and now each time that I select to copy a text in konsole, and I use mouse middle button for pasting it paste text as "editorsendcheckplugins
    " => it's not good

    but CTRL+C/CTRL+V works fine.

    Regardd

  • I think the bug is clear: text (line 2181) is created with the ConvertToHtml flag, and used for both plain text and html. Before this commit, selectedText() was called twice, with different options, for plain text and for html. The copyToX11Selection method needs to take two QStrings as input, one for each format. I would do it, but I'm on the stable branch, not master :-)

    @tcanabrava @jbbgameich

  • Thanks for investigating, I'll look into it.

    PS: Sorry for breaking it in the first place.

    Edited by Jonah Brüchert
  • @mlaurent I'm not 100% sure I understand the bug correctly, but can you test whether !421 (merged) fixes it for you?

  • IIUC, the bug happens when you select text, then go to paste it with middle click, then you get the extra html tags. But if you copy the text, the paste it with e.g. Ctrl+V then everything works as expected and the copy text as HTML config option is respected.

  • @jbbgameich I confirm that !421 (merged) fix my bug. Thanks

  • mentioned in commit df9ec0ad

    Toggle commit list
  • mentioned in commit 3baa2aa8

    Toggle commit list
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment