MyHome-Web icon indicating copy to clipboard operation
MyHome-Web copied to clipboard

Colour of notification bell has improved, and the bell is pointing to notification route

Open some1das opened this issue 3 years ago β€’ 0 comments

…on route

πŸš€ Description

I have changed the colour of the notification bell to black because the previous colour was not looking suitable, also pointed the bell icon to the notification route

πŸ“„ Motivation and Context

This is my first open-source contribution, I always wanted to contribute an open-source project finally I found this project which is looking quite interesting, I am trying to find some more ways to contribute to this project.

The bell icon was not visible properly so I have changed its colour to black and now it is visible properly On clicking the bell icon there was no activity happened to the application, idealy it should point to notification route, and this is exactly what I have done here

πŸ§ͺ How Has This Been Tested?

I have tested it on my local machine and it was working perfectly, I am sure it will work perfectly in your machine too.

No my changes are not effecting in the other areas of the code

πŸ“· Screenshots (if appropriate)

πŸ“¦ Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

βœ… Checklist

  • [x] My code follows the code style of this project(Do your best to follow code styles. If none apply just skip this).
  • [ ] My change requires a change to the documentation.
  • [ ] I have updated the documentation accordingly.

some1das avatar Aug 05 '21 16:08 some1das