There are static methods on CoreRenderer that I have no idea why they are
static. The two in particular that I noticed:

static public void renderStyleClass
static public void renderStyleClasses

These are methods for renderers, why would they be static?

There are some util style classes that are calling them, because of this, I
would think these methods should be moved into a Util class and have the
Renderer methods invoke the Util ones. IMO, static utility functions do not
belong on classes that are meant to be extended. Do others agree, or am I
solo on this opinion?

-Andrew

Reply via email to