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).
+1
-1
appview/pages/templates/repo/index.html
+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
jensroemer.com
submitted
#0
1 commit
expand
collapse
Appview:pages: decrease spacing between languages in repo summary
Signed-off by: Jens RΓΈmer Hesselbjerg <jh.roemer@gmail.com>
expand 3 comments
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
lgtm. Thanks!