TsSkinManager.SetSkinName calls OnActivate?

Viewing 8 posts - 1 through 8 (of 8 total)
  • Author
    Posts
  • #37453
    HeDiBo
    Participant

    OnDeactivate is only called if SkinManager.Active is set to False.

    So, to get things straight when the skin manager is made active, OnActivate should also only be called when SkinManager.Active is set to True.

    As it is now, it is not symmetric, because OnActivate is also called on a skin change. That's where the OnAfterChange event is for.

    It is awkward now, because you cannot do initialization stuff when the skin manager becomes active.

    #56568
    Support
    Keymaster

    You are right, I will check and change it soon.

    #56657
    HeDiBo
    Participant
    'Support' wrote:

    You are right, I will check and change it soon.

    No change in AC 12.08

    #56669
    Support
    Keymaster

    This behaviour will ne changed in the next release.

    #56736
    HeDiBo
    Participant
    'Support' wrote:

    This behaviour will ne changed in the next release.

    Although OnActivate is now called only on activation, OnDeActivate is not called when the SkinManager is destroyed. Allocations in OnActivate will leak memory when SkinManager is destroyed.

    #56737
    HeDiBo
    Participant
    'HeDiBo' wrote:

    Although OnActivate is now called only on activation, OnDeActivate is not called when the SkinManager is destroyed. Allocations in OnActivate will leak memory when SkinManager is destroyed.

    Don't put too much effort into this. It's really a very minor detail.

    #56762
    Support
    Keymaster

    Changed in the v12.11

    #56808
    HeDiBo
    Participant
    'Support' wrote:

    Changed in the v12.11

    Thanksa3.gif

Viewing 8 posts - 1 through 8 (of 8 total)
  • You must be logged in to reply to this topic.