search

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

Refactor loading and `Backdrop` usages #285

Open ssciolla opened 2 years ago

ssciolla commented 2 years ago

We have a functional way of setting up loading display in the UI right now, but it requires repetitive styling and Material UI component use to make it work. Some of this could be extracted into a wrapper component. This may also be a good opportunity though to make sure this is the preferred and most accessible way to present a loading state.

pushyamig commented 2 years ago

@ssciolla is this going in this release? if yes, can you give me a estimate on this issue

ssciolla commented 2 years ago

Kinda depends on whether the loading pattern is deemed to be accessible.