Skip to content

feat: implement basic blog#3

Open
gibbiemonster wants to merge 7 commits intomainfrom
feature/blog
Open

feat: implement basic blog#3
gibbiemonster wants to merge 7 commits intomainfrom
feature/blog

Conversation

@gibbiemonster
Copy link
Contributor

Implement basic blog.

Open Q's:

  • where does this go on the main site?
  • do we put the posts somewhere on the main site?

This comment was marked as resolved.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 26, 2026

Deploying site with  Cloudflare Pages  Cloudflare Pages

Latest commit: fa8596f
Status: ✅  Deploy successful!
Preview URL: https://ae4f4d74.site-ehu.pages.dev
Branch Preview URL: https://feature-blog.site-ehu.pages.dev

View logs

Copy link
Contributor

@cerqiest cerqiest left a comment

Choose a reason for hiding this comment

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

Personally, I think the way that the app is routed is a little odd and surely there must be a better solution. I do not love the idea of duplicating the HTML shell, especially the meta tags. You should look into https://www.npmjs.com/package/@lit-labs/router or some other routing solution (maybe even build your own).

In general with the font, as the size goes down it becomes more and more illegible. I think either picking a new font (maybe less narrow) and also removing/toning down the text effect may help improve readability. It works fine in the titles, but in the body of the text which is meant to be read for what could be a long period of time, it is uncomfortable.

@cerqiest
Copy link
Contributor

Clicking on either of the 2 new projects added to the homepage opens /# in a new tab.

@gibbiemonster
Copy link
Contributor Author

Clicking on either of the 2 new projects added to the homepage opens /# in a new tab.

This is intentional. Some projects don't have a public facing URL yet

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