modified the login form and background image by Maxwell-Ipeni · Pull Request #7955 · ChurchCRM/CRM

Maxwell-Ipeni · 2026-02-07T16:44:01Z

What Changed

Fixes #

Type

Testing

Screenshots

Security Check

Introduces new input validation
Modifies authentication/authorization
Affects data privacy/GDPR

Code Quality

Database: Propel ORM only, no raw SQL
No deprecated attributes (align, valign, nowrap, border, cellpadding, cellspacing, bgcolor)
Bootstrap CSS classes used
All CSS bundled via webpack

Pre-Merge

Tested locally
No new warnings
Build passes
Backward compatible (or migration documented)

DawoudIO · 2026-02-07T18:16:48Z

Thanks @Maxwell-Ipeni for the PR, can you please fill in the desc, please include before and after images etc

DawoudIO · 2026-02-07T18:17:03Z

src/session/templates/begin-session.php

 use ChurchCRM\dto\SystemURLs;

-$sPageTitle = gettext('Login');
+$sPageTitle = 'Login';


we need this for localization

DawoudIO · 2026-02-07T18:17:31Z

src/session/templates/begin-session.php

+<style>
+@import url('https://fonts.googleapis.com/css2?family=Playfair+Display:ital@0;1&family=Inter:wght@400;500;600;700&display=swap');
+
+body.login-page {


all css should move from inline CSS to webpack

Copilot

Pull request overview

Updates the unauthenticated session “begin session” (login) template to use a new visual design and adds a new background image asset for the login page.

Changes:

Replaces the existing login page markup with a new custom-styled layout (inline CSS + updated structure).
Adds a new login background image (church-bg.png) referenced by the template.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 6 comments.

File	Description
src/session/templates/begin-session.php	Reworks login UI, styling, and displayed links/messages.
src/Images/church-bg.png	Adds new background image used by the login page.

Copilot · 2026-02-07T21:23:11Z

src/session/templates/begin-session.php

 use ChurchCRM\dto\SystemURLs;

-$sPageTitle = gettext('Login');
+$sPageTitle = 'Login';


$sPageTitle was changed from a translated string to a hard-coded literal. This makes the page title non-localizable; please keep using gettext('Login') (consistent with other session templates).

Suggested change

$sPageTitle = 'Login';

$sPageTitle = gettext('Login');

Copilot · 2026-02-07T21:23:12Z

src/session/templates/begin-session.php

+body.login-page {
+    background: url('/Images/church-bg.png') no-repeat center center fixed;
+    background-size: cover;


The CSS background image uses an absolute path (/Images/...) which will break when ChurchCRM is installed in a subdirectory. Use SystemURLs::getRootPath() (or the provided sRootPath) to build the image URL.

Copilot · 2026-02-07T21:23:12Z

src/session/templates/begin-session.php

+            <h1 class="login-title-church">CHURCH <i class="fa-solid fa-cross login-title-icon"></i> <span class="login-title-crm">CRM</span></h1>
+            <p class="login-subtitle">Main St. Cathedral</p>
+            <p class="login-subtitle-2">Secure Login</p>


The login header/subtitle is hard-coded (including a specific church name). This should use ChurchMetaData::getChurchName() and wrap user-facing text in gettext() so it works for different deployments and locales.

Copilot · 2026-02-07T21:23:12Z

src/session/templates/begin-session.php

            <?php
            if (isset($_GET['Timeout'])) {
-                $loginPageMsg = gettext('Your previous session timed out.  Please login again.');
+                echo '<div class="login-error">Your previous session timed out. Please login again.</div>';


The timeout message is now hard-coded and no longer localized. Please use gettext() (and keep wording consistent with the previous translated message) so this notice is translatable.

Suggested change

echo '<div class="login-error">Your previous session timed out. Please login again.</div>';

echo '<div class="login-error">' . gettext('Your previous session timed out. Please login again.') . '</div>';

Copilot · 2026-02-07T21:23:12Z

src/session/templates/begin-session.php

+                    <input type="text" id="UserBox" name="User" class="login-input" value="<?= $prefilledUserName ?>" placeholder="Email/Username" required autofocus>
+                    <span class="input-icon"><i class="fa-solid fa-envelope"></i></span>
                </div>
-                <div class="input-group mb-3">
-                    <input type="password" id="PasswordBox" name="Password" class="form-control" placeholder="<?= gettext('Password') ?>" required autofocus>
-                </div>
-                <div class="row">
-                    <div class="col-12">
-                        <button type="submit" class="btn btn-primary btn-block">Sign In</button>
-                    </div>
-                    <!-- /.col -->
+                <div class="login-input-group">
+                    <input type="password" id="PasswordBox" name="Password" class="login-input" placeholder="Password" required>
+                    <span class="input-icon"><i class="fa-solid fa-lock"></i></span>
                </div>
+                <button type="submit" class="login-btn">
+                    <i class="fa-solid fa-cross"></i>
+                    Sign In
+                </button>


Several user-facing strings in the form (placeholders, button label) are no longer wrapped in gettext(), so they won’t be translated. Please restore gettext() usage for these strings to match other session templates.

Copilot · 2026-02-07T21:23:12Z

src/session/templates/begin-session.php

+            <div class="login-forgot">
+                <a href="<?= $forgotPasswordURL ?>">I forgot my password</a>
+            </div>
+
+            <div class="login-divider">
+                <span>OR</span>
+            </div>
+
+            <div class="login-register">
+                <p>New to Main St. Cathedral?</p>
+                <a href="<?= SystemURLs::getRootPath() ?>/external/register/" class="login-register-btn">
+                    <i class="fa-solid fa-user-check"></i>
+                    Register a New Family
+                </a>
+            </div>


The Lost Password and Self Registration links are now shown unconditionally. This ignores the existing SystemConfig flags (bEnableLostPassword / bEnableSelfRegistration) and can expose broken links (404 / “not available”). Please gate these sections behind the same config checks as before.

Maxwell-Ipeni · 2026-02-07T22:38:52Z

@DawoudIO Apologies for not adding the description and the images. I'll do better next time.

✨ Feature
Redesigned login screen with immersive church interior background for contextual relevance
Implemented modern gradient overlay with optimized opacity (70%) for text readability
Revamped logo treatment: "CHURCH + CRM" with gold accent and refined typography
Enhanced form elements with:
Rounded inputs with subtle shadows
Custom iconography (envelope/lock)
Improved spacing (24px vertical padding)
Gold accent on primary CTA button

♻️ Refactor
Main Change: Made login screen fully mobile-responsive (tested down to 320px) with SVG logo, accessible HTML semantics, CSS variables (--primary-gold, --bg-overlay), and 18% smaller CSS bundle.

Bug fix
Fixed login form overflow on mobile viewports
Corrected inconsistent input focus states

Security
No security changes required (UI-only update)

Original UI

New UI

modified the login form and background image

e0f7bc5

Maxwell-Ipeni requested a review from a team as a code owner February 7, 2026 16:44

Maxwell-Ipeni requested review from DAcodedBEAT, DawoudIO, MrClever, bigtigerku, grayeul and respencer and removed request for a team February 7, 2026 16:44

Merge branch 'master' into new

bd6fee5

DawoudIO requested changes Feb 7, 2026

View reviewed changes

DawoudIO requested a review from Copilot February 7, 2026 21:17

Copilot started reviewing on behalf of DawoudIO February 7, 2026 21:18 View session

Copilot AI reviewed Feb 7, 2026

View reviewed changes

Merge branch 'master' into new

2cc24b4

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

modified the login form and background image#7955

modified the login form and background image#7955
Maxwell-Ipeni wants to merge 3 commits intoChurchCRM:masterfrom
Maxwell-Ipeni:new

Maxwell-Ipeni commented Feb 7, 2026

Uh oh!

DawoudIO commented Feb 7, 2026

Uh oh!

DawoudIO Feb 7, 2026

Uh oh!

DawoudIO Feb 7, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Maxwell-Ipeni commented Feb 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	echo '<div class="login-error">Your previous session timed out. Please login again.</div>';
	echo '<div class="login-error">' . gettext('Your previous session timed out. Please login again.') . '</div>';

Conversation

Maxwell-Ipeni commented Feb 7, 2026

What Changed

Type

Testing

Screenshots

Security Check

Code Quality

Pre-Merge

Uh oh!

DawoudIO commented Feb 7, 2026

Uh oh!

DawoudIO Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

DawoudIO Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Maxwell-Ipeni commented Feb 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants