-
Notifications
You must be signed in to change notification settings - Fork 361
Cleanup settings #580
Comments
+1 |
My next couple of weeks are bit busy but I am going to start contributing to the UI efforts by working on the settings page. |
@umairidris might beat you to it since all this is blocking deploying. Since we are unhappy with the drawer. Also, might make it a modal. A whole page might not be needed for it. |
You can see some of the groundwork I was doing here: https://github.com/umairidris/quran.com-frontend/tree/settingspage May help! I will let you continue from here. It makes sense to get the basics done (not everything needs to be done right away) and things can be tweaked/improved for future releases when the groundwork is laid. p.s. I am seeing a lot of new warnings, was some sort of checker enabled? |
Keeping font size out of settings modal. Might make sense to have it more accessible? |
Sample: http://i.imgur.com/JroM2lH.png
The text was updated successfully, but these errors were encountered: