[Github-comments] Re: [geany/geany] wrong functions showed in symbols (Issue #3661)

2023-10-26 Thread elextr via Github-comments
The `sh` parser was swapped to the ctags parser from the Geany one.

`function` and other keywords are actually only keywords at the start of a 
command (for bash and zsh at least).  But the ctags `sh` parser is pretty 
simple and does not track start of command.  Detecting command start might not 
be simple since they can be embedded.

That said there are some changes around `function` in the ctags parser that are 
newer than the one in Geany, but not sure if they will help.

 @techee maybe add `sh` to the list of parsers that need to be updated, 
especially to get the excellent work of @b4n ;-)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3661#issuecomment-1782082151
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] [geany/geany] wrong functions showed in symbols (Issue #3661)

2023-10-26 Thread zenobit via Github-comments
`if [[ $(type -t "editions_${OS}") == function ]]; then`

showing this line as function in symbols, even it isn't really function.
Wasn't like that in older version

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3661
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Convert JSON to builtin filetype and use Scintilla JSON lexer for it (PR #3647)

2023-10-26 Thread Jiří Techet via Github-comments
@techee commented on this pull request.



> @@ -0,0 +1,59 @@
+# For complete documentation of this file, please see Geany's main 
documentation
+[styling]
+# Edit these in the colorscheme .conf file instead
+default=default
+number=number_1
+string=string_1
+stringeol=string_eol
+propertyname=attribute
+escapesequence=string_2
+linecomment=comment_line
+blockcomment=comment
+operator=operator
+uri=string_1
+compactiri=string_1

> It might be better to map one consistent style to property names and compact 
> IRIs

Done now in the latest commit.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3647#discussion_r1373838355
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Convert JSON to builtin filetype and use Scintilla JSON lexer for it (PR #3647)

2023-10-26 Thread Jiří Techet via Github-comments
@techee pushed 1 commit.

f53b82cd6747495062d3e44ea710c9cd03b3ff0d  Use identical style for compactiri 
and propertyname

-- 
View it on GitHub:
https://github.com/geany/geany/pull/3647/files/2298cac67b23fdf7d087a57b099cd0136ea8b3c0..f53b82cd6747495062d3e44ea710c9cd03b3ff0d
You are receiving this because you are subscribed to this thread.

Message ID: 


[Github-comments] Re: [geany/geany] Pasting via middle-click clears X selection (Issue #3658)

2023-10-26 Thread Markus-N via Github-comments
I'm using X11.  
AFAIK, XFCE does not yet support Wayland.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3658#issuecomment-1781401841
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] meson: Use only a single type for a given configuration data item (PR #3642)

2023-10-26 Thread Christoph Reiter via Github-comments
I can confirm that things build with this at least.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3642#issuecomment-1781184462
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] meson: correctly set WIN32 in config.h (PR #3660)

2023-10-26 Thread Christoph Reiter via Github-comments
An alternative would be to replace all checks for WIN32 in the code with 
_WIN32, but there are lots of places..

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3660#issuecomment-1781180150
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] [geany/geany] meson: correctly set WIN32 in config.h (PR #3660)

2023-10-26 Thread Christoph Reiter via Github-comments
To match the autotools build.
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany/pull/3660

-- Commit Summary --

  * meson: correctly set WIN32 in config.h

-- File Changes --

M meson.build (4)

-- Patch Links --

https://github.com/geany/geany/pull/3660.patch
https://github.com/geany/geany/pull/3660.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3660
You are receiving this because you are subscribed to this thread.

Message ID: geany/geany/pull/3...@github.com


[Github-comments] Re: [geany/geany] New feature requests? (Issue #3659)

2023-10-26 Thread Jing Hua Zhao via Github-comments
Thanks! Perhaps it will take time for me to learn and be familiar with these.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3659#issuecomment-1781117679
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] CSS nesting syntax coloring (Issue #3523)

2023-10-26 Thread jansalleine via Github-comments
I opened an issue about this at lexilla: 
https://github.com/ScintillaOrg/lexilla/issues/210

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3523#issuecomment-1781051943
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Pasting via middle-click clears X selection (Issue #3658)

2023-10-26 Thread elextr via Github-comments
I thought I saw a contribution by @eht16 so he may be able to add more info.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3658#issuecomment-1780947012
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Pasting via middle-click clears X selection (Issue #3658)

2023-10-26 Thread elextr via Github-comments
Yes, middle click pasting multiple in Geany used to work with 1.38, but I can 
test only on X11, not Wayland/Xwayland.  The change is deliberate in Scintilla 
(wade through the Scintilla bugs/fixes linked from #2629 to get 
https://sourceforge.net/p/scintilla/code/ci/921cb3dea3112a) and related to 
fixing something for Wayland selections.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3658#issuecomment-1780945613
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Pasting via middle-click clears X selection (Issue #3658)

2023-10-26 Thread Colomban Wendling via Github-comments
I don't see the same behavior, although kinda similar: I cannot paste multiple 
times from the editor to itself.  On the first paste it clears the initial 
selection, and that clears the PRIMARY X selection.  I believe that used to 
work, which was a lot more convenient I think.
However, with other apps (or documents) as either source or target I don't have 
the issue and can paste as many times as I want.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3658#issuecomment-1780928844
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Replace utils_make_human_readable_str() with g_format_size() (PR #3657)

2023-10-26 Thread Colomban Wendling via Github-comments
> Edit: are you _sure_ you want to use a functions that was only introduced in 
> Glib 2.30 12 years ago, 2011, thats the same year as C11, surely thats way 
> too new ;-P

Hehe :)  Hold tight though, I'm kind of planning to see what our minimal GTK 
dependency brings as of minimal GLib dependency and start using API as new as 
that if it makes sense! (I have seen a check for `g_strv_equal()` that got me 
wondering)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3657#issuecomment-1780920863
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] New feature requests? (Issue #3659)

2023-10-26 Thread Colomban Wendling via Github-comments
> select and write out a particular chunk of code outside to be used elsewhere.

You could use the [context action](https://geany.org/manual/#context-actions) 
with a custom script, but note that this can be set per-filetype as well, in 
which case it's less convenient for this.  You could use a script like so:
Example script, use script.sh "%s" as the 
context action command


```shell
#!/bin/sh

# just in case, because it ruins the zenity call if set
unset G_MESSAGES_DEBUG

target="$(zenity --title="Choose a file where to write the selection" 
--file-selection --save --confirm-overwrite)" || exit 1
printf '%s' "$*" > "$target"
```


 

You can also abuse [*send selection 
to*](https://geany.org/manual/#sending-text-through-custom-commands) for this.  
It's kind of a hack because that feature is used to *transform* the selection, 
but you can still get away with it either by having your script exit with 
failure (then your selection won't be replaced), or have it send out the same 
data it got in, replacing your selection with the same contents.

Example script, use script.sh as the 
command


```shell
#!/bin/sh

# just in case, because it ruins the zenity call if set
unset G_MESSAGES_DEBUG

target="$(zenity --title="Choose a file where to write the selection" 
--file-selection --save --confirm-overwrite)" || exit 1
tee "$target"

# and possibly:
#exit 1
```


You can set custom [keybindings](https://geany.org/manual/#keybindings) for the 
first 9 configured commands.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3659#issuecomment-1780915819
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] New feature requests? (Issue #3659)

2023-10-26 Thread Jing Hua Zhao via Github-comments
Thanks for refreshing my understandings! The sequences of ctrl-c,ctrl-n,ctrl-v 
as with ctrl-a on a file would help; probably I have been too used to the 
ctrl-^, ctrl-o (to write) and ctrl-r (to read) from pico/nano; copy/paste might 
be to do with the console I am using (mobaXterm) as it won't copy to anywhere 
(Bash,python,pico) where I use a lot.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3659#issuecomment-1780908470
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] New feature requests? (Issue #3659)

2023-10-26 Thread elextr via Github-comments
> select and write out a particular chunk of code outside to be used elsewhere.

This is not directly available in base Geany and AFAIK not in any of the 
plugins collection.  Would be best in a plugin.

The workaround is to copy and paste in a new file, `select ctrl+c ctrl+n 
ctrl+v` and 
[saveactions](https://www.geany.org/manual/current/index.html#instant-save) can 
even make it a  file automatically and you can save-as with your own name.

> insert from another file at the spot.

There is no ability to do anything with a closed file, it has to be opened, 
this ensures that encoding issues have been addressed so that non-UTF-8 content 
is not pasted into the buffer.  When open `ctrl_a` selects the whole file for 
you to copy and paste.

> select a code chunk and paste to a console such as R.

Not sure why you can't copy and paste to R, that sounds like an R problem.  I 
don't have R but can successfully copy and paste into the Python terminal.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3659#issuecomment-1780829471
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] Pasting via middle-click clears X selection (Issue #3658)

2023-10-26 Thread elextr via Github-comments
Are you using Wayland or X11?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3658#issuecomment-1780795608
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] [geany/geany] New feature requests? (Issue #3659)

2023-10-26 Thread Jing Hua Zhao via Github-comments
I installed 1.38 on our CentOS 7 system and have been very impressed with the 
implementation over gedit, for instance smart indentation is cool!

As a user of pico from pine/alpine (or nano) for many years, I get used to 
several features,

1. select and write out a particular chunk of code outside to be used elsewhere.
2. insert from another file at the spot.
3. select a code chunk and paste to a console such as R.

Surely I may have missed many things, as I am new and still not sure about all 
its functions and yet to get 2.0 going as pango, glib and GTK+ are all out of 
dates 

Any comments are welcome.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3659
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] [geany/geany] Pasting via middle-click clears X selection (Issue #3658)

2023-10-26 Thread Markus-N via Github-comments
I'm using geany 2.0-1 on Arch Linux with XFCE.

Since the newest update, I can not use paste via middle-click repeatedly 
anymore. This was a frequent use case for me: select some text and then paste 
it at a few places. It does not matter if the selection was made inside or 
outside geany.
What makes me think  geany clears the X selection:
* I can select and paste into a different window multiple times, but after 
pasting into geany, I can't anymore.
* Also, the clipboard manager plugin for the XFCE panel shows no mouse 
selection anymore afterwards.


-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3658
You are receiving this because you are subscribed to this thread.

Message ID: