Commit e02f1e9a by Braden MacDonald

Merge pull request #67 from open-craft/jill/fix-a11y-remove-role

Removes role="application"
parents 5a5efe9a a6786f47
...@@ -279,7 +279,7 @@ function DragNDropTemplates(url_name) { ...@@ -279,7 +279,7 @@ function DragNDropTemplates(url_name) {
problemHeader, problemHeader,
h('p', {innerHTML: ctx.problem_html}), h('p', {innerHTML: ctx.problem_html}),
]), ]),
h('section.drag-container', { attributes: { role: 'application' } }, [ h('section.drag-container', {}, [
h( h(
'div.item-bank', 'div.item-bank',
renderCollection(itemTemplate, items_in_bank, ctx) renderCollection(itemTemplate, items_in_bank, ctx)
......
...@@ -161,7 +161,7 @@ class TestDragAndDropRender(BaseIntegrationTest): ...@@ -161,7 +161,7 @@ class TestDragAndDropRender(BaseIntegrationTest):
def test_drag_container(self): def test_drag_container(self):
self.load_scenario() self.load_scenario()
item_bank = self._page.find_element_by_css_selector('.drag-container') item_bank = self._page.find_element_by_css_selector('.drag-container')
self.assertEqual(item_bank.get_attribute('role'), 'application') self.assertIsNone(item_bank.get_attribute('role'))
def test_zones(self): def test_zones(self):
self.load_scenario() self.load_scenario()
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment