Skip to content

Dynamic paint tools use potentially overlapping IDs #851

@luxaritas

Description

@luxaritas

See

private setupDynamicPaintTools() {

Here's an example of what could happen: Puzzle 1 has a single paint tool, "stamper", Puzzle 2 has a single paint tool, "randomizer". Adding or removing one from the toolbar will automatically add/remove the other when opening the other puzzle, because both of them have an ID of DYNAMIC-1. This is probably unintended. Instead, we should probably use the script ID.

Metadata

Metadata

Assignees

No one assigned

    Labels

    priority: p3/standardEnhancement with nominal value or bug with nominal impactsize: smtype: bugSomething isn't working

    Type

    No type

    Projects

    Status

    Todo

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions