What is the purpose of the constructor:
Icon(String collection, String icon)
Of course, if I use
new Icon("vaadin", "abacus");
it works.
Since, I can’t use any other value for the first parameter, is this constructor not redundant?
It would have been nice if it can automatically translate to some other collection. (Maybe by default, polymer iron icon).
For example:
new Icon("maps", "directions-bus"); // Creates iron icon
new Icon("xxx", "yyy", "some other icon set"); // In the 3rd parameter, an Icon set can be selected. I just used a String there but it can be a class by itself: IconSet or IconCollection (or some other idea to represent an external icon collection)
This way, IronIcon class can be dropped.