[MGNLUI-1991] Cleanup Actionbar dependencies and apply proper MVP pattern Created: 28/Aug/13 Updated: 15/Oct/15 Resolved: 04/Apr/14 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | actionbar |
| Affects Version/s: | 5.1 |
| Fix Version/s: | 5.3 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Mikaël Geljić | Assignee: | Mikaël Geljić |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | tech-debt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Template: |
|
||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||
| Task DoR: |
Empty
|
||||||||||||||||
| Epic Link: | Widgetset | ||||||||||||||||
| Description |
|
The Actionbar widget currently implements the View interface (ui-api) instead of having the ActionbarPresenter produce a ViewImpl that uses the Widget internally. There are also further issues that do not fall inline with the MVP pattern.
|
| Comments |
| Comment by Mikaël Geljić [ 02/Sep/13 ] |
|
Decided to attach this as a subtask of the
The sooner we review it and merge it in, the bigger the chance to spot any potential issue. |
| Comment by Mikaël Geljić [ 17/Sep/13 ] |
|
Was never integrated so far, not to increase merge hell for the i18n branch. Change needs to be readapated accordingly, and will no land in 5.1. |