Code review comment for lp:~abreu-alexandre/ubuntu-ui-toolkit/add-public-actionmanager-alias
Revision history for this message
| Alexandre Abreu (abreu-alexandre) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
> 20 + /*!
> 21 + The current ActionManager that supervises the global and local
> ActionContexts.
> 22 +
> 23 + \qmlproperty UnityActions.
> 24 + */
> 25 + property alias actionManager: unityActionManager
> 26 +
>
> I wouldn't say "current" because an app normally has just one ActionManager.
>
> Please add a note with a reference to the actions property which should be
> used in most cases, and mention that the developers only need to use the
> actionManager for specific cases (list example). That will avoid developers
> creating complex code using the actionManager if it is not really needed.
done