> Yeah, will happily add it once it is merged 爛
It just got in :wink:
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3593#issuecomment-1765856443
You are receiving this because you are subscribed to this thread.
Message ID:
Merged #3602 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3602#event-10674479504
You are receiving this because you are subscribed to this thread.
Message ID:
Closed #2026 as completed via d23c9279e57a145dfbe297a356fe9e5824190c47.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/2026#event-10674479924
You are receiving this because you are subscribed to this thread.
Message ID:
See #2398
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/2906#issuecomment-1765889149
You are receiving this because you are subscribed to this thread.
Message ID:
> Maybe
> https://mesonbuild.com/Reference-manual_functions.html#library_name_suffix
Cool, thanks, that appears to be it, I'll test.
> I was thinking of people who download plugins from the internet, most will be
> for linux, will they work? I guess so far there has been no complaint, so
>
Merged #3049 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3049#event-10674470211
You are receiving this because you are subscribed to this thread.
Message ID:
Me too. I'm meging it.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3602#issuecomment-1765845538
You are receiving this because you are subscribed to this thread.
Message ID:
@b4n commented on this pull request.
On doc/images/build_menu_commands_dialog.png:
I'll try and grab my girlfriend's Bullseye (yeah, not updated yet…) which has
whatever version of GNOME shell that came with it, and see if it looks more
"normal". If it does, I'll take the screenshots later
> "improved support of ctags file format" could be mentioned so users know they
> can use these files and there should be no issues with them.
Maybe also worth mentioning that this is the recommended way of generating tag
files.
--
Reply to this email directly or view it on GitHub:
Closed #2798 as completed.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/2798#event-10677129264
You are receiving this because you are subscribed to this thread.
Message ID:
This resolves #3376.
I thought about adding more commands, for example to automatically write the
output into a file, but I think, only the command I added is generally
desirable.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/3607
--
@b4n commented on this pull request.
On doc/images/build_menu_commands_dialog.png:
FWIW I just tried in Bookworm VM as well, and was shocked that @eht16 is not
crazy and it *does* look like this, be it on Wayland or X11… playing a tad with
it, it's due to the Adwaita icons, if I use that on
@b4n commented on this pull request.
On doc/images/build_menu_commands_dialog.png:
@eht16 if you move those out of the way:
* */usr/share/icons/Adwaita/48x48/legacy/edit-clear-all.png*
* */usr/share/icons/Adwaita/48x48/legacy/edit-clear.png*
*
@b4n approved this pull request.
WFM
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3599#pullrequestreview-1682534844
You are receiving this because you are subscribed to this thread.
Message ID:
@b4n approved this pull request.
LGTM (not tested)
Indeed the check is wrong in many aspects -- event 0.0.2 would be < then 1.38.1
Also, it uses `GEANY_VERSION`, which is a compile-time thing, so better update
your update checker! It could benefit from Geany exposing
Closed #865 via 38a138f8c0df31787a3f579c5a606c6a71bb1fd9.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/865#event-10680086514
You are receiving this because you are subscribed to this thread.
Message ID:
Merged #952 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/952#event-10680086535
You are receiving this because you are subscribed to this thread.
Message ID:
Thanks for the reply. I will try with them too. Thank You.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/issues/1282#issuecomment-1766503676
You are receiving this because you are subscribed to this thread.
Message ID:
@b4n commented on this pull request.
> @@ -23,6 +23,7 @@ s/^\(#define VER_FILEVERSION_STR *\)[^ ].*$/\1"'"$VER"'"/
' -i geany_private.rc
sed -e 's/^\(AC_INIT([^,]*, *\[\)[^]]*\(\],\)/\1'"$VER"'\2/' -i configure.ac
+sed -e 's/^\( *version: *\)[^,]*\(,\)/\1'"\'$VER\'"'\2/' -i meson.build
The condition is wrong and for version 2.0 it will constantly report
theres an update available when upgrading from 1.x (where x 0).
I had a visionary moment regarding possible 2.0 problems and had a
look at the updatechecker plugin if it handles major versions - id does, but
wrongly (in
Geany currently loads only plugins with the .so extension on macOS.
This extension is still needed since we use autotools for geany-plugins which
also generate .so.
I tested it and it does nothing on linux while on macOS it changes the
extension correctly to .so.
Unless its a big a problem,
Could at least the part setting the version in meson.build to "2.0" be merged?
geany-plugins now depend on 2.0 and this prevents a successful build when meson
is used for Geany.
(I'd like to test if everything is alright for the macOS build before the
release.)
--
Reply to this email
Yeah let's get it in not to have other people waste time trying to "fix" it.
This won't remove all deprecation warnings (the plugin still uses
`gtk_vbox_new()`) though. I can very easily fix that now we depend on GTK3
anyway though, but that can be for later.
--
Reply to this email directly
I had removed all reference to fileheader3 and the po files, as you had asked.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3549#issuecomment-1766983604
You are receiving this because you are subscribed to this thread.
Message ID:
@b4n commented on this pull request.
On doc/images/build_menu_commands_dialog.png:
@eht16 do you want me to take care of the screenshots, or can you do it with
what I mentioned?
--
Reply to this email directly or view it on GitHub:
I use arch with xfce4. After last update of geany-git (git >= 1b06889fe) I have
blurry text inside editor. Only in editor, fonts in menu/settings are good.
Check out font view from editor settings and inside editor.
@b4n commented on this pull request.
> +* Improved support of ctags file format, this is also the
+ recommended format for generating tags files (PR#3049).
```suggestion
* Improved support for the ctags file format, which is now the
recommended format for tags files
Same as @eht16, it looks good if I believe the docs that a value of `[]` make
it use the default behavior.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3608#issuecomment-1767017197
You are receiving this because you are subscribed to this thread.
@eht16 pushed 1 commit.
1b2d4f04b7a1026148a12eee3e0a39f789f9797c Update screenshots - without super
bloated Adwaita legacy icons
--
View it on GitHub:
https://github.com/geany/geany/pull/3593/files/36930bc6bfff50be25831760db3aec9311b519fd..1b2d4f04b7a1026148a12eee3e0a39f789f9797c
You are
@eht16 commented on this pull request.
On doc/images/build_menu_commands_dialog.png:
Mighty @b4n, that's it.
Screenshots are updated again and look a lot better now.
The Gnome default screenshot tool adds a think transparent border around the
images. I could remove it but actually I think
@eht16 commented on this pull request.
On doc/images/build_menu_commands_dialog.png:
> @eht16 do you want me to take care of the screenshots, or can you do it with
> what I mentioned?
Decide yourself :).
I was too busy with making screenshots to notice your comment.
--
Reply to this email
@b4n commented on this pull request.
On doc/images/pref_dialog_templ.png:
Haha I didn't remember I was there :D
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3593#pullrequestreview-1683202948
You are receiving this because you are subscribed to
@eht16 pushed 1 commit.
c0b429f59fd2dc4436366e146a5016ccedf2f4fa Update NEWS
--
View it on GitHub:
https://github.com/geany/geany/pull/3593/files/1b2d4f04b7a1026148a12eee3e0a39f789f9797c..c0b429f59fd2dc4436366e146a5016ccedf2f4fa
You are receiving this because you are subscribed to this thread.
@eht16 commented on this pull request.
On doc/images/pref_dialog_templ.png:
Last chance to get you out of there :).
To be honest, I just re-used the values from the 1.38 screenshot.
--
Reply to this email directly or view it on GitHub:
@b4n commented on this pull request.
On doc/images/replace_dialog.png:
That one still needs updating, sorry
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3593#pullrequestreview-1683207407
You are receiving this because you are subscribed to this
@b4n commented on this pull request.
On doc/images/build_menu_commands_dialog.png:
> The Gnome default screenshot tool adds a think transparent border around the
> images. I could remove it but actually I think it's nice because it gives the
> images some margin when embedded in the HTML
> it looks good if I believe the docs that a value of [] make it use the
> default behavior.
Yeah, kind of weird because otherwise it expects a string but it works. I
tested it on linux, using some crazy suffix in the "darwin" branch and it gets
ignored and plugins with the "so" extension are
Merged #3608 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3608#event-10682898554
You are receiving this because you are subscribed to this thread.
Message ID:
Closed #3606 as completed via #3608.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3606#event-10682898784
You are receiving this because you are subscribed to this thread.
Message ID:
I don't know Meson nor macOS very well but the changes look good.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3608#issuecomment-1766918324
You are receiving this because you are subscribed to this thread.
Message ID:
@alberic89 pushed 1 commit.
431790c1750f915f21585ea1823cf5d97b0b487c Delete fileheader3
--
View it on GitHub:
https://github.com/geany/geany/pull/3549/files/d8912aca788ef9fe9759bd3e09c33bf0a926aac3..431790c1750f915f21585ea1823cf5d97b0b487c
You are receiving this because you are subscribed to
> Indeed the check is wrong in many aspects -- event 0.0.2 would be < then
> 1.38.1
Here you'll be saved by the copy-paste error of the current code where the last
digit is ignored ;-).
The current code won't behave as bad as I thought originally because for
releases the value on the server
I'm not sure if I understand the problem correctly. "root" for the plugin is
- either the project directory itself, e.g. `/home/techet/projects/geany`
- or added external directories, e.g. `/home/techet/projects/geany-plugins` and
possibly some more
Since `/home/techet/projects/geany` is the
Merged #1284 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/1284#event-10681174674
You are receiving this because you are subscribed to this thread.
Message ID:
> Well, one issue is that people using the plugin won't be notified about 2.0
> anyway, as it'll make it older than basically whatever their version is… but
> we can't do anything about that.
They should get notified because of
```
geany_running.major < geany_current.major
```
The problem was
Yeah, it happens if `next to current` is checked.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3609#issuecomment-1766878125
You are receiving this because you are subscribed to this thread.
Message ID:
@eht16 pushed 1 commit.
36930bc6bfff50be25831760db3aec9311b519fd Update NEWS
--
View it on GitHub:
https://github.com/geany/geany/pull/3593/files/b11817519cb0e9d2232958c54ce6c3d37c1ec599..36930bc6bfff50be25831760db3aec9311b519fd
You are receiving this because you are subscribed to this thread.
@alberic89 pushed 1 commit.
d8912aca788ef9fe9759bd3e09c33bf0a926aac3 Revert "regenerate .po files"
--
View it on GitHub:
https://github.com/geany/geany/pull/3549/files/588a0f007c7e8869348e54d221706a61314db753..d8912aca788ef9fe9759bd3e09c33bf0a926aac3
You are receiving this because you are
> Here you'll be saved by the copy-paste error of the current code where the
> last digit is ignored ;-).
Ah indeed, missed that typo :)
> The current code won't behave as bad as I thought originally because for
> releases the value on the server is the highest number. It would be just in
>
Restoring the session (from my upgraded ~/.config, which has seen all versions
since the dawn of time^W^W^W^W I started using Geany) restores the files, but
not:
* the order
* the current file
It kinda looks like the first element is right, but the reset is sort of loaded
in reverse… Might
Add NEWS for 2.0.
Remarks for the NEWS items:
- I skipped many smaller or non user visible changes or follow-up PRs to not
bloat the list too much
- if anyone miss his favorite PR or so, shout out
You can view, comment on, or merge this pull request online at:
Ah right, it was notifying when *either* `major` or `minor` was higher. So
yeah, only false positives if the local version is newer than the release.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/1284#issuecomment-1766870584
You are receiving
Merged #62 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-themes/pull/62#event-10683074972
You are receiving this because you are subscribed to this thread.
Message ID:
@b4n commented on this pull request.
On doc/images/main_window.png:
Agreed it could be bit improved having more room for the editor, event with
Adwaita. And having it a bit taller is probably no problem. Something like
this (though beware! it's a Git build and you can see it)
LGTM.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/1285#issuecomment-1767409003
You are receiving this because you are subscribed to this thread.
Message ID:
@kugel- commented on this pull request.
> +* Add a confirmation dialog on search & replace for the whole session
+ (PR#3033).
+* Filter entry for symbol tree (PR#3055).
+* Simplify project creation from existing directories with sources
+ (PR#3042).
+* Add option to
@kugel- commented on this pull request.
On doc/images/main_window.png:
I like the proportions of the older screenshot better (msgwin is huge now)
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3593#pullrequestreview-1683324927
You are receiving this
To be clear, the ignore patterns are only applied _within_ the project
directory tree, not to the path to the root, so as @techee said, unless the
mount points are inside the tree they will not be filtered. Any filter applied
to the path to root could only ignore the root and therefore the
LGBI
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3611#issuecomment-1767470099
You are receiving this because you are subscribed to this thread.
Message ID:
@eht16 approved this pull request.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-themes/pull/55#pullrequestreview-1683241926
You are receiving this because you are subscribed to this thread.
Message ID:
Merged #55 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-themes/pull/55#event-10683069370
You are receiving this because you are subscribed to this thread.
Message ID:
@eht16 commented on this pull request.
On doc/images/pref_dialog_various.png:
This is also theme-dependent, isn't it? I don't remember we changed anything in
the dialog?
We could discuss if we do not want to use that theme for screenshots but given
the time left, I would postpone this
@kugel- commented on this pull request.
> +* Add a confirmation dialog on search & replace for the whole session
+ (PR#3033).
+* Filter entry for symbol tree (PR#3055).
+* Simplify project creation from existing directories with sources
+ (PR#3042).
+* Add option to
@eht16 commented on this pull request.
On doc/images/main_window.png:
So far I tried to keep the width and height of the old screenshot. But I don't
mind much.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3593#discussion_r1362754296
You are
OK to merge so I can test the full macOS build + bundling tomorrow?
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3599#issuecomment-1767171751
You are receiving this because you are subscribed to this thread.
Message ID:
@eht16 pushed 1 commit.
a6a36c276cea1d193081987310eec657986c9606 Update NEWS
--
View it on GitHub:
https://github.com/geany/geany/pull/3593/files/f87a878c9c77404f46fbe3af348c04cb8b9543f6..a6a36c276cea1d193081987310eec657986c9606
You are receiving this because you are subscribed to this thread.
@eht16 commented on this pull request.
> +* Add a confirmation dialog on search & replace for the whole session
+ (PR#3033).
+* Filter entry for symbol tree (PR#3055).
+* Simplify project creation from existing directories with sources
+ (PR#3042).
+* Add option to
@kugel- commented on this pull request.
On doc/images/pref_dialog_various.png:
Hm, the older screenshot shows some contrast between the list items. We kind of
regressed here.
--
Reply to this email directly or view it on GitHub:
@eht16 commented on this pull request.
> +* Add a confirmation dialog on search & replace for the whole session
+ (PR#3033).
+* Filter entry for symbol tree (PR#3055).
+* Simplify project creation from existing directories with sources
+ (PR#3042).
+* Add option to
@techee commented on this pull request.
On doc/images/main_window.png:
I think Enrico wanted to show all the tabs of the message window but
unfortunately these are the Adwaita's infamous "bloated and scaled up" tabs
from #3063 so they take a huge amount of space :(.
--
Reply to this email
@eht16 commented on this pull request.
On doc/images/replace_dialog.png:
I'm sorry!
Forgot this one but now added.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3593#discussion_r1362664620
You are receiving this because you are subscribed to this
@eht16 pushed 1 commit.
f87a878c9c77404f46fbe3af348c04cb8b9543f6 Update screenshots
--
View it on GitHub:
https://github.com/geany/geany/pull/3593/files/c0b429f59fd2dc4436366e146a5016ccedf2f4fa..f87a878c9c77404f46fbe3af348c04cb8b9543f6
You are receiving this because you are subscribed to this
@techee commented on this pull request.
On doc/images/main_window.png:
But I think the message window could be made a tiny bit smaller and the whole
window could be taller to compensate for this (if Enrico has a big enough
monitor).
--
Reply to this email directly or view it on GitHub:
@eht16 commented on this pull request.
> +* Add a confirmation dialog on search & replace for the whole session
+ (PR#3033).
+* Filter entry for symbol tree (PR#3055).
+* Simplify project creation from existing directories with sources
+ (PR#3042).
+* Add option to
Should we postpone? Review the comments and merge?
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3396#issuecomment-1767210063
You are receiving this because you are subscribed to this thread.
Message ID:
You need to find out what font is "Monospace Regular", that name is an alias
for another font, not a font itself. Without knowing the actual font we have
no way of replicating the problem as @techee said.
--
Reply to this email directly or view it on GitHub:
@b4n commented on this pull request.
On doc/images/pref_dialog_various.png:
> This is also theme-dependent, isn't it?
It is, unless we're explicitly asking for the theme to do something like that
(there's an option I don't remember, but you're only supposed to use that if
things would be
hum… it's bad, but I'm wondering for how long this existed. I didn't bisect,
but quickly going through this code seems to suggest it's nothing recent (yet,
I might be wrong).
--
Reply to this email directly or view it on GitHub:
@elextr commented on this pull request.
> +.. image:: ./images/edit_change_history.png
+
+The image shows the default visuals:
+
+* inserted characters appear with coloured underlines
+* points where characters were deleted are shown with small triangles
+* the margin shows a block indicating
You both seem to ignore the fact that one can have projects where the plugin
makes sense and others where it doesn't (work).
@techee In the instance I am describing here, the whole project directory is
remote. The problem is not the indexing itself but the directory traversal
which happens no
> You both seem to ignore the fact that one can have projects where the plugin
> makes sense and others where it doesn't (work).
Plugins can be disabled in the plugin manager.
> The problem is not the indexing itself but the directory traversal which
> happens no matter what the index
Can you bisect the exact commit where you started getting the blurry text? (I
don't see anything too bad myself.)
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3610#issuecomment-1767109517
You are receiving this because you are subscribed to this
Awesome! Can't find words for how much I appreciative your effort.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3593#issuecomment-1767128585
You are receiving this because you are subscribed to this thread.
Message ID:
When the `tab_order_beside` option is enabled, tabs are opened on either side
of the active tab, depending on `tab_order_ltr`, rather than at either end.
However, when loading startup files we dont switch to the last opened
document right away, but postpone this to the very end. This affects
Given our discussions on themes and screenshots, maybe in the "Geany Workspace"
section note that the screenshots are dead tortoise ... erm sorry Adwaita theme
;-P
Perhaps after the first screenshot add "Screenshots in the document are the
default GTK Adwaita theme, but Geany will adapt to the
Given that it is done in `notebook_new_tab()` which all file creates use I
would guess its been happening since that option was added by
https://github.com/geany/geany/commit/fdd451f0d4c702d97f398176b41cc2baebe5ee49
(note see the "by" author name :-D
--
Reply to this email directly or view it
@eht16 commented on this pull request.
On doc/images/main_window.png:
Me too but this is the great Adwaita we all love so much...
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3593#discussion_r1362739094
You are receiving this because you are
Merged #3599 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3599#event-10684019579
You are receiving this because you are subscribed to this thread.
Message ID:
Here it looks as identical as my olde eyes can tell no matter how far I zoom.
![Screenshot from 2023-10-18
10-56-11](https://github.com/geany/geany/assets/811085/cb435c89-8293-4e3e-86d6-962179fd8834)
--
Reply to this email directly or view it on GitHub:
89 matches
Mail list logo