-
Notifications
You must be signed in to change notification settings - Fork 192
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
Changed UI of suggested_coding_site.html #551
base: master
Are you sure you want to change the base?
Conversation
hey @harikesh409 I have made a PR regarding the enhancement .waiting for your reply |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where are the images being used?
I have applied the changes you suggested and its working fine on local machine . i have pushed them also so that you can see the changes |
Hey actually i don't have monitor of this size so i made keeping in mind of the laptops and mobile screen .. so i am having hard time visualizing it ..so could you help a bit 🙂 |
@abhiraj-ku |
@harikesh409 i am actually having hard times figuring media queries for this resolution even using devtools so if you could suggest something that would be good 🙂 |
Description
Best and Free Online Websites to Learn Coding for Beginners was not looking good as all the text was in long paragraph style
and also the background gradient colors were not complementary so it not visible also ..
Please include a summary of the change and which issue is fixed.
Fixes #550
The changes made are that the paragraph style text is been changed into responsive cards design made Flex-box and CSS only
List any dependencies that are required for this change
No , Dependencies were required just pure CSS was applied
Type of change
Enhancement of suggested_coding_site.html
How Has This Been Tested?
With the help of developer tools
UI /UX changes
Attach gif or screenshot for changes.
Checklist: