Mokse 24 home page by BorDevTech · Pull Request #96 · codingUnited/MokseWebsite

BorDevTech · 2026-04-10T02:25:17Z

No description provided.

…-footer Remove unused "Home" link from footer component

-WARN this is a mockup, official terms needed.

…plate and adjusting SectionTemplate structure

…ss hydration warning in layout

…usting layout structure

…s-and-Conditions-page Add Terms and Conditions page with structured content and templates

…ed-in-footer Added club recognition to footer with link to club website

…n-on-the-home Mokse 20 update donate button on the home

Mokse 48 footer component

Add Programs page and enhance Services page layout - Created a new Programs page with a header template and introductory text. - Updated Services page layout to improve structure and added HStack for better alignment of support options. - Modified RootLayout to suppress hydration warnings. - Enhanced Navbar links with badges for new sections.

Refactor HeaderTemplate and ServiceCard components for improved layout and consistency; update SupportCard interface to remove unused properties.

…MokseWebsite into MOKSE-24-Home-Page

Copilot

Pull request overview

Updates the Mokse site’s home/services experience and introduces new top-level pages by adjusting service/support card behavior, navigation badges, and header layout flexibility.

Changes:

Refactors Services “support options” cards/data to remove image/link fields and updates service CTA links.
Enhances HeaderTemplate with configurable grid span/width for title positioning and adds “New” badges in the navbar.
Adds new Programs and Terms and Conditions pages; updates the home-page Donate CTA and root layout hydration handling.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 11 comments.

Show a summary per file

File	Description
`data/services.ts`	Updates service/support option data and links; removes support option media/link fields.
`components/services/support-card.tsx`	Simplifies SupportCard rendering (icon + text) and comments out image/link support.
`components/services/service-card.tsx`	Tweaks service card icon/link rendering and layout.
`components/page-builder/template.tsx`	Adds new header layout controls via grid span/width props.
`components/common/navbar.tsx`	Wraps nav items to properly float “New” badges over text.
`components/common/footer.tsx`	Updates footer attribution link text/content.
`app/page.tsx`	Changes Donate button destination to PayPal.
`app/layout.tsx`	Adds `suppressHydrationWarning` on `<html>`.
`app/(Pages)/terms/page.tsx`	Adds a new Terms and Conditions page.
`app/(Pages)/services/page.tsx`	Replaces the support options grid with a wrapping centered HStack layout.
`app/(Pages)/programs/page.tsx`	Adds a new Programs landing page using `HeaderTemplate`.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-10T02:39:40Z

+import { Card, Image as ChakraImage, Float, Heading } from "@chakra-ui/react";
 import { Link as ChakraLink, SimpleGrid } from "@chakra-ui/react";



SimpleGrid is imported but unused in this component. This will be reported by the Next.js TypeScript ESLint preset; remove the unused import (or use it if intended).

Copilot · 2026-04-10T02:39:41Z

+import { Card, Image as ChakraImage, Heading, Text } from "@chakra-ui/react";
 import { Link as ChakraLink, SimpleGrid } from "@chakra-ui/react";


ChakraImage, ChakraLink, and SimpleGrid are currently unused because the image/link sections are commented out. Please remove unused imports (or restore usage) to avoid lint failures.

Copilot · 2026-04-10T02:39:41Z

  title: string;
  description: string;
-  image: string;
-  imageAlt: string;
-  iconLink: string;
+  // image: string;
+  // imageAlt: string;
+  // iconLink: string;


SupportCardProps keeps removed fields as commented-out lines. This makes the component contract ambiguous; prefer deleting these fields entirely or reintroducing them as optional, typed props if they’re still part of the intended UI.

Copilot · 2026-04-10T02:39:41Z

  description,
-  image,
-  imageAlt,
-  iconLink,
+  // image,
+  // imageAlt,
+  // iconLink,


The component parameter list also contains commented-out props. Keeping dead/commented code here makes it harder to see what the component actually supports; please remove the commented parameters (or re-enable the feature and update types/data accordingly).

Copilot · 2026-04-10T02:39:41Z

  title: string;
  description: string;
-  image: string;
-  imageAlt: string;
-  iconLink: string;
+  // image: string;
+  // imageAlt: string;
+  // iconLink: string;


SupportOption has its removed fields left in as commented-out properties. Prefer removing these lines or modeling them as optional properties (e.g., image?: string) so the type accurately reflects what supportOptions can contain.

Copilot · 2026-04-10T02:39:42Z

 import { services, supportOptions, faqItems } from "../../../data/services";
-import { Link as ChakraLink, SimpleGrid } from "@chakra-ui/react";
+import { Link as ChakraLink, HStack, SimpleGrid } from "@chakra-ui/react";


faqItems and ChakraLink are imported but unused (FaqItem pulls faqItems internally). Please remove unused imports to avoid lint failures.

