Skip to content

Conversation

xoxys
Copy link
Member

@xoxys xoxys commented Jul 11, 2022

No description provided.

@thegeeklab-bot
Copy link

LHCI Report Overview

Report URL Performance Accessibility Best-Practices SEO PWA
Link http://localhost:41775/ 97 % 100 % 92 % 98 % 30 %
Link http://localhost:41775/404.html 86 % 100 % 100 % 100 % 30 %
Link http://localhost:41775/posts/ 98 % 100 % 100 % 100 % 30 %

@xoxys xoxys merged commit 3a26b6e into main Jul 11, 2022
@delete-merged-branch delete-merged-branch bot deleted the profile-img branch July 11, 2022 18:46
{{- $source := ($.Page.Resources.ByType "image").GetMatch (printf "%s" (.Get "name")) }}
{{- $customAlt := .Get "alt" }}
{{- $customSize := .Get "size" | lower }}
{{- $lazyLoad := default (default true $.Site.Params.GeekblogImageLazyLoading) (.Get "lazy") }}
Copy link

@wolfspyre wolfspyre Jul 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't this be Geekdoc?

Copy link

@wolfspyre wolfspyre Jul 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking about this in general and thought it MIGHT be worth switching all the namespaced vars/attrs/etc to geeklab and only having a select few which are geeklabGB geeklabGD ... might make things easier? or not.... idk really. (I'm rarely convinced my ideas are GOOD...)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the hint! I've created a PR to fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants