Changed the scintilla default in (geany root/scintilla/src/Editor.cxx
Line 167) from SC_MULTIPASTE_ONCE to SC_MULTIPASTE_EACH - fixes the paste only
one line when a vertical selection is selected.
You can view, comment on, or merge this pull request online at:
> Ok @elextr you talked me into looking into the code finally.
Heh, I wasn't actually trying to do that, but while you are there, "just" a
menu item and keybinding to go. :grin:
I was actually trying to suggest to be patient until the Scintilla 4 upgrade,
the "it needs c++17" excuse is getting
Ok @elextr you talked me into looking into the code finally. I've got it
working with the current Scintilla.
### line 167 geany/scintilla/src/Editor.cxx:
**from** `multiPasteMode = SC_MULTIPASTE_ONCE;`
**to** `multiPasteMode = SC_MULTIPASTE_EACH;`
Note, beware that most of the Scintilla multiple selection documents section is
only for Scintilla 4, which Geany does not use, and which could allow the whole
area of rectangular/multiple selections and pastes to be re-assessed.
--
You are receiving this because you are subscribed to this
Ahh, the obligatory last minute windows change, the release can go ahead as
soon as this is merged :grin:
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Thanks.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2315#issuecomment-535678680
Merged #2315 into master.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2315#event-244105
@AdamDanischewski in this case you don't need an option, just put both items in
the menu, and make both keybindable and bind one or both by default.
But note that I doubt that multipaste will help with pasting the output of
`send selection to` since it would paste the multiple lines on every
I tested the changes last week when creating the PR and I think the changes are
simple enough. Most of the work has been done already when doing the same
changes on the Geany installer a few months ago.
--
You are receiving this because you are subscribed to this thread.
Reply to this email
@Akronix I agree with your sentiments on this.
@elextr I would be fine with there being an option for it.
Perhaps it could be the default provided no one can give a better use case for
it not being. To only paste one line when a vertical multiple line
selection/rectangular selection were held
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/2324
-- Commit Summary --
* Update zh_CN.po
-- File Changes --
M po/zh_CN.po (10)
-- Patch Links --
https://github.com/geany/geany/pull/2324.patch
Merged #2318 into master.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2318#event-2664116967
@eht16 Any tests needed or just merging?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/905#issuecomment-535365584
13 matches
Mail list logo