[
https://issues.apache.org/jira/browse/RAVE-613?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13270117#comment-13270117
]
[email protected] commented on RAVE-613:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5061/
-----------------------------------------------------------
Review request for rave.
Summary
-------
This adds the ability to register new popup definitions to be used inside the
container. This is to overcome the issue where we can't skin the current popups.
This is my initial approach. Let me know if there is a better way to handle
this. Ref mailing list discussion at:
http://markmail.org/thread/nipyntndlktosi67
This addresses bug RAVE-613.
https://issues.apache.org/jira/browse/RAVE-613
Diffs
-----
/trunk/rave-portal-resources/src/main/webapp/script/rave.js 1335243
Diff: https://reviews.apache.org/r/5061/diff
Testing
-------
This capability requires having custom js so it isn't able to test as part of
the standard rave install. I use the files below to register my popup which
works. Then I use a modified open_views gadget to load it. The JS below needs
to be loaded through the custom_js.tag mechanism into the container.
var rave = rave || {};
var fullwidth_dialog = {
name:"dialog",
containerSelector:'.popup.dialog',
contentSelector:'.modal-body',
markup:'<div class="popup dialog modal fade"><div
class="modal-body"></div></div>',
initialize:function (container) {
container.find(this.contentSelector).data('popupType', this.name);
var cfg = {
};
container.modal(cfg);
container.on('hidden', function () {
container.detach();
})
},
cleanup:function (content) {
var container = content.parents(this.containerSelector);
container.modal('hide');
},
singleton:false
};
rave.registerPopup("fullwidth_dialog", fullwidth_dialog);
Thanks,
Chris
> Support registering custom popup types with the container
> ---------------------------------------------------------
>
> Key: RAVE-613
> URL: https://issues.apache.org/jira/browse/RAVE-613
> Project: Rave
> Issue Type: Improvement
> Components: rave-portal
> Reporter: Chris Geer
>
> Right now the popup types are hardcoded in the rave.js. We need to support
> integrators defining their own custom popup types since there is no way to
> dynamically skin the popups at the gadget level.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
