Stop using class for ‘singletons’

[yuchi]

Using

class dom {
  static addClasses() { /*…*/ }
}

instead of

const dom = {
  addClasses() { /*…*/ }
};

does not bring all that value…

[eduardolundgren]

+1

[yuchi]

Even better?

export function addClasses() { /*…*/ }

[yuchi]

Arguments against named exports:

1. no support for decorators;
2. almost impossible to overwrite a method later.

Arguments for named exports:

1. support for export * from 'dom' in a 'dom-extensions' package;
2. almost impossible to overwrite a method later.

[mairatma]

I'll close this since it's very old. For now we're still using class for singletons, mainly because we want to allow overwriting methods, having the library be flexible. We can think about changing this later.

If you think we should still discuss this for now, feel free to reopen this on metal/metal though.

[yuchi]

but this makes the code fragile and this bound.