When using *Tools->Config Files* to open a user file that doesn't exist. This fixes the first line of `geany.css` being wrongly uncommented (and the rest unchanged) due to toggle comment being used and CSS not having a line comment.
Split out from #3396 as it's independent of it. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/3450
-- Commit Summary --
* Don't comment global content unless it is Conf
-- File Changes --
M src/ui_utils.c (3)
-- Patch Links --
https://github.com/geany/geany/pull/3450.patch https://github.com/geany/geany/pull/3450.diff
There is something seriously broken with `Tools->Config Files` that needs to be fixed first, see #3451. If thats working this might not be needed.
Ok, #3451 solved, my bad.
So, if I run current Geany git the first line in `geany.css` is not uncommented.
This does not appear to be needed, maybe check your repo is up to date so #3413 and #3428 are included?
@ntrel As @elextr said, I think you are not using an up-to-date Geany, the code looks a bit different now. In #3413 I wasn't sure what to do with the non-conf files and left the old code here
https://github.com/geany/geany/blob/5298f51f41bb29e41970456652d2314ce609bcce...
but I agree with the idea of your patch and I think the whole `else` branch should be dropped.
I've just created https://github.com/geany/geany/pull/3576 which does the same thing as this patch, it's just done against the latest code in master where the affected code got modified.
@ntrel Does it look OK to you?
Closed #3450.
I've just merged https://github.com/geany/geany/pull/3576 so I'm closing this one.
github-comments@lists.geany.org