Skip to content

Commit

Permalink
Merge pull request #635 from plonejp/master
Browse files Browse the repository at this point in the history
Fix `aria-hidden` attribute control problem on folder content panel
  • Loading branch information
vangheem committed Feb 18, 2016
2 parents 2955ae8 + 4f6cf9f commit 0b37181
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 2 additions & 0 deletions CHANGES.rst
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@ Fixes:
- Fix XSS vulnerability issues in structure and relateditem pattern.
[metatoaster]

- Fix `aria-hidden` attribute control problem on folder content panel
[terapyon]
- Trim links in tinymce before inserting them in the source.
[Gagaro]

Expand Down
3 changes: 2 additions & 1 deletion mockup/js/ui/views/popover.js
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ define([
this.bindTriggerEvents();

this.on('render', function() {
this.$el.attr('role', 'tooltip').attr('aria-hidden', 'true');
this.$el.attr('role', 'tooltip').attr('aria-hidden', 'false');
this.renderTitle();
this.renderContent();
}, this);
Expand Down Expand Up @@ -186,6 +186,7 @@ define([
this.$el.removeClass('active');
if (this.triggerView) {
this.triggerView.$el.removeClass('active');
this.triggerView.$el.attr('aria-hidden', 'true');
}
this.uiEventTrigger('hide', this);
this.$el.attr('aria-hidden', 'true');
Expand Down

0 comments on commit 0b37181

Please sign in to comment.