[MGNLVA-1] Migrate ContextMenu addon on the fly Created: 12/Oct/17 Updated: 19/Oct/17 Resolved: 18/Oct/17 |
|
| Status: | Closed |
| Project: | Magnolia Vaadin Compatibility Addons |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.0 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Ilgun Ilgun | Assignee: | Aleksandr Pchelintcev |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoR: |
Empty
|
| Epic Link: | Vaadin 8 |
| Sprint: | Basel 118 |
| Story Points: | 5 |
| Description |
|
Previously we used to fork the source code of third party ContextMenu add-on and add the required features to the fork (icon font support, additional RPC API etc). We also had a separate workaround for the user menu case. With this effort we move out the forked code from UI to the compatibility add-ons project and use patching mechanism to add the required functionality. We also reduce the difference between ordinary ctx menu and user menu to minimum. |