-
-
Notifications
You must be signed in to change notification settings - Fork 655
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
Add a new default avatar using a greyscale Laravel logo #759
Add a new default avatar using a greyscale Laravel logo #759
Conversation
Can you post some screenshots of the profile page, thread detail page, forum overview page and homepage with the user cloud so we can see how this looks? |
@driesvints Yup, I took screenshots, but in my haste forgot to upload them with the PR 🤦 |
@joelbutcher hmm I'm now thinking that maybe the Laravel.io logo might be a bit better. Can you replace that with https://github.com/laravelio/laravel.io/blob/main/public/images/laravelio-icon.svg ? It can even be a slightly more lighter gray so it blends in with the background a bit more. The profile image setting one can be a bit smaller like the one on the profile page. |
@driesvints like this? |
@driesvints how do these screenshots look to you? |
@joelbutcher I haven't found time yet to look at this yet, sorry. Might be a while. |
@joelbutcher heya, I'm so sorry I didn't get to this yet. Could you maybe rebase this PR with master and make sure it's adapted for Tailwind v3? I'll try to work on it in the next couple of weeks. Thanks! |
ea2ab61
to
0d8e599
Compare
@driesvints done 👍 |
0d8e599
to
0e3133a
Compare
Thanks for this @joelbutcher! Sorry it took this long before I got to this. |
@driesvints No problems - I just assumed you're busy with the next Laravel release this month! |
Resolves #688