[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
Merged #3593 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#event-10713731869 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 commented on this pull request. On doc/images/edit_change_history.png: Thanks and sorry. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1365723022 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n approved this pull request. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#pullrequestreview-1686329485 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n commented on this pull request. On doc/images/edit_change_history.png: @eht16 I took the liberty to push to this PR with the fix above. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1364648967 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n pushed 1 commit. 37647e32ca6861f7d06000d8acf8f8f8e079a405 Distribute new documentation image -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/d2186cccddf8a885eae0ccb7c894564f1ac669a0..37647e32ca6861f7d06000d8acf8f8f8e079a405 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n requested changes on this pull request. New image is missing from the build system. Otherwise I didn't check all PR and issue references, but looks good! :+1: On doc/images/edit_change_history.png: This is not installed! ```diff diff --git a/doc/Makefile.am b/doc/Makefile.am index ea0e0f697..45dbeecef 100644 --- a/doc/Makefile.am +++ b/doc/Makefile.am @@ -4,6 +4,7 @@ if INSTALL_HTML_DOCS htmldocimagesdir = $(docdir)/html/images dist_htmldocimages_DATA = \ images/build_menu_commands_dialog.png \ + images/edit_change_history.png \ images/find_dialog.png \ images/find_in_files_dialog.png \ images/main_window.png \ diff --git a/doc/meson.build b/doc/meson.build index a9a2c0257..2fa8cab6d 100644 --- a/doc/meson.build +++ b/doc/meson.build @@ -19,6 +19,7 @@ tarball = run_command('test', '-f', 'geany.html', check: false) if tarball.returncode() == 0 or rst2html.found() install_data( 'images/build_menu_commands_dialog.png', + 'images/edit_change_history.png', 'images/find_dialog.png', 'images/find_in_files_dialog.png', 'images/main_window.png', ``` -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#pullrequestreview-1686319476 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
LGTM -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-176945 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 commented on this pull request. > +* Update Python filetype config (PR#3039). +* Add AutoIt filetype (Skif-off, PR#1752). +* Add GDScript filetype (David Yang, PR#3012). +* Remove Ferite filetype (PR#3075). +* Enable local variables and improve autocompletion for C/C++ and + some other languages (PR#3185, PR#3275). + +Windows +* Remove native file and color dialogs on Windows, GTK dialogs will + be used instead (#3219). +* Use "Prof-Gnome" GTK theme by default (same theme as used on macOS) + for a better experience, the "Adwaita" theme can still be activated + (PR#3129). + +Internationalization +* Updated translations: Done and rebased on master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1364579284 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 pushed 6 commits. 641a09159a047d5df5adaf4b7675aa242e72a101 Add NEWS for Geany 2.0 dbf3c8e0141da0e14a619787c2d14a2aa47130c7 Add documentation for new Scintilla feature "Change history" 35a208aa24e78887109fa3d1993548492abc516d Update documentation screenshots c1be9eca9f12c38d12762c6472ad0e50c6db690f Add #3611 to NEWS 22cdf9cd61fa70b8d0c21d87024bd8cf04ada5da Document Change History is disabled by default d2186cccddf8a885eae0ccb7c894564f1ac669a0 Remove duplicate Internationalization item -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/85febd1e301506e8a22cedcc4eb177dd919719c5..d2186cccddf8a885eae0ccb7c894564f1ac669a0 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 commented on this pull request. > +* Update Python filetype config (PR#3039). +* Add AutoIt filetype (Skif-off, PR#1752). +* Add GDScript filetype (David Yang, PR#3012). +* Remove Ferite filetype (PR#3075). +* Enable local variables and improve autocompletion for C/C++ and + some other languages (PR#3185, PR#3275). + +Windows +* Remove native file and color dialogs on Windows, GTK dialogs will + be used instead (#3219). +* Use "Prof-Gnome" GTK theme by default (same theme as used on macOS) + for a better experience, the "Adwaita" theme can still be activated + (PR#3129). + +Internationalization +* Updated translations: This was a stub to be filled later. @frlan pushed the updated list already in master in the meantime. I'd expect that GIT will detect a merge conflict here but it seems it does not. I will remove the two lines, so after a merge it should fit. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1364576852 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 the overall state of the line, prioritizing the more consequential modified states +* the states are modified (orange), saved (green), saved then reverted to modified (green-yellow), and + saved then reverted to original (cyan). + +Show in markers margin +Changes are shown in the markers margin as vertical bars + +Show as underline indicators +Changes are shown as underlines in the text directly + Yes, thats why I marked resolved -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1364573536 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
I just did a quick rescan, raised one question, not critical just ignore if stupid :-) LGTM -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1769368276 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@elextr commented on this pull request. > +* Update Python filetype config (PR#3039). +* Add AutoIt filetype (Skif-off, PR#1752). +* Add GDScript filetype (David Yang, PR#3012). +* Remove Ferite filetype (PR#3075). +* Enable local variables and improve autocompletion for C/C++ and + some other languages (PR#3185, PR#3275). + +Windows +* Remove native file and color dialogs on Windows, GTK dialogs will + be used instead (#3219). +* Use "Prof-Gnome" GTK theme by default (same theme as used on macOS) + for a better experience, the "Adwaita" theme can still be activated + (PR#3129). + +Internationalization +* Updated translations: Why the `:`, were you going to add a list of updated translations? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#pullrequestreview-1686198756 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 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 the overall state of the line, prioritizing the more consequential modified states +* the states are modified (orange), saved (green), saved then reverted to modified (green-yellow), and + saved then reverted to original (cyan). + +Show in markers margin +Changes are shown in the markers margin as vertical bars + +Show as underline indicators +Changes are shown as underlines in the text directly + Ok. Is 85febd1 fine? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1364565237 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 pushed 1 commit. 85febd1e301506e8a22cedcc4eb177dd919719c5 Document Change History is disabled by default -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/e35069844226d60456eb397e6095116b099eb74a..85febd1e301506e8a22cedcc4eb177dd919719c5 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@elextr @kugel- @b4n could you have a look on the yet open discussions and either resolve or comment on them? We are slightly running out of time. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1769341258 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
> @eht16 looks like there has been a rebase mishap, and you grabbed some > unrelated things in there Oops, thanks for notifying. Should be fixed. I started a first round of squashing the incremental changes and then came #3611 :). > You can add #3611 now ;) Done. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1769337606 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 the overall state of the line, prioritizing the more consequential modified states +* the states are modified (orange), saved (green), saved then reverted to modified (green-yellow), and + saved then reverted to original (cyan). + +Show in markers margin +Changes are shown in the markers margin as vertical bars + +Show as underline indicators +Changes are shown as underlines in the text directly + Well at least say it is disabled so users know to enable it. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1364550225 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 commented on this pull request. On doc/images/main_window.png: No, it just that the user reading the documentation would need a resolution big enough to match the screenshot dimensions or the user needs to scroll. But probably no big deal nowadays. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1364548749 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 pushed 4 commits. b73b29b90cc1e1526ea77096d59953610777d2d6 Add NEWS for Geany 2.0 6a61051feccacb0698308d256d29f787bff4705c Add documentation for new Scintilla feature "Change history" f90c81347d9542c85bc6aeddbabc6becc7b4311c Update documentation screenshots e35069844226d60456eb397e6095116b099eb74a Add #3611 to NEWS -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/09e82b5781bc1a5cdf2b26cad4c88c2d791900b7..e35069844226d60456eb397e6095116b099eb74a You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 looks like there has been a rebase mishap, and you grabbed some unrelated things in there :confused: -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1769322566 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
You can add #3611 now ;) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1769320562 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
> You guys like to have me updating screenshots I guess... Nah, it could have totally stayed the way it was. > The screenshot for the prefs dialog with the changed default for the tab > label length is updated for 1000 chars, pixels, apples, Euros . We are aiming for 1000 screenshot updates in this PR :-). -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1769245236 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
You guys like to have me updating screenshots I guess... The screenshot for the prefs dialog with the changed default for the tab label length is updated for 1000 chars, pixels, apples, Euros :smile:. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1769241290 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 pushed 1 commit. 8ac78832fb7e758784ed65dc50eb7ad766ba9eaf Update doc/geany.txt -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/143f8e399684624dbf3b5f0147209ce9f1bd628d..8ac78832fb7e758784ed65dc50eb7ad766ba9eaf You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 pushed 1 commit. 143f8e399684624dbf3b5f0147209ce9f1bd628d Update screenshots -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/50f76291b2e9b3ff55fe655109d532ed221f46a1..143f8e399684624dbf3b5f0147209ce9f1bd628d You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n commented on this pull request. On doc/images/main_window.png: @eht16 yeah… is it a problem in the generated file? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1363823645 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n commented on this pull request. > +Screenshots in the document are the default GTK Adwaita theme, +but Geany will adapt to the desktop GTK theme that is set. ```suggestion Screenshots in this document are taken with the default GTK Adwaita theme, but Geany will adapt to the desktop GTK theme that is set. ``` -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#pullrequestreview-1684983740 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 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 the overall state of the line, prioritizing the more consequential modified states +* the states are modified (orange), saved (green), saved then reverted to modified (green-yellow), and + saved then reverted to original (cyan). + +Show in markers margin +Changes are shown in the markers margin as vertical bars + +Show as underline indicators +Changes are shown as underlines in the text directly + Saying that "is still being evaluated" could lead to issues asking how the status of the evaluation is. IMO this creates expectations that there is someone actively working on this. This might happen but looking back the last two years, it is not that unlikely that it won't happen. Declaring it as "it is there, disabled and being evaluated" might cause more confusion than it tries to solve. I think it is better to leave it as is. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1363467685 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 commented on this pull request. On doc/images/main_window.png: Ok, so it's just getting bigger again. I pushed a variant with similar dimensions as @b4n's demo. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1363460371 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
> Perhaps after the first screenshot add "Screenshots in the document are the > default GTK Adwaita theme, but Geany will adapt to the desktop GTK theme that > is set." Done. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1767937367 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 pushed 2 commits. 86754ddaf839503681e6f3939424883ed3e1d97d Update main window screenshot 50f76291b2e9b3ff55fe655109d532ed221f46a1 Add note about screenshots GTK theme -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/a6a36c276cea1d193081987310eec657986c9606..50f76291b2e9b3ff55fe655109d532ed221f46a1 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
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 desktop GTK theme that is set." The fact that Geany follows the desktop setting actually isn't documented anywhere else AFAICT. Otherwise LGTM -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1767450871 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 the overall state of the line, prioritizing the more consequential modified states +* the states are modified (orange), saved (green), saved then reverted to modified (green-yellow), and + saved then reverted to original (cyan). + +Show in markers margin +Changes are shown in the markers margin as vertical bars + +Show as underline indicators +Changes are shown as underlines in the text directly + Maybe add "This feature is still being evaluated and is currently disabled by default." At least anyone who RTFM will be prompted to look for a setting, not raise an issue "change history doesn't work" -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#pullrequestreview-1683714018 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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) ![main_window](https://github.com/geany/geany/assets/793526/dabf8d21-a4f9-41f0-a4c2-c929c71727c3) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362839167 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 show symbols in symbol tree without category groups + (PR#3172). +* Add option to only show line endings if they differ from file default + (PR#3287). +* Make tab label length and window title length configurable + (Abdul Rafey, #3365). +* Make Go to Symbol commands show signature list (PR#3475). + +Editor +* Update Scintilla to 5.3.7 and Lexilla to 5.2.7 (PR#2867, PR#3551). +* Add "Change history" feature (PR#3551). Done. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362825723 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 hard to comprehend without, and otherwise let the theme decide). -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362756939 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 show symbols in symbol tree without category groups + (PR#3172). +* Add option to only show line endings if they differ from file default + (PR#3287). +* Make tab label length and window title length configurable + (Abdul Rafey, #3365). +* Make Go to Symbol commands show signature list (PR#3475). + +Editor +* Update Scintilla to 5.3.7 and Lexilla to 5.2.7 (PR#2867, PR#3551). +* Add "Change history" feature (PR#3551). Oops, my tired brain read "with explaining why". Sorry. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362755340 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 show symbols in symbol tree without category groups + (PR#3172). +* Add option to only show line endings if they differ from file default + (PR#3287). +* Make tab label length and window title length configurable + (Abdul Rafey, #3365). +* Make Go to Symbol commands show signature list (PR#3475). + +Editor +* Update Scintilla to 5.3.7 and Lexilla to 5.2.7 (PR#2867, PR#3551). +* Add "Change history" feature (PR#3551). That's why I said "(without explaining why)" ;-) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362753305 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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: https://github.com/geany/geany/pull/3593#discussion_r1362748518 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 show symbols in symbol tree without category groups + (PR#3172). +* Add option to only show line endings if they differ from file default + (PR#3287). +* Make tab label length and window title length configurable + (Abdul Rafey, #3365). +* Make Go to Symbol commands show signature list (PR#3475). + +Editor +* Update Scintilla to 5.3.7 and Lexilla to 5.2.7 (PR#2867, PR#3551). +* Add "Change history" feature (PR#3551). `* Add "Change history" feature, disabled by default (PR#3551).` I don't know if it's worth explaining why it is disabled. The explaination will probably be longer than a NEWS item should be. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362746366 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362745514 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 discussion to 2.1+. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362742427 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
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:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 show symbols in symbol tree without category groups + (PR#3172). +* Add option to only show line endings if they differ from file default + (PR#3287). +* Make tab label length and window title length configurable + (Abdul Rafey, #3365). +* Make Go to Symbol commands show signature list (PR#3475). + +Editor +* Update Scintilla to 5.3.7 and Lexilla to 5.2.7 (PR#2867, PR#3551). +* Add "Change history" feature (PR#3551). Might mention that it's disabled by default (without explaining why) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#pullrequestreview-1683326769 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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: https://github.com/geany/geany/pull/3593#pullrequestreview-1683319978 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 document. > What do you think? LGTM -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362658115 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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: https://github.com/geany/geany/pull/3593#discussion_r1362655467 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362653077 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 it's nice because it gives the images some margin when embedded in the HTML document. What do you think? And I'm happy to be confirmed not being crazy, at least in this matter :). -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362648113 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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: https://github.com/geany/geany/pull/3593#discussion_r1362636399 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 (PR#3049). ``` We're not generating them, so I suggest amending the wording -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#pullrequestreview-1683169914 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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* * */usr/share/icons/Adwaita/48x48/legacy/window-close.png* * */usr/share/icons/Adwaita/24x24/legacy/window-close.png* and then run `sudo gtk-update-icon-cache /usr/share/icons/Adwaita/` you'll get a more reasonable look. Looks like what's happening is that they distribute those three icons (well, *edit-clear-all* is probably not necessary for us) in only *some* sizes, and although there is a `-symbolic` version, it's not picked up by GTK if there's a non-symbolic at any size. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1362011703 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 my regular desktop it goes crazy as well. Will see if it's the same on Bullseye, and if it is… I guess it won't be Adwaita icons for the screenshots :) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1361993779 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
> "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: https://github.com/geany/geany/pull/3593#issuecomment-1765934225 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
> 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:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@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 today or tonight. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1361659910 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 commented on this pull request. On doc/images/build_menu_commands_dialog.png: It won't get better. No idea what I'm doing wrong. I managed to scale the display to 60% (after enabling some experimental feature in mutter via gsettings and then changing the scale in some XML file) but it this really scales the display only, the GTK widgets are still huge. Completely unmodified Gnome: ![debian_bookworm_vm_huge](https://github.com/geany/geany/assets/617017/3aa0190c-897d-428c-8a64-e580114711ea) Scaled down to 60% but the widgets are still huge: ![debian_bookworm_vm_scaled_60](https://github.com/geany/geany/assets/617017/4fdb1d11-3135-4b25-9e42-312f5e4ec225) Either someone else can create the screenshots or I will use my local Xfce with Adwaita theme to create them. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1357417559 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
> > Should we maybe better highlight the many ctags changes? > > I don't think it's necessary, the big refactorings weren't user-visible. But > maybe if #3049 gets merged, an "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. Yeah, will happily add it once it is merged :crossed_fingers: > > if anyone miss his favorite PR or so, shout out > > I would maybe just rephrase "Improve user experience when creating new > projects" to something like "Simplify project creation from existing > directories with sources" to make it clearer what it is about. Done, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1758560124 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 pushed 1 commit. b11817519cb0e9d2232958c54ce6c3d37c1ec599 Improve wording for PR3042 item -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/e754f79d16dcac69bcad0da13a0fd2aaf03e6a65..b11817519cb0e9d2232958c54ce6c3d37c1ec599 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 commented on this pull request. On doc/images/build_menu_commands_dialog.png: I'll give it another try tomorrow. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1355787792 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 Thanks for doing the boring job! > I left out names for Jiří and Thomas as you count as core developers for long > I'd say. This doesn't mean your work is less appreciated ❤️ No problem at all :-). > @techee are tere any macOS specific items missing? There were some fixes but nothing really major worth mentioning I think. > Should we maybe better highlight the many ctags changes? I don't think it's necessary, the big refactorings weren't user-visible. But maybe if https://github.com/geany/geany/pull/3049 gets merged, an "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. > if anyone miss his favorite PR or so, shout out I would maybe just rephrase "Improve user experience when creating new projects" to something like "Simplify project creation from existing directories with sources" to make it clearer what it is about. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1758463037 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 commented on this pull request. On doc/images/build_menu_commands_dialog.png: I totally don't mind how we create the screenshots. I just don't want to do it 10 times and do not want to install a dozen VMs :). Trying X11 might get better results. Within Gnome, I didn't find any way to make it look less huge. There are simply no helpful settings :(. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1355159869 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n commented on this pull request. On doc/images/build_menu_commands_dialog.png: Reducing the font size might be the reason for icons to look too big in comparison, whereas the display was just entirely scaled up somehow… I have no idea (my Bookworm looks as above, but it's not 100% stock), but maybe try X11 instead of Wayland (if that was Wayland), just in case it's a weird misdetection of the VM's screen characteristics? I might give this a try later if need be, I have a VM at hand (not under GNOME per se, but that could change and could be made not to matter) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1355148780 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
> Thanks for updating the docs - I made a few tweaks. Thank you, applied them all. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1757863336 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 pushed 1 commit. e754f79d16dcac69bcad0da13a0fd2aaf03e6a65 Update NEWS -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/6601ca02e3e589f1dc76ad89893cbf929420fe50..e754f79d16dcac69bcad0da13a0fd2aaf03e6a65 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 pushed 1 commit. 6601ca02e3e589f1dc76ad89893cbf929420fe50 Apply geany.txt suggestions from code review -- View it on GitHub: https://github.com/geany/geany/pull/3593/files/02238c4c91b84fc5acd1907dcc6e015490f0fc2e..6601ca02e3e589f1dc76ad89893cbf929420fe50 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@eht16 commented on this pull request. On doc/images/build_menu_commands_dialog.png: I booted up a Debian Bookworm VM from the live ISO image without any modifications to the theme. The only thing changed was to reduce the font size, otherwise everything would be even bigger. There was nothing HiDPI related involved. Do you have any suggestion on how to get a sane looking Gnome running? That whole desktop felt very bugged to me but this is another topic :). -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1355134430 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@elextr commented on this pull request. On doc/images/build_menu_commands_dialog.png: Yeah maybe, but there is a lot of padding above and below the icons, anyway its something to be looked at when re-taking the screengrab after the delete icon is fixed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1355101747 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@ntrel commented on this pull request. Thanks for updating the docs - I made a few tweaks. > Interface * The document list in the sidebar has a new tree view. This mode is the new default and existing installations automatically use it (PR#1813). +* Use dark-theme friendly colors for compiler messages (PR#3013). +* Add a confirmation dialog on search & replace for the whole session + (PR#3033). +* Filter entry for symbol tree (PR#3055). +* Improve user experience when creating new projects (PR#3042). +* Add option to show symbols in symbol tree without root groups (PR#3172). ```suggestion * Add option to show symbols in symbol tree without category groups (PR#3172). ``` > @@ -2383,6 +2383,37 @@ Always Always show virtual spaces beyond the end of lines +Change History +`` + +The change history features enables displaying of document changes in the markers margin or in the text. ```suggestion The *change history* feature enables changed text in a document to be shown in the markers margin or by underlining the text. ``` > @@ -2383,6 +2383,37 @@ Always Always show virtual spaces beyond the end of lines +Change History +`` + +The change history features enables displaying of document changes in the markers margin or in the text. + +Newly added, modified and removed lines or words are highlighted to easily track changes to the opened +document. The changes can be shown as vertical bars in the markers margin and/or as underlines in +the text directly. + +.. note:: +This feature may use a moderate amount of memory, especially if there are many or big changes in the document. +Also, the state of the modifications is not persisted, i.e. when closing the document or Geany, all change markers will be lost. ```suggestion Also, modification information is not kept when re-opening a document - all change markers will be lost. ``` -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#pullrequestreview-1671316423 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n commented on this pull request. On doc/images/build_menu_commands_dialog.png: > Also as I said the set build tools dialog seems to have been expanded > vertically (as well as the icons). To me that looks like the dialog is making room for those ginormous icons :) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1355054598 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@elextr commented on this pull request. On doc/images/build_menu_commands_dialog.png: Yeah ok, the delete icon seems to be larger than any other icon, see also the general startup preference dialog, compare the delete icons and the folder icons size on the paths lines, on my system the folder is bigger than the delete, here the delete is bigger than the folder. Maybe Enrico's system is missing an icon size for delete and its picking the next up. Also as I said the set build tools dialog seems to have been expanded vertically (as well as the icons). -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1355051101 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n commented on this pull request. On doc/images/build_menu_commands_dialog.png: Look at the clear icons. My version (yeah, no window borders, but you get the idea): ![image](https://github.com/geany/geany/assets/793526/c4a0bb35-960a-4db1-9a1c-d3c9a057e1ac) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1355003900 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@elextr commented on this pull request. On doc/images/build_menu_commands_dialog.png: Which icons? Looks pretty normal (for dead tortoise) except the set build commands looks like the window has been stretched vertically by the user. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#discussion_r1354995385 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
Thanks for updating all this! But… I really think there's a problem with the inline icons in the screenshots, they shouldn't be that big -- and they aren't on my end with Adwaita (which seems to be what you used, which makes sense to use IMO). -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#issuecomment-1757592993 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] Release 2.0 documentation (PR #3593)
@b4n commented on this pull request. On doc/images/build_menu_commands_dialog.png: wait… how come the icons are so big? HiDPI issue? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3593#pullrequestreview-1671065957 You are receiving this because you are subscribed to this thread. Message ID: