moksha/public/javascript/moksha.extensions.js | 7 +++++-- moksha/public/javascript/ui/moksha.ui.popup.js | 2 +- 2 files changed, 6 insertions(+), 3 deletions(-)
New commits: commit bcddee6c5016ed66af6e03233901996ffb52436c Author: John (J5) Palmieri johnp@redhat.com Date: Thu Jul 1 15:47:49 2010 -0400
fix some IE issues - use divs for extention points instead of custom tags
* we were using <moksha_extpoint> tags to denote extention points but this does not work in IE. Namespacing as moksha:extpoint worked but was inconsistant between browsers * we now use <div class="moksha_extpoint"> as it works in all browsers
diff --git a/moksha/public/javascript/moksha.extensions.js b/moksha/public/javascript/moksha.extensions.js index f805913..622dc7f 100644 --- a/moksha/public/javascript/moksha.extensions.js +++ b/moksha/public/javascript/moksha.extensions.js @@ -30,12 +30,15 @@ _extensions.prototype = { }, grep_extensions: function(frag) { - var extension_points = jQuery('moksha_extpoint', frag); + // FIXME: I would like to use namespaces and custom tags but + // IE makes this dificult (and JQuery doesn't fully support it + // so use div with a class instead + var extension_points = jQuery('div.moksha_extpoint', frag); extension_points.each ( function(i) { - var ep = $(this) + var ep = $(this); var data = ep.text(); //remove the extension point ep.replaceWith(''); diff --git a/moksha/public/javascript/ui/moksha.ui.popup.js b/moksha/public/javascript/ui/moksha.ui.popup.js index 117b74e..a3e8038 100644 --- a/moksha/public/javascript/ui/moksha.ui.popup.js +++ b/moksha/public/javascript/ui/moksha.ui.popup.js @@ -35,7 +35,7 @@ $.widget("ui.moksha_popup", { panelClass: 'ui-moksha-popup-panel', hoverTimeout: 500, triggerEvent: 'hover', - fx: null, // e.g. { height: 'toggle', opacity: 'toggle', duration: 200 } + fx: null // e.g. { height: 'toggle', opacity: 'toggle', duration: 200 } },
_create: function() {
moksha-commits@lists.fedorahosted.org