Copilot · 2026-04-10T02:39:42Z

 export function HeaderTemplate({
  title,
+  titleAlignment = 1,
+  titleLocation = 75,
  image,


The new titleAlignment / titleLocation names are misleading: they represent a grid column span and a width percentage, not alignment/location in the typical sense. Consider renaming to something like titleColSpan / titleWidthPercent to make call sites clearer.

Copilot · 2026-04-10T02:39:42Z

        <SimpleGrid columns={2} gap={6}>
-          <VStack direction={direction} w={"75%"} pl={16}>
-            <Heading as={"h1"} p={2} alignItems={"left"}>
-              <Text
-                textStyle={"7xl"}
-                className={poppins.className}
-                // textAlign={"start"}
-                textTransform={"capitalize"}
-                overflow={"hidden"}
-                wordBreak={"none"}
-                verticalAlign={"baseline"}
-                _light={{ color: "white" }}
-              >
-                {title}
-              </Text>
-            </Heading>
-            <Container fluid>
-              <Text _light={{ color: "white" }}>{description}</Text>
-            </Container>
-          </VStack>
+          <GridItem colSpan={titleAlignment}>
+            <VStack direction={direction} w={`${titleLocation}%`} pl={16} >
+              <Heading as={"h1"} p={2} alignItems={"center"}>


titleAlignment is passed directly to GridItem colSpan while SimpleGrid is hard-coded to columns={2}. To prevent invalid layouts, constrain/validate this prop (e.g., type it as 1 | 2, or clamp values) so it can’t exceed the grid’s column count.

Copilot · 2026-04-10T02:39:43Z


 import { IconName } from "../ui/icons/icon-registry";
 import { Icon } from "../ui/icons/icon";
-import { Card, Image as ChakraImage, Heading } from "@chakra-ui/react";
+import { Card, Image as ChakraImage, Heading, Text } from "@chakra-ui/react";
 import { Link as ChakraLink, SimpleGrid } from "@chakra-ui/react";


This import section has multiple unused imports now that the image/link portions of SupportCard are commented out (including Image/NextLink above, and some Chakra imports here). With the Next.js TypeScript ESLint preset, unused imports typically fail linting; please remove the unused imports or restore usage.

Copilot · 2026-04-10T02:39:43Z

 import SupportCard from "../../../components/services/support-card";
 import FaqItem from "../../../components/services/faq-item";
 import { services, supportOptions, faqItems } from "../../../data/services";
-import { Link as ChakraLink, SimpleGrid } from "@chakra-ui/react";
+import { Link as ChakraLink, HStack, SimpleGrid } from "@chakra-ui/react";


There are unused imports at the top of this file (e.g., Image and NextLink above) after the recent refactor. With the Next.js TypeScript ESLint preset, unused imports typically fail linting; please remove them or reintroduce usage.

BorDevTech and others added 19 commits March 27, 2026 06:06

Remove unused "Home" link from footer component

d197911

Merge pull request #86 from codingUnited/MOKSE-13-Remove-Home-link-in…

02d7f44

…-footer Remove unused "Home" link from footer component

Add Terms and Conditions page with structured content and templates

d192c22

-WARN this is a mockup, official terms needed.

Refactor Terms and Conditions page layout by commenting out HeaderTem…

fd9c4c9

…plate and adjusting SectionTemplate structure

Add Box component for spacing in Terms and Conditions page and suppre…

d0ff48a

…ss hydration warning in layout

Refactor Terms and Conditions page by removing HeaderTemplate and adj…

5bd845d

…usting layout structure

Update Terms and Conditions page styling for improved readability

9e55967

Rename export function from Page to Terms for clarity

6b0f4a0

Merge pull request #87 from codingUnited/MOKSE-9-Create-and-link-Term…

3f44ce6

…s-and-Conditions-page Add Terms and Conditions page with structured content and templates

Added club recognition to footer with link to club website

a5b8ddf

Merge pull request #88 from codingUnited/MOKSE-15-Feature-Coding-Unit…

7432cb6

…ed-in-footer Added club recognition to footer with link to club website

MOKSE-20 pulled changes into branch

98e287b

MOKSE-20 added donate link

f6a3b58

Merge pull request #90 from codingUnited/MOKSE-20-update-Donate-butto…

64275d3

…n-on-the-home Mokse 20 update donate button on the home

Merge pull request #91 from codingUnited/MOKSE-48-Footer-Component

391607f

Mokse 48 footer component

MOKSE-13

bea1ae9

Refactor HeaderTemplate and ServiceCard components for improved layout and consistency; update SupportCard interface to remove unused properties.

Merge branch 'MOKSE-24-Home-Page' of https://github.com/codingUnited/…

4aa718f

…MokseWebsite into MOKSE-24-Home-Page

Refactor Programs page structure for improved readability

8c57269

Copilot AI review requested due to automatic review settings April 10, 2026 02:25

BorDevTech merged commit 932f332 into master Apr 10, 2026
1 check passed

Copilot started reviewing on behalf of BorDevTech April 10, 2026 02:25 View session

BorDevTech deleted the MOKSE-24-Home-Page branch April 10, 2026 02:38

Copilot AI reviewed Apr 10, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Mokse 24 home page#96

Mokse 24 home page#96
BorDevTech merged 19 commits into
masterfrom
MOKSE-24-Home-Page

BorDevTech commented Apr 10, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		import { Card, Image as ChakraImage, Float, Heading } from "@chakra-ui/react";
		import { Link as ChakraLink, SimpleGrid } from "@chakra-ui/react";

		import { Card, Image as ChakraImage, Heading, Text } from "@chakra-ui/react";
		import { Link as ChakraLink, SimpleGrid } from "@chakra-ui/react";

Conversation

BorDevTech commented Apr 10, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants