dynamic inline fields focus #93

Merged
merged 2 commits into from Sep 24, 2011

Projects

None yet

2 participants

@diramazioni
Contributor

fix issue with the mouse loosing focus in dynamic inline fields. Affects Firefox users

added an Italian translation message

@diramazioni diramazioni fix issue with the mouse loosing focus in dynamic inline fields. Affe…
…cts Firefox users

added an Italian translation message
f7df23a
@diramazioni diramazioni and 1 other commented on an outdated diff Sep 22, 2011
mezzanine/core/media/js/dynamic_inline.js
@@ -38,6 +38,8 @@ $(function() {
placeholder: 'placeholder'});
$(parentSelector).disableSelection();
$('.ordering').css({cursor: 'move'});
+ $(parentSelector).enableSelection();
+
@diramazioni
diramazioni Sep 22, 2011 Contributor

no sure if this is the best place to re-enable but looks to work nice

@stephenmcd
stephenmcd Sep 23, 2011 Owner

I think what you've done actually removes any selection disabling entirely.

What might be a better fix is to just change:

$(parentSelector).disableSelection();

to:

$(parentSelector + ' .order').disableSelection();

Can you give that a try and see if it works in FF6?

@diramazioni
diramazioni Sep 23, 2011 Contributor

yes, your real fix works in FF6 I made the commit.
sorry, anyway I don't see anything selectable even enabling it
btw thanks

@stephenmcd stephenmcd merged commit 69e3be3 into stephenmcd:master Sep 24, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment