-
-
Notifications
You must be signed in to change notification settings - Fork 123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Removed inline CSS #348
Removed inline CSS #348
Conversation
Are you sure there aren't anymore inline css present? |
Ok, I will check again in using Find. |
Hey @jajodiaraghav should I also work on inline style tags which have dynamic values. |
@goyal-sidd No. You do not have to remove them. Leave them as is |
I have removed most of style tags, the ones remaining are that are with dynamic entries or are single. |
This looks much better. Thanks @goyal-sidd |
* Removed inline CSS * Removed style from subscriptions' * Removed style from profile
* Removed inline CSS * Removed style from subscriptions' * Removed style from profile
I have removed inline CSS from all the template html files. The some places left are dynamic.
If I have missed anything kindly tell me.
Solves #236