I setup geany on a new machine and spent quite a bit of time trying to figure out why the **strip-trailing spaces** option wouldn't work.
It seems the global? setting in **Preferences/Files/Saving files** is ignored in favour of the same settings in **Project Properties/Files/Saving files**.
These features now operate if either the global setting or project setting is enabled You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/1749
-- Commit Summary --
* Fix strip-trailing-spaces global setting * Fix remaining Files/saving-files global prefs * Improve documentation for file-save preferences
-- File Changes --
M doc/geany.txt (1) M src/document.c (10)
-- Patch Links --
https://github.com/geany/geany/pull/1749.patch https://github.com/geany/geany/pull/1749.diff
It seems the global? setting in Preferences/Files/Saving files is ignored in favour of the same settings in Project Properties/Files/Saving files.
That is the correct and intended operation, project settings are supposed to __override__ global ones. It would be extremely confusing if one setting behaved differently to the others so I do not think this should be applied.
There is/was an issue (don't think it got to a PR) about making that fact more obvious in the UI, but I can't find it now.
How does one use the global settings so?
If they cannot be used maybe they should be removed?
The global settings apply if no project is open.
Okay, now I understand why it is the way it is.
The real issue is that new users are unlikely to know that some of the options are duplicated in the project-properties menu.
Or perhaps the duplicated options should be disabled/hidden when a project is open.
There doesn't seem to be a clean & easy solution.
I think we should have an issue for this, will I open one?
I think we should have an issue for this, will I open one?
It is a known problem that which settings can be overridden by projects is not visible. As I said, I remember a discussion about making settings that can be overridden visible in the UI but I can't find it.
But like you said, we did not find a clean and easy solution IIRC.
If you can't find an issue for it (your search may be better than mine :) then certainly make one, it won't hurt if someone finds a duplicate later.
Perhaps the discussion was just on some unrelated issue/PR, we have the bad habit of doing that :(
There is already an issue here: https://github.com/geany/geany/issues/1363
Closed #1749.
github-comments@lists.geany.org