In current Markers extensions class, we're comparing JS objects with '==='. And this result in errors removing markers with accuracy.

[KhozmoS]

current class

class Marker {
    constructor(square, type) {
        this.square = square
        this.type = type
    }

    matches(square = undefined, type = undefined) {
        if (!type && !square) {
            return true
        } else if (!type) {
            if (square === this.square) {
                return true
            }
        } else if (!square) {
            if (this.type === type) {
                return true
            }
        } else if (this.type === type && square === this.square) {
            return true
        }
        return false
    }
}

suggested change:

class Marker {
    constructor(square, type) {
        this.square = square
        this.type = type
    }

    matches(square = undefined, type = undefined) {
        if (!type && !square) {
            return true
        } else if (!type) {
            if (square === this.square) {
                return true
            }
        } else if (!square) {
            if (JSON.stringify(this.type) === JSON.stringify(type)) {
                return true
            }
        } else if (JSON.stringify(this.type) === JSON.stringify(type) && square === this.square) {
            return true
        }
        return false
    }
}

[shaack]

I don't think so. In that case, the matching of the markers would match all markers, looking the same. But it makes more sense for me to match markers of a specific type, independent from their values.