Re: [Github-comments] [geany/geany] Question: Where are the "Send selection to"/Custom commands stored? (#2735)
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
Re: [Github-comments] [geany/geany] Question: Where are the "Send selection to"/Custom commands stored? (#2735)
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
Re: [Github-comments] [geany/geany] Question: Where are the "Send selection to"/Custom commands stored? (#2735)
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 it on GitHub: https://github.com/geany/geany/issues/2735#issuecomment-767983487
[Github-comments] [geany/geany] Question: Where are the "Send selection to"/Custom commands stored? (#2735)
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://github.com/geany/geany/issues/2735
Re: [Github-comments] [geany/geany] Can't see underscores on Ubuntu 20.04 (#2502)
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
Re: [Github-comments] [geany/geany] Changed default title length from 30 to 80. (#2335)
@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 email directly or view it on GitHub: https://github.com/geany/geany/pull/2335#issuecomment-538702317
Re: [Github-comments] [geany/geany] Multipaste (#2328)
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 are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/2328#pullrequestreview-297814298
Re: [Github-comments] [geany/geany] Changed default title length from 30 to 80. (#2335)
@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. Maybe a refactoring beautification PR? FYI: this value is hardcoded twice, it's also in document.c: ``` 419 if (length < 0) 420 length = 30; ``` I left it 30 since a filename length less than zero probably doesn't need 80. -- 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/2335#issuecomment-537978528
[Github-comments] [geany/geany] Changed default title length from 30 to 80. (#2335)
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 length from 30 to 80. -- File Changes -- M src/ui_utils.c (2) -- Patch Links -- https://github.com/geany/geany/pull/2335.patch https://github.com/geany/geany/pull/2335.diff -- 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/2335
[Github-comments] [geany/geany] Enhancement Request - File Name in Title Bar Too Short (#2334)
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 2019-10-02 09-58-06](https://user-images.githubusercontent.com/10157303/66050708-b076a300-e4fb-11e9-9901-071d85d70cee.png) This is especially useful when writing unit test cases, the file names are similar and often very long. -- 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/2334
Re: [Github-comments] [geany/geany] Multipaste (#2328)
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
Re: [Github-comments] [geany/geany] Multipaste (#2328)
@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 because you are subscribed to this thread. View it on GitHub: https://github.com/geany/geany/pull/2328/files/b78159d71fa989132155ccd879e79ae24df00e76..79923d281814d95ffebe1a2cb44381cd31d7d4b1
Re: [Github-comments] [geany/geany] Multipaste (#2328)
@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/pull/2328#issuecomment-537078532
Re: [Github-comments] [geany/geany] Multipaste (#2328)
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/2328#issuecomment-537068513
Re: [Github-comments] [geany/geany] Multipaste (#2328)
@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/921142866e85a851ca2341f4ce16e96c6d6d6959..b78159d71fa989132155ccd879e79ae24df00e76
Re: [Github-comments] [geany/geany] Multipaste (#2328)
@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/f22fc13f9d19b68c5dcac2bba3165dfc575ecaef..921142866e85a851ca2341f4ce16e96c6d6d6959
Re: [Github-comments] [geany/geany] Multipaste (#2328)
@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..f22fc13f9d19b68c5dcac2bba3165dfc575ecaef
Re: [Github-comments] [geany/geany] Multipaste (#2328)
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 *text); -gboolean sci_has_selection (ScintillaObject *sci); -void sci_end_undo_action (ScintillaObject *sci); +voidsci_set_multipaste (ScintillaObject *sci, gint mpval); Again the reason I did this is because @elextr suggested that it should be an option. -- 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#discussion_r330068868
Re: [Github-comments] [geany/geany] Multipaste (#2328)
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); gint sci_get_selection_end (ScintillaObject *sci); -void sci_replace_sel (ScintillaObject *sci, const gchar *text); +void sci_replace_sel (ScintillaObject *sci, const gchar *text); The indentation is horrid in this file - I tried to be nice, it won't happen again. -- 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#discussion_r330066565
Re: [Github-comments] [geany/geany] Multipaste (#2328)
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(ScintillaObject *sci, gint mpval) { + SSM(sci, SCI_SETMULTIPASTE, mpval, 0); +} The reason I did this is because @elextr suggested that it should be an option. I agree that it should be the only behavior. -- 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#discussion_r330064994
Re: [Github-comments] [geany/geany] Multipaste (#2328)
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 You can open up your own PR for the cosmetics. ;-) -- 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-536293221
Re: [Github-comments] [geany/geany] Multipaste (#2328)
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
Re: [Github-comments] [geany/geany] Multipaste (#2328)
@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 directly or view it on GitHub: https://github.com/geany/geany/pull/2328#issuecomment-536214261
[Github-comments] [geany/geany] Multipaste (#2328)
Changed sci_wrapper.c::sci_set_multipaste() from hardcoded value SC_MULTIPASTE_EACH to function parameter val. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/2328 -- Commit Summary -- * Changed the scintilla default from 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 * Update sciwrappers.h * Update Editor.cxx * Update sciwrappers.c -- 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/2328.patch https://github.com/geany/geany/pull/2328.diff -- 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
[Github-comments] [geany/geany] Multipaste SC_MULTIPASTE_EACH default moved to Geany. (#2327)
Scintilla is at defaults. Geany has a new wrapper `voidsci_set_multipaste (ScintillaObject *sci, gint mpval);` set to SC_MULTIPASTE_EACH in geany/src/editor.c::editor_create(). You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/2327 -- Commit Summary -- * Changed the scintilla default from 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 * 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.patch https://github.com/geany/geany/pull/2327.diff -- 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/2327
Re: [Github-comments] [geany/geany] Changed the scintilla default from SC_MULTIPASTE_ONCE to SC_MULTIPAST… (#2325)
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
Re: [Github-comments] [geany/geany] Changed the scintilla default from SC_MULTIPASTE_ONCE to SC_MULTIPAST… (#2325)
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
Re: [Github-comments] [geany/geany] Changed the scintilla default from SC_MULTIPASTE_ONCE to SC_MULTIPAST… (#2325)
@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..f76cf9d4f52388f4358db897779b6ae2726e20af
Re: [Github-comments] [geany/geany] Changed the scintilla default from SC_MULTIPASTE_ONCE to SC_MULTIPAST… (#2325)
@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 receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/geany/geany/pull/2325/files/5a127803b5c15f100114992ddb2955484a9b748d..a33ec297e6ef648373bda4aa7f7c3304b9e4700f
Re: [Github-comments] [geany/geany] Changed the scintilla default from SC_MULTIPASTE_ONCE to SC_MULTIPAST… (#2325)
@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..5a127803b5c15f100114992ddb2955484a9b748d
[Github-comments] [geany/geany] Changed the scintilla default from SC_MULTIPASTE_ONCE to SC_MULTIPAST… (#2325)
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: https://github.com/geany/geany/pull/2325 -- Commit Summary -- * Changed the scintilla default from SC_MULTIPASTE_ONCE to SC_MULTIPASTE_EACH - fixes the paste only one line when a vertical selection is selected. -- File Changes -- M scintilla/src/Editor.cxx (2) -- Patch Links -- https://github.com/geany/geany/pull/2325.patch https://github.com/geany/geany/pull/2325.diff -- 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
Re: [Github-comments] [geany/geany] Enhancement - Vertical Editing Sequential Numbering (#2317)
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.githubusercontent.com/10157303/65728722-cdd5e800-e089-11e9-8d12-cbe4c6934a94.gif) -- 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#issuecomment-535706906
Re: [Github-comments] [geany/geany] Enhancement - Vertical Editing Sequential Numbering (#2317)
@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 on prior to pasting seems to imply that a user error is being corrected since it ignores the additional user efforts. -- 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#issuecomment-535602733
Re: [Github-comments] [geany/geany] Enhancement - Vertical Editing Sequential Numbering (#2317)
@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 some debate about it. Maybe a vote would help? -- 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#issuecomment-535054702
Re: [Github-comments] [geany/geany] Enhancement - Vertical Editing Sequential Numbering (#2317)
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
Re: [Github-comments] [geany/geany] Enhancement - Vertical Editing Sequential Numbering (#2317)
@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 {} mylib/ \; ``` Then in Geany, add the plugin path: Edit->Preferences->Extra Plugin Path (add /path/to/mylib). @LarsGit223 If you look at the plugin there looks like there is unfinished implementation of incrementing letters too. That would probably come in handy from time to time, so maybe keeping it flexibly named with "Increment" vs enumerate would be easier over the long haul. -- 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#issuecomment-534723786
Re: [Github-comments] [geany/geany] Enhancement - Vertical Editing Sequential Numbering (#2317)
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-insert-a-column-of-increasing-numbers-in-sublime-text-2/44384986 https://packagecontrol.io/packages/Increment%20Selection -- 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#issuecomment-534547978
Re: [Github-comments] [geany/geany] Enhancement - Vertical Editing Sequential Numbering (#2317)
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-Selection -- 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#issuecomment-534539941
[Github-comments] [geany/geany] Enhancement - Vertical Editing Sequential Numbering (#2317)
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 addition to Geany. It probably would be best/proper to add it to Scintilla but it could probably be added to Geany by cycling over the vertical selections or prior to paste. ![geany_feat](https://user-images.githubusercontent.com/10157303/65474182-e2776d80-de48-11e9-90f3-e35b33072b51.gif) -- 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
Re: [Github-comments] [geany/geany] [Feature Request] -- Graphics for Cut/Copy/Paste (#1739)
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 have an overlay grid of the editor to pin point where they should appear - sound effects wouldn't hurt either that way you don't need to move your eyes to know that what you wanted to happen has happened. At the moment, this editor seems fine the way it is, no one thinks its odd or weird to have such a quiet editor. Yet if you implemented this feature in the appropriate way, my guess is that people would not be capable of letting go of them and would be locked in/hooked on them. I think its useful not for purposes of locking users into Geany, yet for the reduction in stress and I think that is a big reason why it would lock people in. When you eliminate sources of hidden stress you are alleviated into a higher state of well being. -- 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/1739#issuecomment-357436467
[Github-comments] [geany/geany] [Feature Request] -- Graphics for Cut/Copy/Paste (#1739)
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, but your brain could construct the idea that you hit the wrong key that causes an invisible quasi-imperceptible amount of stress. That stress could be removed making the editor much more enjoyable to work with by adding a graphic. For example, a cutesy little graphic that appears to the right hand side of the editor border, a gradient trail of smaller versions of it self (. .o 0 O 0 o .) while swerving back and forth before disappearing off the edge of the editor (with a label - for example Copy). Something like hello-kitty meets ICQ would probably be pleasant, yet if you could make it a skin style that could be changed that would probably be ideal. This behavior should probably be capable of being turned off in the settings, yet would probably make a decent 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/issues/1739
Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)
@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 for handling selects at some point). -- 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-353639455
Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)
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 this email directly or view it on GitHub: https://github.com/geany/geany/issues/1549#issuecomment-345836542
Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)
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 using Scintilla defaults to a different functionality than what the other Scintilla editors have as standard with regards to multi-caret on Ctrl-click. I vote to make the new Ctrl-click functionality standard in accordance with the other Scintilla editors. Btw: you are in the same situation, I was in, with regards to fixing it yet not being able to add it to the user interface. Hopefully elextr knows how to route this to the appropriate parties for implementation. -- 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-345402564
Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)
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". Add this to your ~/.textadept/init.lua file: ui.set_theme(not CURSES and 'your-new-theme') Then modify the colors and type Ctrl-E for a command window in textadept and enter: reset() Keep modifying it to your preference (hint you can choose more than the colors listed (e.g. colors.red works), once you get the color scheme you like it's not that different from Geany, and it seems a lot faster. -- 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-345283901
Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)
@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 Lua state. * Generates 'buffer_before_switch' and 'buffer_new' events. * @param doc Almost always zero, except for the first Scintilla view created, * in which its doc pointer would be given here since it already has a * pre-created buffer. * @see lL_adddoc */ static void new_buffer(sptr_t doc) { if (!doc) { doc = SS(focused_view, SCI_CREATEDOCUMENT, 0, 0); // create the new document lL_event(lua, "buffer_before_switch", -1); lL_adddoc(lua, doc); lL_gotodoc(lua, focused_view, -1, FALSE); } else lL_adddoc(lua, doc), SS(focused_view, SCI_ADDREFDOCUMENT, 0, doc); #if GTK ... ``` https://foicica.com/textadept/TECHNOLOGY.html -- 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-345281595
Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)
@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 to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1141#issuecomment-345245883
Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)
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 the user interface complexity. What I did was simply hook the functionality to an existing loaded user interface component for testing - to make sure it was turned on. That way you don't have to worry about the user interface - you can just turn on the functionality and test that it works then upload what you have and ask a developer to integrate it into the user interface. As far as the Scintilla functions, I think these may be the ones you want to play around with: SCI_SWAPMAINANCHORCARET moves the caret to the opposite end of the main selection. SCI_ROTATESELECTION makes the next selection be the main selection. SCI_MULTIPLESELECTADDNEXT adds the next occurrence of the main selection within the target to the set of selections as main. If the current selection is empty then select word around caret. The current searchFlags are used so the application may choose case sensitivity and word search options. SCI_MULTIPLESELECTADDEACH is similar to SCI_MULTIPLESELECTADDNEXT but adds multiple occurrences instead of just one. Look in the Geany code where SCI_SETMULTIPLESELECTION is set, to see how to turn on these on then look at the keybindings code to see if you can remap them to buttons (easier for testing). Try to focus on one thing that you can see working or not to get it going then ramp it up the full functionality. -- 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-343918500
Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)
@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 thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1141#issuecomment-343908907
Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)
> 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 email directly or view it on GitHub: https://github.com/geany/geany/issues/1141#issuecomment-343905001
Re: [Github-comments] [geany/geany] Geany should check Scintilla status after operations (#1569)
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 looks like for my typical use: ``` $> strace -e trace=memory geany &> /tmp/mystrace.txt $> ./memtrace.rb -f /tmp/mystrace.txt max: heap7675904 (7.32M) mmap 17944616 (17.1M) vm 25620520 (24.4M) ``` So, for my typical use case having around 50MB should cover what is actually required - but the healthy buffer itself is beyond what is required to run so adding a few more MB (~5MB) should normally be okay. If the user strays outside of the healthy reserve then Geany/Scintilla should try to allocate what will be required + however much more to get back the healthy buffer. If Geany cannot allocate the memory for both then a warning should be thrown telling the user of the situation and allowing the user to take corrective measures (e.g. saving) and preclude data loss. So to be clear the healthy buffer is not necessarily stopping the user from any operations, it is simply a safety margin to allow for saves and clean shutdown of what Geany is already dealing with *before* Geany actually runs out of memory. -- 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/1569#issuecomment-320150354
Re: [Github-comments] [geany/geany] Geany should check Scintilla status after operations (#1569)
@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://github.com/geany/geany/issues/1569#issuecomment-320144909
Re: [Github-comments] [geany/geany] Geany should check Scintilla status after operations (#1569)
> 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 allocation failure. I'm talking about keeping a certain memory reserve at all times, that is set to something sensible that could be modified in a setting by the user and if the healthy buffer theshold amount cannot be allocated then the warning happens. This would allow the user to "tidy up" and save the program before any crash/freeze event occurs. Thereafter the save the buffers are wiped out and there is enough memory again. Not simply allocating a healthy buffer initially and then writing to it until it runs all the way out and hope that it can allocate more - it could be dangerous letting things go that far. -- 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/1569#issuecomment-320142642
Re: [Github-comments] [geany/geany] Geany should check Scintilla status after operations (#1569)
> 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 do, yet what it can do is keep a healthy buffer amount of memory allocated and if it fails to grab more then warn the user. It may take up some more memory as a result but if you make it an option it may be useful for people dealing with large files. -- 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/1569#issuecomment-320137651
Re: [Github-comments] [geany/geany] Geany should check Scintilla status after operations (#1569)
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 > is stored in a gap buffer (the substance field in the code shown), like > EMACS. Line start positions (added by the InsertLine method) are also stored > in a gap buffer but with a 'step' which enables modifications in close > proximity to affect few elements." > What is inefficient about this is if you have a giant line, Scintilla will > slow right down, because it goes through every character in the block to > determine where new lines are. So if you copy & paste from an external > source, you could potentially see a slow down. As a code editor though, this > should rarely be a problem. Like GtkTextView, Scintilla's Editor interface is > lacking key binding overlays to allow users to use Vi or Emacs keybindings > out of the box. https://ecc-comp.blogspot.com/2015/05/a-brief-glance-at-how-5-text-editors.html >From what I've read, the main problem has to do with dynamic line sizes, >Geany/Scintilla doesn't know where the next line will end so it has to scan >every character to keep its loaded undo buffer. I'm not sure why you guys >reported that it consumes twice the size of the file size. It sounds like it >shouldn't, but it I don't have time to test it at the moment. Not sure if it would be possible to improve geany to using a static line size setting that eliminates the scanning and possibly the dynamic undo buffer to ensure memory is not allocated if its not necessary. It would also be nice to warn the user if Geany is in danger of running out of memory. Then a user could commit changes as necessary and reset the undo buffer while not continuing to consume more memory. -- 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/1569#issuecomment-320127293
Re: [Github-comments] [geany/geany] Geany Locking up when typing or copying of text (#1540)
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/geany/issues/1540#issuecomment-317491704
Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)
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 develop the remainder of the code then uncomment the array assignments when you are ready to start executing. -- 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-317087596
Re: [Github-comments] [geany/geany] Geany Locking up when typing or copying of text (#1540)
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 are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1540#issuecomment-317086480
Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)
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
Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)
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#issuecomment-316759763
Re: [Github-comments] [geany/geany] Geany Locking up when typing or copying of text (#1540)
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 directly or view it on GitHub: https://github.com/geany/geany/issues/1540#issuecomment-316693481
[Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)
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: ``` #!/bin/bash declare -a MYARRAY MYARRAY+=("intentional aor unintentional features") MYARRAY+=("(intentional aor unintentional) features") MYARRAY+=("((intentional) aor (unintentional)) errors") MYARRAY+=("((intentionally) aor (unintentionally)) broken") echo "this should not look this way" ``` Then open it in Geany and have a look at the coloring. It breaks on the nested parenthesis in the third array element. -- 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
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
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 the cursor where you want and paste it like normal. I realize this enhancement request probably won't get implemented anytime soon, but it would save people time - at the moment to repeat a single row to multi-line paste selection semi-quickly I count the number of lines to paste to, then open up a new scratch tab, type in what I want pasted, hit Ctrl-d (copy line) for the amount of lines to paste to, copy the text, make the multi-line select, then paste, then close my opened scratch tab. Instead of - not counting the number of lines - simply selecting the text, copying it then doing a multi-line select, pasting it to all rows. -- 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-308980339
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
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 rows then it should be presumed that you expect the text to be copied into each row - otherwise, you would not have made the multi-row selection for the paste. -- 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-308876847
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
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-line select and then paste the copied text into each multi-line row selected. Instead this is what is happening as of Geany 1.31: ![geany](https://user-images.githubusercontent.com/10157303/27189120-5cbf5f18-51be-11e7-8543-e2924a07f1b3.gif) It would also be useful if you could copy multiple lines of text then select the same number or more lines of text and paste the multi-line text copied into the selected area. This doesn't work either. -- 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-308775839
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
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
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
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
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
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
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
![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/issues/850#issuecomment-245316005
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
@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
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
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
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
@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-241820555
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
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 selected then the issue this Enhancement Request refers to is fulfilled. Before it would only paste one line. -- 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-241479799
Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)
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
Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)
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