Closed #2735.
--
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/issues/2735#event-4253729045
My bad guys - mine was located elsewhere and got overridden by something on my
mach. 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/issues/2735#issuecomment-767986941
It's not being stored there, using Geany 1.36 Using GTK+ v3.24.20 and GLib
v2.64.3 runtime libraries. I stored a command for send input to Ctrl - 1, it is
not found in the geany.conf file
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view i
What config file if any stores the "Send selection to"/Custom commands? I have
had my custom commands wiped out twice recently, not sure why maybe because of
an update?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https:
What worked for me is changing the font, View->Change Font to Nimbus Mono PS
Regular
--
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/issues/2502#issuecomment-744836684
@codebrainz If removing the ellipsizing works reliably I'm fine with it, but if
that needs more extensive testing maybe we can put that into a new PR? This
will improve it quick and safe for the time being.
--
You are receiving this because you are subscribed to this thread.
Reply to this emai
AdamDanischewski commented on this pull request.
The change request from b4n is already implemented.
> editor->sci = editor_create_widget(editor);
+ sci_set_multipaste(editor->sci, SC_MULTIPASTE_EACH);
That's where it is now.
--
You are receiving this because you
@elextr:
> There may be lots of complicated things that could be done here, but hey if a
> fixed 30 has worked for most people to date then a fixed 80 is no worse.
@elextr Thanks, my sentiments exactly.
Eventually the hardcoded value could/should probably be moved to a defined
constant. May
Changed the default file name length in the title bar from 30 to 80.
Fulfills: Enhancement Request - File Name in Title Bar Too Short #2334
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/2335
-- Commit Summary --
* Changed default title
The title of the file that I am working on is truncated in the geany title bar.
![Screenshot from 2019-10-02
09-59-12](https://user-images.githubusercontent.com/10157303/66050675-9fc62d00-e4fb-11e9-87bc-6b23e2bc8a93.png)
I'd like it to show the full name of the file, like this:
![Screenshot from
Don't worry guys, all fixed. =)
--
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/2328#issuecomment-537102001
@AdamDanischewski pushed 2 commits.
a1a185e87c085d330c15e623f5005886ce5c24b0 Reverted - a blank line at the end
was missing
79923d281814d95ffebe1a2cb44381cd31d7d4b1 Moved SC_MULTIPASTE_EACH setting to
create_new_sci() where the other direct scintilla settings are.
--
You are receiving this
@b4n It sounds like you're not really interested in crediting me for fixing
this - does this mean I won't be added to the list of contributors?
--
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/
Updated - SC_MULTIPASTE_EACH is now set in
geany/src/editor.c::editor_create_widget() without any wrappers since this is
now the default.
--
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/
@AdamDanischewski pushed 1 commit.
b78159d71fa989132155ccd879e79ae24df00e76 Setting SC_MULTIPASTE_EACH is no
longer exposed
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/geany/geany/pull/2328/files
@AdamDanischewski pushed 1 commit.
921142866e85a851ca2341f4ce16e96c6d6d6959 Setting SC_MULTIPASTE_EACH no longer
exposed
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/geany/geany/pull/2328/files
@AdamDanischewski pushed 1 commit.
f22fc13f9d19b68c5dcac2bba3165dfc575ecaef SC_MULTIPASTE_EACH is default
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/geany/geany/pull/2328/files/63d3a9b04230129a296d991a73388412971966e3
AdamDanischewski commented on this pull request.
> @@ -39,9 +39,10 @@ sptr_t sci_send_message_internal (const gchar *file, guint
> line, ScintillaObject
# endif
#endif
-void sci_set_text
(ScintillaObject *sci, const gchar
AdamDanischewski commented on this pull request.
> @@ -56,162 +57,162 @@ void
> sci_set_current_position(ScintillaObject *sci, gint position, gboolean
gint sci_get_selection_start
(ScintillaObject *sci);
AdamDanischewski commented on this pull request.
> @@ -93,6 +93,10 @@ sptr_t sci_send_message_internal (const gchar *file, guint
> line, ScintillaObject
}
#endif
+ /* Set multi paste setting (at 3104 scintilla defaults to SC_MULTIPASTE_ONCE)
*/
+void sci_set_multipaste(Scintilla
I tried to be nice and clean it up, I really don't have time for this project.
Go ahead and review the code and put it through or let me know if there is an
actual problem. The team here seems very ungrateful for an extra helping hand
at fixing a problem that's making Geany look bad.
@elextr Y
This also has comments, it still needs to be re-reviewed. No?
--
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/2328#issuecomment-536234304
@codebrainz They are the same. #2325 was approved but had a subsequent update.
I'll keep it on the same PR in the future. For now I'll leave this open since
there is no point to going back to #2325.
--
You are receiving this because you are subscribed to this thread.
Reply to this email direct
SC_MULTIPASTE_ONCE to SC_MULTIPASTE_EACH
- fixes the paste only one line when a vertical selection is selected.
* Update Editor.cxx
* Moved scintilla SC_MULTIPASTE_EACH default setting to Geany.
* Merge branch 'multipaste' of
https://github.com/AdamDanischewski/geany into multipaste
'multipaste' of
https://github.com/AdamDanischewski/geany into multipaste
* Update sciwrappers.h
* Update Editor.cxx
-- File Changes --
M src/editor.c (3)
M src/sciwrappers.c (4)
M src/sciwrappers.h (165)
-- Patch Links --
https://github.com/geany/geany/pull/2327.p
Copy paste error.
--
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/2325#issuecomment-535943033
Closed #2325.
--
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/2325#event-2668676984
@AdamDanischewski pushed 1 commit.
f76cf9d4f52388f4358db897779b6ae2726e20af Update sciwrappers.h
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/geany/geany/pull/2325/files/a33ec297e6ef648373bda4aa7f7c3304b9e4700f
@AdamDanischewski pushed 2 commits.
f4672f1ff64a045aaa1845edee57dfa02ce0a809 Moved scintilla SC_MULTIPASTE_EACH
default setting to Geany.
a33ec297e6ef648373bda4aa7f7c3304b9e4700f Merge branch 'multipaste' of
https://github.com/AdamDanischewski/geany into multipaste
--
You are
@AdamDanischewski pushed 1 commit.
5a127803b5c15f100114992ddb2955484a9b748d Update Editor.cxx
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/geany/geany/pull/2325/files/3a1df4e7e44246b3b1353dd6583c1b4b3514e374
Changed the scintilla default in (/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:
https://github.com/geany/geany
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;`
![vertical_paste_fix](https://user-images.g
@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
@ntrel Yea, I played around with it before and it's expected behavior. I kind
of think multi-paste should work the way @Akronix mentioned in #850. If you
have one line and then make a vertical selection (rectangular selection) that
it multi-pastes across the entire selection but there's still so
Closed #2317.
--
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/issues/2317#event-2659856736
@intact Yep yep! That plugin does the job - thanks! I didn't notice it for some
reason.
If you are running Geany 1.36 you can add them by:
```
$> git clone https://github.com/geany/geany-plugins.git
$> ./autogen.sh
$> ./configure
$> make
$> mkdir mylib; find . -type f -name '*.so' -exec cp {
@LarsGit223 Lol, when it rains it pours! I couldn't find anything nor common
terminology so I thought I'd save others the trouble after stumbling onto other
implementations.
@ntrel I've tried using the Send Selection To --> interface but
it doesn't paste back across the vertical selection. It
Sublime also has Increment Selection - I guess that's what it should be
called?? - available via the Increment Selection package:
![9kBFH](https://user-images.githubusercontent.com/10157303/65513663-a9bbb080-dea9-11e9-8186-247ff82f251f.gif)
https://stackoverflow.com/questions/14640093/how-to-inse
This feature is also available in Visual Studio via extension:
![demo](https://user-images.githubusercontent.com/10157303/65512377-13868b00-dea7-11e9-9f41-3defd5c06daa.gif)
https://marketplace.visualstudio.com/items?itemName=albymor.increment-selection
https://github.com/albymor/Increment-Selectio
I've been searching for vertical editing sequential numbering, I've found:
https://www.emeditor.com/text-editor-features/coding/multiple-selection-editing/how-to-use-vertical-editing/
On vertical selection, a menu option allows for sequential numbering to be
inserted. This would be a welcome add
You seem to have not really wrapped your brain around the idea here. What I
meant to articulate is a way to provide visuals with regards to ANY editor
behavior. Ideally you could launch a notifications editor and select the skins
for various items and attach them to any functionality and perhaps
I think it would help productivity and reduce stress if there were little
graphics that kind of sauntered up when doing things that are presently met
with invisibility.
For instance when you copy something, you hit Ctrl-C that's fine, most people
know that it works and are somewhat comfortable
@Ryan1729 That sounds like a reasonable plan to move forward -- since it is
unknown what the effects will be, just make it an option to let those who want
to use it, give it a shot and they can report if there are issues - fixing them
where possible (although it probably will require an overhaul
Note: This issue is fixed, yet the colorization is still wrong for HERE-DOC's.
It is broken in Scintilla there is an open bug yet it's a low priority, it
should probably get "auto-fixed" via Scintilla at some point.
--
You are receiving this because you are subscribed to this thread.
Reply to
Since the current default behavior is different, it may be argued to not make
it a default. Yet the functionality seems to be much better and I personally
would prefer to see Geany remain consistent with other editors that offer this
same functionality. It is a quirk in my opinion that Geany usi
Kind of like textadept now that I've played around with it, you have to a hack
a colorizer (theme) to get the color scheme to Geany though. Copy and modify a
theme in the themes directory under the source directory (exploded tar ball).
Presuming you named your cloned theme to be "your-new-theme
@elextr It does say that it uses Scintilla as the core text editing component,
when I first browsed the code it looks like mainly lua, looking closer I see
some calls that create a Scintilla document. See textadept.c line no. 1355.
```
/**
* Creates a new Scintilla document and adds it to the
@diohja Textadept does seem to have the feature yet it doesn't look like it's
really using Scintilla. It includes ScintillaLua a lexer for Scintilla. The
Scintilla functionality appears to have been recreated in Lua.
--
You are receiving this because you are subscribed to this thread.
Reply t
Lol, yea I was only scrolling up only to his new comment for some reason -
disregard the clarification request. I see what you are looking for now, best
of luck. I developed the first patch for the multiselect functionality that's
working now. I didn't implement it ultimately in Geany because of
@elextr It sounds like you want something like crtrl-click multipositional
multiselect
Can you describe in words what you mean by the missing functionality? (or
better yet post an animated Gif from it working on another editor)
--
You are receiving this because you are subscribed to this th
> How do you know it didn't work?
Ryan by Ctrl-click do you mean Ctrl + Shift Click? If you haven't tried that,
give it a shot I think that provides the functionality that is referred to in
the API documents.
--
You are receiving this because you are subscribed to this thread.
Reply to this e
I don't have much time - but if anyone wants to take a crack at finding
sensible settings for a healthy memory reserve threshold, to preclude freezes
and data loss. I took a quick look via strace, and a strace output parser I
found (https://github.com/gmarcais/memtrace), here is what my system l
@elextr You can profile the application for average use cases. Then choose a
sensible default setting that can be adjusted by the user if they know their
needs better.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https:
> To be clear Scintilla does keep a "healthy" amount of memory allocated, but
> it decreases as you add characters to it until its full then reallocates, and
> if you do a big copy and paste you can exceed ANY pre-allocation, so there is
> no point of trying to guess, you always have to handle a
> Its not even possible to report if a program is running out of address space
> (which is actually what the 32bit windows problem is) because many library
> functions use memory that Geany has no idea about.
@elextr I realize that Geany/Scintilla won't know what the rest of the machine
will
This may be beyond the scope of Geany, but it looks like `vi` was designed to
be efficient for use on a 300 baud modem, so it loads lines on demand. It may
be worth looking into the source code, yet is a bit hackish.
> "Author of Scintilla here. Scintilla does not use a list of lines. The text
At 300MB on a Linux box my Geany freezes up too. Have you considered breaking
the file into smaller chunks for processing? E.g. The way syslog manages logs
by creating sequenced files. If you need to keep that size and edit it every so
often, I recommend you use an editor that can *NOT* load the
It may seem daunting because of the amount of rows, but it shouldn't take very
long since its a binary search the complexity is O(log n) at worst.
--
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/gea
Thanks, when the next release is out it should be easy to quickly test and then
we can close this. In the meantime if anyone is frustrated about this a
workaround is to temporarily comment out the problematic array element
assignments - that will put the highlighting back to normal and you can d
Have you tried segmenting the file? Copy it someplace and cut it in half, are
both sides broken? If so keep cutting it in half again until you find a half
not broken then focus on the smallest broken half continuing to narrow it down
to the problem area.
--
You are receiving this because you a
`grep`'ing `'[^\x00-\x7F]'` will show you everything that is not in the ascii
range. You are negating the set so it's not going to show you `\x00` that's
null and represented in the ascii range. Is there are a reason there are so
many non ascii characters in this file? If there is other code hit
Thanks, I hope so. I'll let them close it themselves if it has, I don't have
the environment to test.
--
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/issues/1549#issuecomment-316772331
It looks like someone broke Scintilla, I've launched a bug ticket on Scintilla:
https://sourceforge.net/p/scintilla/bugs/1963/
--
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/issues/1549#issuec
I had a problem with a file not opening at all, it turned out that there was a
NULL byte embedded in the file. Check the file for special characters, `cat
-vet` may help if its not too big otherwise `grep`.
--
You are receiving this because you are subscribed to this thread.
Reply to this email
I've got a Bash script and it is using arrays (something the other shells don't
have, maybe that's why this isn't more thoroughly tested?). It cannot handle
nested parenthesis, I'm not sure if this problem is actually Scintilla.
Here is how to reproduce, enter the following into a file t.bsh:
Its not exactly "mind reading" to presume that someone wants a single row
pasted into every row of a multi-line selection. An option being required to
turn off this behavior? If you really want to paste to a single line to a
single row then don't do a multi-line select beforehand - simply put th
Lol, I'm not sure of the Geany version at the time of the correspondence - yet
your issue is now fixed to the best of my ability to discern. Now, it is this
new "problem"/non-feature of not multi-pasting text from a single row that is
copied. If you copy text from a single row then multi-select
The multiple line past still needs work, as of Git 1.31.
Here is what works:
https://cloud.githubusercontent.com/assets/2631430/18294391/1b2bdeb2-7499-11e6-904b-d0b0db5d735d.gif"/>
What I would like to see working - is when you copy some text single line, then
select multiple rows in a multi-li
Reopened #850.
--
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/issues/850#event-1125297704
Closed #850.
--
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/issues/850#event-1125268346
Reopened #850.
--
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/issues/850#event-1125240382
![myanimated](https://cloud.githubusercontent.com/assets/10157303/18317818/3ea19b44-74ed-11e6-81da-c6a6d285ecab.gif)
Is what I'm talking about.
--
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/i
@Akronix It does that already, that's why this closed.
--
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/issues/850#issuecomment-244852751
Closed #850.
--
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/issues/850#event-764880086
@codebrainz Thanks, that's the functionality I was referring to when I opened
this -- I am going to close this now.
--
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/issues/850#issuecomment-2418
Are you sure you know what I'm talking about? I don't have 1.29 to test, but
hold `Shift+Alt` in geany and select a block of vertical text, then `Ctrl+C`
and move down and enter some empty spaces (enough lines to cover your
selection) and go to the top and `Ctrl+V` if you get pasted what you sel
It looks like the multi-select copy/paste insert portion of this request was
added as of 1.23.1.
--
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/issues/1141#issuecomment-241274900
As of Geany 1.23.1 this seems to be working now. Thanks, I'm pretty sure this
can be closed now.
--
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/issues/850#issuecomment-241267784
>From my uses, I think it makes sense to default to presuming the user means
>every line. I'm not sure I know why a user, especially if they have a big
>clipboard stored, would expect it to land on line one only if they have
>multilines selected.
Perhaps you could put it as a checkbox in Prefer
I would like to see the Multipaste option for Scintilla implemented I hacked
together something that works, but someone just needs to add a new option in
Preferences somewhere to set it
add to sciwrappersc:
void sci_set_multi_paste(ScintillaObject *sci, gboolean mpval) { SSM(sci,
SCI_SETMULTI
I really like the vertical editing capability of Geany but I noticed that the
paste feature only hits one line, I think the logic should presume when
multiple lines are vertically selected that a paste issued should be applied to
all lines in the vertical selection (instead of only the first one
81 matches
Mail list logo