I really doubt that anyone is using the embed property that is in the TextIcon code.
The only renderer I see that is using it is the IconRenderer.
As part of the work to create <span> with attributes from the renderer and <img>/Text with attributes from the skinning definition, I'd like to get rid of the embed property that is in the TextIcon. Any objections? This is a private API right now, which is why I doubt it is being used.

- Jeanne

Jeanne Waldman wrote:

Does anyone know why we have 2 IconRenderers... one in
org.apache.myfaces.trinidadinternal.ui.laf.base.xhtml; (non Faces-major package)

and one in
package org.apache.myfaces.trinidadinternal.renderkit.core.xhtml; (Faces-major package)

I can't tell if the non-Faces-major one is being used anywhere.

Can I delete the non-Faces-major one?

Thanks,
Jeanne




Reply via email to