[patch] [cla] uacss is adding class every time it is included

[iDo]

If multiple file are including dojo/uacss, the html get "spammed" by duplicated classnames.

uacss does not check if the class are already added or not.

[iDo]

See: ​https://github.com/dojo/dojo/pull/57

[bill]

No matter how many files "include'" dojo/uacss, it's only loaded once. This is how AMD's define() works. So, I doubt there's any actual bug here. Can you provide a test case showing a failure?

[dylan]

Assuming there is an issue, it would probably be better to use dojo/dom-class.add (though pulling in that dependency is probably a bit large)... so probably should at least use the same logic we use in dom-class.add if this fix is needed.

[Bryan Forbes]

There is no bug here. The define factory function will only ever run once. The only time I could see this being a problem is if someone is using two versions of Dojo on the same page. However, having the same class multiple times on a node doesn't have an adverse effect: one of the instances is effectively ignored.

With regards to the addition of dojo/dom-class as a dependency, it doesn't really matter. This is a pretty simple module that doesn't really need the code to do replace, toggle, and remove to function.

[iDo]

Replying to bill:

No matter how many files "include'" dojo/uacss, it's only loaded once. This is how AMD's define() works. So, I doubt there's any actual bug here. Can you provide a test case showing a failure?

You're right but dojo give the possibility to load multiple dojo instance in a page. Each instance will add the class. For instance, in our project we have 4 times the class

[bill]

Ah, OK, yes I didn't think of that case. However, since it's unusual to have multiple dojo instances on a page, and since having a class name repeated is harmless, I would also vote to not fix this.