Why Listeners are not themable ?

[reverseblade]

I've seen

[Meta]
        [ConfigOption("listeners", JsonMode.Object)]
        [Category("2. Observable")]
        [Themeable(false)]
        [NotifyParentProperty(true)]
        [PersistenceMode(PersistenceMode.InnerProperty)]
        [DesignerSerializationVisibility(DesignerSerializationVisibility.Visible)]
        [ViewStateMember]
        [Description("Client-side JavaScript Event Handlers")]
 public ButtonListeners Listeners
        {
            get
            {
                if (this.listeners == null)
                {
                    this.listeners = new ButtonListeners();
                }

                return this.listeners;
            }
        }

Making listeners Themeable might help to reuse them via Skins. I think this would be handy even if listeners is irrelevant with looks.

[geoffrey.mcgill]

Hi,

Ya, I'm not entirely sure why [Themeable(false)] would be required for the Listeners and DirectEvents.

I just ran several tests without the Attribute and setting the properties work perfectly in a .skin file.

I've remove the Themeable Attribute from all .Listeners and .DirectEvents. The revised code has been committed to SVN and will be publicly available with the upcoming v1.0 release.

Thanks for pointing out the problem.

Hope this helps.