Redesign pre-login pages, add blog and contribute routes by hendriebeats · Pull Request #97 · HopeCommunityChurch/project-2-15

hendriebeats · 2025-04-04T21:13:23Z

Overhauls the home page, adds a blog article page, and adds a contribute page — all the public-facing (non-login) pages. Introduces a shared pre-login.css stylesheet, a reusable header partial, floating box animations, and scroll-reveal on the home page.

Reviewer Q&A

Shared CSS (pre-login.css)
home.css now just imports pre-login.css — all pre-login page styles live in one file so the home, blog, and contribute pages share layout primitives, typography, and the .button-3d component without duplication.

pre-login-header.html partial
The old home page had its own inline <header> + mobile nav <dialog>. Extracting it into a partial means the blog and contribute pages get the same header without copy-paste.

Floating box animations / scroll-reveal
The seven floating-box-*.png images are decorative elements that animate on a float keyframe loop; the JS-driven .reveal class triggers fadeInUp as sections enter the viewport.

home.css reduced to one line
The old home-specific styles were either deleted (superseded by the new design) or moved into pre-login.css. The import stub keeps the <link> in home.html working without a path change.

logInfoSH user in new route handlers
Consistent with Home.hs and Signup.hs — the pattern logs the requesting user (or Nothing) on each page hit.

e.stopPropagation() removed from invite keydown handler (studyGroup.js)
It was preventing the Enter key event from reaching HTMX's form-submit trigger. Removing it lets the form submit correctly without side effects.

base.html whitespace / self-closing tags
Formatting-only change to the <head> font links — no functional difference.

README.md direnv allow steps
Two missing setup steps that tripped up new contributors; added to the numbered list.

boxyoman · 2025-04-05T00:36:38Z

        </a>
      </div>
-      <img src="{{base}}/static/img/meme.png" class="image" >
+      <img src="{{base}}/static/img/meme.png" class="image" />


The / at the end of the tag like that isn't good. At least from what I've read. Never tested it myself.

boxyoman · 2025-04-05T00:38:07Z

-          <li><a href="https://experiencethehope.com/equipping" target="_blank">Equipping</a></li>
-          <li><a href="https://messaging.subsplash.com/25FXCW/auth" target="_blank">Messaging</a></li>
+          <li>
+            <a href="https://experiencethehope.com/teaching" target="_blank"


I don't think I like this formatting...

boxyoman · 2025-04-05T00:40:28Z

 <header id="header">
  <a class="logo" href="{{base}}/">
-    <img class="logo" src="{{base}}/static/img/p215-circle.svg" >
+    <img class="logo" src="{{base}}/static/img/p215-circle.svg" />


remove the / at the end

boxyoman · 2025-04-05T00:40:39Z

  {% include "profile/form.html" %}

-  <br>
+  <br />


remove the /

boxyoman · 2025-04-05T00:41:17Z

-{% include "footer.html" %}
-
-{% endblock %}
+{% include "footer.html" %} {% endblock %}


I think these should be on seprate lines.

boxyoman · 2025-04-05T00:42:21Z

+        Take the complexity out of Bible study and delve deeper into the Word
+        with others
+      </p>
+      <img src="{{base}}/static/img/laptop-mockup.png" />


remove the /

boxyoman · 2025-04-05T00:42:39Z

+{% extends "base.html" %} {% block content %}
 <header id="header">
-  <img src="{{base}}/static/img/p215-circle.svg" class="logo">
+  <img src="{{base}}/static/img/p215-circle.svg" class="logo" />


remove the /

boxyoman · 2025-04-05T00:43:13Z

+      <div class="row share" id="share-{{share.token}}">
+        <div>{{ share.groupStudyName }}</div>
+        <div
+          {%


Yeah, I don't like this formatting at all. That's true for pretty much everything on this page.

I reset the whole "Studies" page to your original

boxyoman · 2025-04-05T00:57:45Z

  display: flex;
-  background:
-    url("/htmx/static/img/background-blue-circles.png") 50%/cover no-repeat,
+  background: url("/htmx/static/img/background-blue-circles.png") 50% / cover


Umm, I think /htmx is wrong. It's my fault, but still wrong.

Not sure what you want me to do about this...

Instead of /htmx/static/img/background-blue-circles.png it should be /static/img/background-blue-circles.png

boxyoman · 2025-04-05T00:59:23Z

@@ -0,0 +1,21 @@
+module Api.Htmx.BlogArticleHistoryOfOutlining where


We can probably combine these all into one module so we don't need to make a different module for every static page we want to make.

Ok. I'm assuming that's something you'd do

You can do it. It's nothing crazy. Instead of putting getBlogArticleHistoryOfOutlining and getContribute in different files, you put them in the same file.

I hear you, but I don't know how to do that... :(

boxyoman · 2025-04-05T01:06:04Z

I don't think the button is supposed to be over like that.

boxyoman · 2025-04-05T01:07:17Z

@@ -1,36 +1,35 @@
-{% extends "base.html" %}
-{% block content %}
+{% extends "base.html" %} {% block content %}


This breaks this page.

boxyoman · 2025-04-05T01:07:27Z

@@ -1,5 +1,4 @@
-{% extends "base.html" %}
-{% block content %}
+{% extends "base.html" %} {% block content %}


This will break this page.

I think I reset this

boxyoman · 2025-04-05T01:07:38Z

@@ -1,5 +1,4 @@
-{% extends "base.html" %}
-{% block content %}
+{% extends "base.html" %} {% block content %}


This will break this page.

boxyoman · 2025-04-05T01:07:47Z

@@ -1,35 +1,32 @@
-{% extends "base.html" %}
-{% block content %}
+{% extends "base.html" %} {% block content %}


This will break this page.

boxyoman · 2025-04-05T01:08:02Z

@@ -1,8 +1,7 @@
-{% extends "base.html" %}
-{% block content %}
+{% extends "base.html" %} {% block content %}


This will break this page.

boxyoman · 2025-04-05T01:08:16Z

-{% extends "base.html" %}
-{% block content %}
-<script async defer src="https://cdn.jsdelivr.net/npm/altcha/dist/altcha.min.js" type="module"></script>
+{% extends "base.html" %} {% block content %}


This will break this page.

boxyoman · 2025-04-05T01:08:25Z

@@ -1,7 +1,6 @@
-{% extends "base.html" %}
-{% block content %}
+{% extends "base.html" %} {% block content %}


This will break this page.

boxyoman · 2025-04-05T01:09:51Z

+      </li>
+      <li>
+        <strong>Connect with Us</strong>: Reach out via
+        <a href="mailto:contact@yourproject.com" target="_blank">email</a> to introduce yourself


You can use developer@p215.church for the email here.

boxyoman · 2025-04-05T01:11:41Z

+      <li>
+        <strong>Explore Our Repository</strong>: Visit our
+        <a href="https://github.com/HopeCommunityChurch/project-2-15" target="_blank"
+          >GitHub page</a


Normally you wouldn't call this a github page. There's actually a feature in github called github pages and people will probably think we used that to make a page, which we didn't. You can probably just word it as "visit the repo" or something like that.

boxyoman · 2025-04-05T01:19:54Z

@hendriebeats From what I can tell, whatever you're using for auto formatting is causing lots of problems. You'll probably have to disable it.

hendriebeats · 2025-04-06T19:07:54Z

Ok, @boxyoman, I made all the changes except for the ones I'd imagine you'd fix. How does it look now? I'll either remove or dramatically pair down my auto formatter

boxyoman · 2025-04-15T23:42:30Z

@hendriebeats Reviewed it again.

hendriebeats · 2025-04-16T19:18:44Z

OK @boxyoman I think i did it

hendriebeats · 2026-02-14T04:21:52Z

@boxyoman checking back in on this branch. I see merge conflicts above. Do you have a sense of the best next steps to get this branch (eventually) merged? I'll plan to not use the formatter I was using before.

boxyoman · 2026-02-18T00:26:54Z

@boxyoman checking back in on this branch. I see merge conflicts above. Do you have a sense of the best next steps to get this branch (eventually) merged? I'll plan to not use the formatter I was using before.

@hendriebeats You can try resolving the conflicts. I can then review it again. (It's not impossible it's easy to just restart).

hendriebeats · 2026-02-18T18:45:50Z

@boxyoman fixed the merge conflicts

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

…ages

- Animate hero heading, subtext, CTA, and image on page load with fadeInUp - Add gradient background to hero section - Add SVG wave dividers between hero/community and community/study-anywhere sections - Add IntersectionObserver scroll-reveal for community, study-anywhere, and custom-study-blocks sections - Set page-level and community section background colours for seamless blending

- Fix translatey typo → translateY in pre-login.css - Remove duplicate @media (max-width: 1130px) block in pre-login.css - Remove shadowed justify-content: center in .page-hero - Remove spurious extra </div> in home.html - Add focusable="false" to decorative wave SVGs - Remove e.stopPropagation() from invite Enter keydown handler

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…lity - Replace instant translateY hover on .button-3d with a single-press bounce keyframe (presses down, slight recovery, settles) - Add z-index: 0 to .section-content so its z-index: -1 floating box children paint behind content but in front of the section background gradient (previously they fell behind the root stacking context and were invisible)

- Wrap all "BEGIN YOUR JOURNEY", "SIGN UP", and "CREATE A STUDY" buttons on the home page and blog article in <a href="/signup"> so they actually navigate to the signup page - Switch header logo to a <picture> element: shows the full P215.png wordmark on wider screens (≥765 px) and the circle SVG on mobile - Add display:block to .logo to eliminate inline-image baseline gap - Replace floating-box-1.png with updated asset; remove misplaced pre-login-pages/P215.png (now served from /static/img/P215.png)

hendriebeats had a problem deploying to dev April 4, 2025 21:13 — with GitHub Actions Failure

hendriebeats had a problem deploying to prod April 4, 2025 21:13 — with GitHub Actions Failure

hendriebeats requested a review from boxyoman April 4, 2025 21:13

boxyoman reviewed Apr 5, 2025

View reviewed changes

boxyoman had a problem deploying to dev April 5, 2025 01:04 — with GitHub Actions Failure

boxyoman had a problem deploying to prod April 5, 2025 01:04 — with GitHub Actions Failure

boxyoman reviewed Apr 5, 2025

View reviewed changes

boxyoman had a problem deploying to prod April 5, 2025 01:14 — with GitHub Actions Failure

boxyoman had a problem deploying to dev April 5, 2025 01:14 — with GitHub Actions Failure

hendriebeats had a problem deploying to prod April 6, 2025 18:05 — with GitHub Actions Failure

hendriebeats had a problem deploying to dev April 6, 2025 18:05 — with GitHub Actions Failure

hendriebeats had a problem deploying to prod April 6, 2025 19:07 — with GitHub Actions Failure

hendriebeats had a problem deploying to dev April 6, 2025 19:07 — with GitHub Actions Failure

hendriebeats had a problem deploying to dev April 16, 2025 19:17 — with GitHub Actions Failure

hendriebeats had a problem deploying to prod April 16, 2025 19:17 — with GitHub Actions Failure

hendriebeats added 5 commits February 19, 2026 03:42

Home page upgrades

ae5e4e5

Formatted pages, kept working on Home

45d5159

Continued building home

1558f8d

Added pages

043742c

Continued building home

9c2922b

hendriebeats added 4 commits February 19, 2026 03:45

Continued building home

9063c66

Make nav bar reuseable

52a771d

better readme

8b72716

made Jon's requested changes

39511a8

hendriebeats force-pushed the updated-non-login-pages branch 3 times, most recently from 56732c1 to 59af445 Compare February 19, 2026 08:55

fixed css bugs

15603b2

hendriebeats force-pushed the updated-non-login-pages branch from 59af445 to 15603b2 Compare February 19, 2026 08:56

Fix share icon sizing and menu link alignment in header

43bbbfc

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

hendriebeats force-pushed the main branch from 3050178 to 37ce5d1 Compare February 20, 2026 19:59

hendriebeats added 2 commits February 20, 2026 22:54

Merge remote-tracking branch 'origin/master' into updated-non-login-p…

b0aa1eb

…ages

hendriebeats changed the title ~~Updated non login pages~~ Redesign pre-login pages, add blog/contribute routes Feb 23, 2026

hendriebeats changed the base branch from main to master February 23, 2026 03:00

hendriebeats and others added 3 commits February 22, 2026 22:03

Allow blog and contribute pages to be viewed by logged-in users

7dc02eb

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Fix extra closing </div> in blog and contribute templates

b54c548

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

hendriebeats force-pushed the updated-non-login-pages branch from df66781 to b54c548 Compare February 23, 2026 03:05

hendriebeats changed the title ~~Redesign pre-login pages, add blog/contribute routes~~ Redesign pre-login pages, add blog and contribute routes Feb 23, 2026

hendriebeats added 2 commits February 22, 2026 22:40

hendriebeats added the Siee: XL / Too large 1,000+ lines: Hard to review; often requires staged PRs or special review strategy. label Feb 24, 2026

		@@ -0,0 +1,21 @@
		module Api.Htmx.BlogArticleHistoryOfOutlining where

Uh oh!

Conversation

hendriebeats commented Apr 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviewer Q&A

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

boxyoman Apr 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

boxyoman commented Apr 5, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

boxyoman commented Apr 5, 2025

Uh oh!

hendriebeats commented Apr 6, 2025

Uh oh!

boxyoman commented Apr 15, 2025

Uh oh!

hendriebeats commented Apr 16, 2025

Uh oh!

hendriebeats commented Feb 14, 2026

Uh oh!

boxyoman commented Feb 18, 2026

Uh oh!

hendriebeats commented Feb 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

hendriebeats commented Apr 4, 2025 •

edited

Loading

boxyoman Apr 5, 2025 •

edited

Loading