Theia: [preferences] bogus initial preference group

Created on 2 May 2019  路  5Comments  路  Source: eclipse-theia/theia

Description

There is a bogus preference group node which does not contain any child preferences.

Screen Shot 2019-05-02 at 2 45 49 PM

Steps to reproduce

  1. Open the preferences widget (F1 and select Settings: Open Preferences).
  2. Notice the bogus group node.

Theia as of c2e2308 (no plugins present).

bug preferences

All 5 comments

@akosyakov is this a preference that should not be displayed in the tree?
I see that VSCode has the same preference defined.

If it is defined in user, workspace or workspace folder settings.json files then yes, if it is coming from launch.json then probably not.

Generally splitting looks bogus, obviously there can be a preference name without ..

Generally splitting looks bogus, obviously there can be a preference name without ..

If for whatever reason we can't calculate the group name, for example like in launch, I'll place these preferences under a common group called Other. Would that be acceptable?

Could be alright for now, alternatively for embeddable configurations use a section name as a group and it children as properties. One can check if with PreferenceConfigurations.isSectionName.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

tetchel picture tetchel  路  3Comments

Beetix picture Beetix  路  3Comments

dhananjayharel picture dhananjayharel  路  3Comments

fangnx picture fangnx  路  3Comments

jankeromnes picture jankeromnes  路  3Comments