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! |
joelbutcher
force-pushed
the
feature/new-default-avatar
branch
from
ea2ab61
to
0d8e599
Compare
|
@driesvints done 👍 |
joelbutcher
force-pushed
the
feature/new-default-avatar
branch
from
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