Skip to content

Remove use of settings.register, not part of the public API #2088

@FireChickenProductivity

Description

settings.register is not part of the public API and has performance implications, so its use is discouraged. We only use this for the draft window and a snippet.

I recommend removing the snippet.

The draft window DraftManager can just update the styling when the draft window is shown, and we can update settings.talon.example and possibly the setting descriptions to explain that seeing the results of changing those settings requires redrawing the window with a command.

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