AskDevs
AskDevs copied to clipboard
fixing some styling bugs
Hey @codewithvoid check out some changes I've made.
On small screen the width full was creating this small issue.
And when we're fetching data for those user image that can't be load I think it's better to specify exactly the alt with the name of the user
For example for some image from this twitter url is giving 404 status for those who live in region like Russia. @codewithvoid How do you think if we set a default profile picture when the image src is not correctly fetched???
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
ask-devs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jun 30, 2024 8:41pm |
@codewithvoid I mean here a default profile picture just in the case the image is not provided from the src
@codewithvoid did you review the code?
Thanks @bruxx-6243 for making the changes. I think suggestions related to images are valid. (alt text and default profile picture)
@PriteshKiri is already fixing the width issue in his PRs.
Also, can you sync your changes once..I am seeing additional files in data/users directory.
Done ✅ @codewithvoid
I am seeing some additional files in this change due to format fix. I have sent it separately in #121