Clock icon in courseware sidebar does not follow conventions for notifications

[aphilipharrell]

The clock icon in the courseware sidebar leads to a Notifications window - the clock icon is misleading and this should be replaced with something more commonly understood to represent notifications.

Suggested fix:

Replace WatchOutline with Notifications in src/courseware/course/sidebar/sidebars/notifications/NotificationIcon.jsx

[jciasenza]

@arbrandes
I assign it to myself

[jciasenza]

Hello aphilipharrell, I already made the PR

#1510

[angonz]

@crathbun428 Can you please check if this request can be implemented? We can do it if approved.

[crathbun428]

@angonz - There is potentially going to be some work done on the right-sidebar soon. I think thinking through what icons are shown will be apart of this work. I want to check in with @brian-smith-tcril and @jmakowski1123 to make sure I have this right.

[itsjeyd]

Hi @aphilipharrell, just a heads-up that we ended up closing #1510 because it looked like someone had already made the same changes for Sumac.

So you might want to consider closing this issue, too.

CC @jciasenza @crathbun428