Re: [Github-comments] [geany/geany] Added support for scoped function calltips (#1177)

2016-08-21 Thread krogank9
Looking at autocomplete_scope I don't see any way this could share its code as this patch stands now. But I find the way that tm_workspace_find returns tags to find_calltips to be satisfactory. For example all of those MeshBuilder and VertexData namespaces are within the namespace BABYLON. I

Re: [Github-comments] [geany/geany] Always load the default builtin color scheme as fallback (#1186)

2016-08-21 Thread elextr
@codebrainz yeah, thats a thought. -- 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/1186#issuecomment-241298563

Re: [Github-comments] [geany/geany] support vte 0.38 in gtk3 build (#336)

2016-08-21 Thread elextr
@Akronix yeah, maybe it was an overreaction on both sides, but as #1181 shows its not a trivial change and effort we could productively use elsewhere. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] Always load the default builtin color scheme as fallback (#1186)

2016-08-21 Thread Matthew Brush
One thing that's better about using the theme's default style as a fallback than the default theme is that the colours will match better. For example if you have a dark theme and the default style is light on dark, then a missing style using that will still be readable, but if it takes the

Re: [Github-comments] [geany/geany] Enhancement Request - Multiselect Multipaste Option (#850)

2016-08-21 Thread elextr
It doesn't work on 1.29 and the word MULTIPASTE does not exist in the Geany sources, so I suspect you have confused it with something else. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)

2016-08-21 Thread elextr
@AdamDanischewski its definitely only possible to have one selection still, you havn't confused it with rectangular selection have you? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] Always load the default builtin color scheme as fallback (#1186)

2016-08-21 Thread elextr
LGBVLT (very limited test) But we should probably advertise this somewhere since it benefits people making themes. I know in the documentation ;-D -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] support vte 0.38 in gtk3 build (#336)

2016-08-21 Thread Abel Serrano Juste
Hi! I just want to say two things: 1) I pushed the issue up because I was using Geany with `enable-gtk3` activated and I was missing the VTE (*I do use it a lot*). I'm on Ubuntu 16.04 and as @techee said in #1181 : > Ubuntu 16.04 has only vte 2.91 for GTK3. 2) I didn't like how some of the

Re: [Github-comments] [geany/geany] Support for VTE 2.91 (#1181)

2016-08-21 Thread Matthew Brush
I'd definitively not include any custom background image implementation. For VTE versions that support it natively, and we already have UI prefs for, it seems OK to leave it, but for newer VTEs it seems better to just guard it out. -- You are receiving this because you are subscribed to this

Re: [Github-comments] [geany/geany] Support for VTE 2.91 (#1181)

2016-08-21 Thread Jiří Techet
I'd personally drop the background image feature completely - it's just silly. It's similarly stupid like if we wanted to add a background image to the Scintilla editor. People for some reason seem to configure their terminals in a very non-ergonomic ways by e.g. adding transparency to the

[Github-comments] [geany/geany] Always load the default builtin color scheme as fallback (#1186)

2016-08-21 Thread Colomban Wendling
Always load the builtin color scheme before the user's one in case the latter lacks some named colors, so it uses the default. This helps us adding new named styles without requiring each and every theme to be updated. You can view, comment on, or merge this pull request online at:

Re: [Github-comments] [geany/geany] Support for VTE 2.91 (#1181)

2016-08-21 Thread Matthew Brush
@techee that's a good 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/pull/1181#issuecomment-241280996

Re: [Github-comments] [geany/geany] Support for VTE 2.91 (#1181)

2016-08-21 Thread Jiří Techet
Looks like a good idea to revive this pull request (especially when https://github.com/geany/geany/pull/1182 gets merged which looks like a good idea to me too) as e.g. Ubuntu 16.04 has only vte 2.91 for GTK3. -- You are receiving this because you are subscribed to this thread. Reply to this

Re: [Github-comments] [geany/geany] Multiple carets, selections, insertion points and paste (#1141)

2016-08-21 Thread AdamDanischewski
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)

2016-08-21 Thread AdamDanischewski
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

Re: [Github-comments] [geany/geany] Allow to configure the error indicator color (#1185)

2016-08-21 Thread Colomban Wendling
I might be interesting to allow a theme to override the default, not sure. -- 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/1185#issuecomment-241255937

[Github-comments] [geany/geany] Allow to configure the error indicator color (#1185)

2016-08-21 Thread Colomban Wendling
Related to #1116. --- trivial enough I guess You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/1185 -- Commit Summary -- * Allow to configure the error indicator color -- File Changes -- M data/filedefs/filetypes.common (3) M

Re: [Github-comments] [geany/geany] Spell checking almost invisible on the last line (#1116)

2016-08-21 Thread Colomban Wendling
Closed #1116. -- 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/1116#event-762271834

Re: [Github-comments] [geany/geany] Spell checking almost invisible on the last line (#1116)

2016-08-21 Thread Colomban Wendling
Closing this, because IMO it's mostly the Ubuntu's theme fault, plus now #1154 is merged so there should be a decent workaround. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] Add support for keeping the cursor a number of lines from the edges (#1154)

2016-08-21 Thread Colomban Wendling
Merged #1154. -- 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/1154#event-762271596

Re: [Github-comments] [geany/geany] Add support for keeping the cursor a number of lines from the edges (#1154)

2016-08-21 Thread Colomban Wendling
> What happens when the setting is set to, say, 30 and the editor pane can only > show, say, 40 lines? A too high value is not really a problem, it will simply always have the current line exactly in the middle of the view. If you really liked that behavior, you can set that to like 100 and

Re: [Github-comments] [geany/geany] Add support for keeping the cursor a number of lines from the edges (#1154)

2016-08-21 Thread elextr
@b4n, if you have the UI might as well leave it, as you say it makes it slightly more discoverable. -- 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/1154#issuecomment-241251061

Re: [Github-comments] [geany/geany] Add support for keeping the cursor a number of lines from the edges (#1154)

2016-08-21 Thread elextr
> What happens when the setting is set to, say, 30 and the editor pane can only > show, say, 40 lines? Should show nothing on the screen except "IDIOT" in big letters :) -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] Add support for keeping the cursor a number of lines from the edges (#1154)

2016-08-21 Thread Frank Drolshagen
My 2ct: Because you, @b4n, have done all that work already, I would love to have the UI, and I think the same as you about people using it. But a possible problem arising from this just came to my mind: What happens when the setting is set to, say, 30 and the editor pane can only show, say, 40

Re: [Github-comments] [geany/geany] Add support for keeping the cursor a number of lines from the edges (#1154)

2016-08-21 Thread Matthew Brush
> Should I drop the UI? Your call. IMO it should be a various preference, but I don't feel strongly at all. -- 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/1154#issuecomment-241249523

Re: [Github-comments] [geany/geany] Add support for keeping the cursor a number of lines from the edges (#1154)

2016-08-21 Thread Colomban Wendling
@b4n pushed 1 commit. 2d2af79 Improve documentation wording -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/geany/geany/pull/1154/files/4ad652aa7d3179dc6da1dcd077d11b21cc5ca0cb..2d2af798cb546bc06c2014b12c4d334a62f71ad9

Re: [Github-comments] [geany/geany] Add support for keeping the cursor a number of lines from the edges (#1154)

2016-08-21 Thread Colomban Wendling
Should I drop the UI? I added it because I felt like this could be something useful to many if they knew it existed, but apparently not even @fdg-org care about the proper UI, so… -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on

[Github-comments] [geany/geany-plugins] File Library for markdown file (#479)

2016-08-21 Thread metasnw
we got lots of markdown files and are looking for a plugin that allows the user to search the file names and content for words and add the link to the file to the current document from the result list. here is are list of source files: