Skip to content

Comments

Templating#23

Open
BenjaRogers wants to merge 6 commits intomainfrom
templating
Open

Templating#23
BenjaRogers wants to merge 6 commits intomainfrom
templating

Conversation

@BenjaRogers
Copy link

Add templates from frontend repo and basic routes for them. Probably worth adding a base template for navbar/footer.

Copy link
Contributor

@FFY00 FFY00 left a comment

Choose a reason for hiding this comment

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

This is good, thank you!

I think the main thing we can improve here is using Jinja's template inheritance, so that we don't need to repeat the header, navbar, footer, etc. everywhere. See, https://jinja.palletsprojects.com/en/3.1.x/templates/#template-inheritance

Other than that, I think the contents of style.css should probably be moved to style.scss. I see this PR just copies how it was done in the front-end repo, @redstarblanket is there any specific reason for this?

Also, for future reference, when building on other's work, there's a tag we can use to document that! You just need to add the Co-authored-by tag to the commit body, which can also be repeated if there's more than one other people. I can do this when I merge this PR, if I select the "squash and merge" option, so don't worry about it for this PR.

Co-authored-by: Em <75779842+redstarblanket@users.noreply.github.com>

(email taken from https://github.com/peepo-world/frontend/commit/5718097a65cefa305a33a4ff5282357ef329457a.patch)
You can find more at https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors.


Not related to this PR, but I just wanted to note here for the future. We should probably add a Jinja+HTML formatter, like https://www.djlint.com/, to pre-commit.

We should also have a "playground", essentially a simple webserver that just renders the templates with dummy data, allowing people to work on the front-end without having to setup the whole project (database, object storage server, etc.).

Comment on lines +7 to +11
@import "node_modules/bulma/sass/utilities/initial-variables";



@import "node_modules/bulma/bulma";
Copy link
Contributor

Choose a reason for hiding this comment

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

Nothing for you to do here, but I just wanted to note that I plan to change this to use git submodules node, like in https://github.com/FFY00/ffy00.github.io/.

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.

2 participants