-
Notifications
You must be signed in to change notification settings - Fork 649
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
Project description placeholder is added #1413
Conversation
β Deploy Preview for 30daysofjs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
If I hover then on whole project image the description is showing then how will I open that project ?
I apologize for not noticing that issue. |
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.
Looks good just remove the lorem ispum and use correct description.
Sure, I will change it with correct project description. |
@Rupa-Rd just make sure all descriptions are of same length and not more than 3 lines. |
I apologize for the later commits. |
@Rupa-Rd Please make it consistent across every project |
@Rupa-Rd Thank you for contributing to this repo. Looking forward for more π |
π οΈ Fixes Issue (Number)
Closes #1199
π¨βπ» Changes proposed and Brief Description
Project description placeholder is added for each project (up-to date). Whenever the user hovers their mouse over the project card, the project description will be shown by fading the background image.
β Check List (Check all the applicable boxes)
π Note to reviewers
I added only the placeholders for the project description.
π· Screenshots