wants to merge 1 commit
Hi.. I think we can reuse the gravatar jinja macro here. WDYT?
[#8304] Use higher resolution for user card avatar icon
Yes good find for the jinja macro. And it actually automatically emits a srcset attribute with 1.5x and 2x rules, so you only need to call it with the actual width 64. Then it'll emit url references for 96px and 128px in the srcset.
The last thing to do, to make those work, is to update the project_icon_sizes setting in the .ini file to also include 128 since it doesn't yet.
You should be able to test this, even if you don't have a hi-dpi (retina) screen or mobile device, by zooming your browser up to 150% and 200% and watch what image is requested.
I am trying to push my changes to this MR. But blocked with following errors with git push
Total 9 (delta 7), reused 0 (delta 0)
remote: fatal: not a git repository: '.'
error: remote unpack failed: unpack-objects abnormal exit
error: RPC failed; curl 56 GnuTLS recv error (-110): The TLS connection was non-properly terminated.
Is there an issue in remote ?
Hm, yea i get a similar error pushing to my fork too. I did update the server a bit recently when blocking a bot. Not sure exactly what is wrong yet.
In the mean time, since you are a project committer, you can push directly to the main "origin" remote https://gitbox.apache.org/repos/asf/allura.git/ instead of to your fork. That's what I do actually a lot. Our pattern of naming branches is to use your initials and ticket number (like ss/8304) so that branches on that remote stay very organized.
With that approach, merge requests aren't possible, but that is ok you can just mention on a ticket when something is ready for review.
Hi.. I've pushed commits to ss/8304. Until we fix the existing issue we can go with this way
@shalithasuranga Pushing to a fork should work now. Problem was in [#8317] due to an earlier update of a Dockfile to use a newer ubuntu. That fix is applied to this allura instance now.
Feel free to use the ss/1234 branch approach too though, either is fine.
Log in to post a comment.