Skip to content

[Bug] Extension shows dictionary as enabled despite it being disabled in the user settings (and it being actually disabled) #5055

@glektarssza

Description

@glektarssza

For example, if I disable the cryptocurrencies dictionary in my user settings it should not remain enabled, as seen in the following screenshot:

Image

For some reason, and I am assuming this is the case mind, the default configuration provided by @cspell/cspell-bundled-dicts appears to be overriding the settings set in my user settings. I figure this is the case because nowhere do I enable this dictionary but it's still enabled. The only other place it seems to be configured is in that package.

Image

This happens via the mergeSettings call over in DefaultSettingsLoader which loads its settings from that package via the default configuration reference in constants.js.

Please fix so that user-disabled dictionaries are respected.

P.S. Thank you for the ongoing amazing work you do on this CLI tool, library, and extension!

Edit: Screenshots failed to upload the first time. Doh!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions