Skip to content

Commit

Permalink
Put role=main on Editor's <div> rather than <body>,
Browse files Browse the repository at this point in the history
since "main" is an invalid role for <body>.

Fixes #18731, refs #16140.
  • Loading branch information
wkeese committed Oct 7, 2015
1 parent cef0a82 commit 1764dea
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions _editor/RichText.js
Expand Up @@ -560,7 +560,7 @@ define([
// The contents inside of <body>. The real contents are set later via a call to setValue().
// In auto-expand mode, need a wrapper div for AlwaysShowToolbar plugin to correctly
// expand/contract the editor as the content changes.
var html = "<div id='dijitEditorBody'></div>";
var html = "<div id='dijitEditorBody' role='main'></div>";

var font = [ _cs.fontWeight, _cs.fontSize, _cs.fontFamily ].join(" ");

Expand Down Expand Up @@ -657,7 +657,7 @@ define([
(has("ie") || has("trident") ? "" : "\tli{ min-height:1.2em; }\n"),
"</style>\n",
this._applyEditingAreaStyleSheets(), "\n",
"</head>\n<body role='main' ",
"</head>\n<body ",

// Onload handler fills in real editor content.
// On IE9, sometimes onload is called twice, and the first time frameElement is null (test_FullScreen.html)
Expand Down

0 comments on commit 1764dea

Please sign in to comment.