Genio icon indicating copy to clipboard operation
Genio copied to clipboard

Settings: Add info about global settings without .editorconfig

Open humdingerb opened this issue 9 months ago • 4 comments

  • As discussed in #363, add the info that the tabs settings in the "Editor" page are only applied if there's no .editconfig in use.

  • Move setting "Editor zoom" to "Visuals" page. Fits there too, and doesn't any more on the "Editor" page after adding the above mentioned info on the no-editconfig-settings.

  • Updated en.catkeys

humdingerb avatar May 05 '24 05:05 humdingerb

I don't know if this is acceptable. I just added the no-editorconfig-used info to the setting "Ignore .editorconfig" as the text will appear after it. Modelled after LSP's "This setting will be updated on restart."

Sure would be nicer if there were a dedicated "just-info-no-actual-config". Possibly with that attention-grabbing horizontal separator line...

humdingerb avatar May 05 '24 05:05 humdingerb

We would have to implement GSettings/ConfigWindow to do something like this. Maybe @Freaxed has any idea about?

jackburton79 avatar May 08 '24 05:05 jackburton79

I found a way to add 'separator' element in the config. The point is that BSeparatorView sacrifices the line space in favour of text and the ending result is not really satisfying.. (see picture) screenshot44

Not sure if there is a way to force the BSeparationView to keep a minimun line space on the borders...

Freaxed avatar May 15 '24 08:05 Freaxed

Can you use ExplicitMinSize() with the string's length + 20 on the thing?

humdingerb avatar May 15 '24 12:05 humdingerb

Good suggestion! screenshot45

Freaxed avatar May 18 '24 07:05 Freaxed

I created a new PR for ConfigSeparator feature. (#391). Let's wait for it to be approved before merging/rebasing this.

Freaxed avatar May 18 '24 07:05 Freaxed

You can merge this after the other one.

jackburton79 avatar May 21 '24 08:05 jackburton79

I rebase after the config separator change was merged and updted the commit message. It's now just the move of the editor zoom setting into "Visuals". Should be ready to merge.

humdingerb avatar May 21 '24 14:05 humdingerb