Heading "Canvas Course Manager" should be an h1

tl-its-umich-edu / canvas-course-manager-next

Canvas Course Manager Next: A redesign of the existing CCM application. It extends Canvas features, makes cumbersome features easier to use, and adds new features.

8 stars 9 forks source link

Heading "Canvas Course Manager" should be an h1 #261

Closed pushyamig closed 2 years ago

pushyamig commented 2 years ago

Location: Home Impact: Major

Issue Heading "Canvas Course Manager" should be an h1

Impact Beneficial for screen readers, specially in iframed apps.

Solution Make an h1. If framework h1 style does not suit, can override the style.

pushyamig commented 2 years ago

This is resolved as part of this issue #240 PR. So closing this

gsilver commented 2 years ago

Fixed.

pushyamig commented 2 years ago

Thanks @gsilver for the feedback!

gsilver commented 2 years ago

Fixed!