Skip to content

Commit

Permalink
Resolves #1489 - Adds 'notify' to HTML tag when invoked (#1490)
Browse files Browse the repository at this point in the history
Removes it again  when closeNotify() runs.
  • Loading branch information
brian-learningpool authored Mar 28, 2017
1 parent 1089ca5 commit b2d8874
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/core/js/views/notifyView.js
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,8 @@ define(function(require) {
/*ALLOWS POPUP MANAGER TO CONTROL FOCUS*/
Adapt.trigger('popup:opened', this.$('.notify-popup'));
$('body').scrollDisable();

$('html').addClass('notify');

//set focus to first accessible element
this.$('.notify-popup').a11y_focus();
}
Expand Down Expand Up @@ -189,6 +190,8 @@ define(function(require) {
}

$('body').scrollEnable();
$('html').removeClass('notify');

Adapt.trigger('popup:closed');
Adapt.trigger('notify:closed');
}
Expand Down

0 comments on commit b2d8874

Please sign in to comment.