Permalink
Browse files

fix bug 1343, so View Diff works in IE

  • Loading branch information...
1 parent fec373e commit 534e739a34e6d71835033bcf560cb54745dd9222 @slanning slanning committed Jul 14, 2008
Showing with 12 additions and 1 deletion.
  1. +5 −0 comp/widgets/story_prof/story_prof.mc
  2. +3 −1 lib/Bric/App/Callback/Profile/Story.pm
  3. +4 −0 lib/Bric/Changes.pod
@@ -101,6 +101,11 @@ if ($id) {
}
+ if (not exists $param->{diff} and exists $param->{'diff.x'}) {
+ # IE only sends diff.x and diff.y for <input type="image">
+ $param->{diff} = 1;
+ }
+
if ($param->{diff}) {
my $version = $story ? $story->get_version : 0;
@@ -330,7 +330,9 @@ sub return : Callback(priority => 6) {
my $story = get_state_data($widget, 'story');
- if ($version_view || $self->value eq 'diff') {
+ # note: $self->value =~ /^\d+$/ is for IE which sends the .x or .y position
+ # of the mouse for <input type="image"> buttons
+ if ($version_view || $self->value eq 'diff' || $self->value =~ /^\d+$/) {
my $story_id = $story->get_id;
$self->clear_my_state if $version_view;
$self->set_redirect("/workflow/profile/story/$story_id/?checkout=1");
View
@@ -94,6 +94,10 @@ and Safari. [Matt Rolf]
Tweaked Bricolage Instance Name display. [Matt Rolf]
+=item *
+
+Fixed bug 1343, View Diff wasn't working in IE. Reported by Ed Guzman. [Scott]
+
=back
=head1 VERSION 1.10.5 (2008-04-29)

0 comments on commit 534e739

Please sign in to comment.