Should we remove the string type at the end of DisplayProperty?

[vensa-albertgao]

export type DisplayProperty = Globals | DisplayOutside | DisplayInside | DisplayInternal | DisplayLegacy | "contents" | "list-item" | "none" | string;

The extra string in the end simply disable the feature of type checking, now the display could be anything like foo or bar.

Furthermore, in Typescript, you can't get intellisense, because it could be anything.
Should we remove it? Or I miss something?

Thanks :)

[pelotom]

See #8, #57, #60.

[vensa-albertgao]

Learned something! Thanks!