Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

MUMUP-2898 Adds ability to write your own directive for removing a widget #393

Merged
merged 3 commits into from
Apr 12, 2017

Conversation

vertein
Copy link
Contributor

@vertein vertein commented Apr 12, 2017

As far as the app-framework is concerned, widgets don't need to be removed, they only need to be rendered. Removal is more for apps that have layouts (like uPortal-home), so this allows the apps using the app-framework to have a removal button and decide what the removal actually does for widgets.


Contributor License Agreement adherence:

@coveralls
Copy link

Coverage Status

Coverage remained the same at 39.018% when pulling ba0728b on MUMUP-2898 into 4b71c17 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 39.018% when pulling ba0728b on MUMUP-2898 into 4b71c17 on master.

@vertein vertein merged commit d017bd3 into master Apr 12, 2017
@vertein vertein deleted the MUMUP-2898 branch April 12, 2017 19:58
@vertein vertein added this to the 4.0.1 milestone Apr 13, 2017
@@ -3,7 +3,7 @@
<!-- OPTION-LINK ICON -->
<div class="option-link-icon">
<a href="{{ widget.selectedUrl }}" target="_blank" rel="noopener noreferrer">
<frame-widget-icon></frame-widget-icon>
<widget-icon></widget-icon>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦‍♂️

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants