Monorepo for Tangled tangled.org

Appview/pages: decrease spacing between languages in repo summary #856

As the title suggests. Currently the language icons have flex-grow, which means they position themselves to take up the full width of the container, which unfortunately makes for awkward readability in most cases. Afaik the description field doesn't allow uploading of images, but you can see the suggestion on discord here. After this change they would just render with the 14 px gap in-between (wrap to second line on overflow, which looks fine too).

Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:laqygfbyvnkyuhsuaxmp6ez3/sh.tangled.repo.pull/3m7gehxrjla22
+1 -1
Diff #0
+1 -1
appview/pages/templates/repo/index.html
··· 47 47 <div class="px-4 py-2 border-b border-gray-200 dark:border-gray-600 flex items-center gap-4 flex-wrap"> 48 48 {{ range $value := .Languages }} 49 49 <div 50 - class="flex flex-grow items-center gap-2 text-xs align-items-center justify-center" 50 + class="flex items-center gap-2 text-xs align-items-center justify-center" 51 51 > 52 52 {{ template "repo/fragments/colorBall" (dict "color" (langColor $value.Name)) }} 53 53 <div>{{ or $value.Name "Other" }}

History

1 round 3 comments
sign up or login to add to the discussion
jensroemer.com submitted #0
1 commit
expand
Appview:pages: decrease spacing between languages in repo summary
expand 3 comments

lgtm. Thanks!

oops, I forgot to mention about the commit message guideline.

It should be

appview/pages: decrease spacing between languages in repo summary

Signed-off-by: Jens RΓΈmer Hesselbjerg <jh.roemer@gmail.com>

instead.

I rephrased the commit after merge this time. Please make sure to follow the commit message guideline next time! πŸ˜…

Ah sorry about that. The '/' is a typo, and the 'signed-off-by' is because I use Tig for git, and have yet to figure out how to configure it with sign off. So I add the line manually (hence the typo). Thanks for the swift review and merge!

pull request successfully merged