-
Notifications
You must be signed in to change notification settings - Fork 1
Mokse 59 dev #98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mokse 59 dev #98
Changes from all commits
f1e025f
acfa99f
66b8cd6
841dc4c
90fc1a1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,6 +13,7 @@ | |
| # testing | ||
| /coverage | ||
| /.sec/ | ||
| *.log* | ||
|
|
||
| # next.js | ||
| /.next/ | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| {"servers": { | ||
| "chakra-ui": { | ||
| "command": "npx", | ||
| "args": ["-y", "@chakra-ui/react-mcp"] | ||
| } | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,5 @@ | ||
| { | ||
| "servers": { | ||
| "chakra-ui": { | ||
| "command": "npx", | ||
| "args": ["-y", "@chakra-ui/react-mcp"] | ||
| } | ||
| }, | ||
| "typescript.tsdk": "node_modules/typescript/lib" | ||
| "typescript.tsdk": "node_modules/typescript/lib", // i get this error: This setting is deprecated. Use js/ts.tsdk.path instead. | ||
| "eslint.validate": ["javascript", "javascriptreact", "typescript", "typescriptreact"] | ||
|
|
||
| } | ||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -14,15 +14,18 @@ import { | |||||||||
| Link as ChakraLink, | ||||||||||
| Image as ChakraImage, | ||||||||||
| Avatar, | ||||||||||
| VStack, | ||||||||||
| } from "@chakra-ui/react"; | ||||||||||
|
|
||||||||||
| import checkDeviceSize from "@/components/ui/breakpoints"; | ||||||||||
| import { | ||||||||||
| HeaderTemplate, | ||||||||||
| PageBuilder, | ||||||||||
| SectionTemplate, | ||||||||||
| } from "@/components/page-builder/template"; | ||||||||||
| import { Icon } from "@/components/ui/icons/icon"; | ||||||||||
| import { poppins } from "@/components/ui/fonts"; | ||||||||||
| import { Section } from "lucide-react"; | ||||||||||
|
||||||||||
| import { Section } from "lucide-react"; |
Copilot
AI
Apr 21, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SectionTemplate always renders a section heading/description container even when title/description are undefined (it will output an empty <h2>). Since this usage doesn’t pass title/description, it may introduce extra blank vertical space / empty heading semantics on the page. Consider passing a title (and moving the existing “Get in Touch” heading into it) or using a simpler wrapper component here.
Copilot
AI
Apr 21, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
w={"xxs"} / h={"xxs"} aren’t standard Chakra size tokens (common ones are xs, sm, etc., or numeric values). If the token isn’t defined, these props will resolve to an invalid CSS value and the cards may render with unexpected sizing. Consider using a supported token (e.g., xs) or an explicit size value.
| w={"xxs"} | |
| h={"xxs"} | |
| w={"xs"} | |
| h={"xs"} |
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -174,35 +174,20 @@ export default function StopTheStigma() { | |||||||||||||||||
| </Center> | ||||||||||||||||||
| </Flex> | ||||||||||||||||||
| </Container> | ||||||||||||||||||
| </SectionTemplate> | ||||||||||||||||||
| {/* </SectionTemplate> | ||||||||||||||||||
|
|
||||||||||||||||||
| <SectionTemplate> | ||||||||||||||||||
| <Container | ||||||||||||||||||
| fluid | ||||||||||||||||||
| // position="relative" | ||||||||||||||||||
| // w={"100%"} | ||||||||||||||||||
| // h={"0vh"} | ||||||||||||||||||
| // bgImage={`url(${"/assets/stop-the-stigma/stop-stigma-sect1_background.webp"})`} | ||||||||||||||||||
| // bgSize={"cover"} | ||||||||||||||||||
| // backgroundPosition={"center"} | ||||||||||||||||||
| // bgRepeat={"no-repeat"} | ||||||||||||||||||
| // aria-label={`${"Stop The Stigma Conference"} Image`} | ||||||||||||||||||
| > | ||||||||||||||||||
| {/* <AbsoluteCenter | ||||||||||||||||||
| textAlign="center" | ||||||||||||||||||
| bg="rgba(0, 0, 0, 0.4)" | ||||||||||||||||||
| w={"100%"} | ||||||||||||||||||
| h={"100%"} | ||||||||||||||||||
| alignItems={"center"} | ||||||||||||||||||
| > */} | ||||||||||||||||||
| <Flex gapX={24} p={20}> | ||||||||||||||||||
| <Heading as={"h2"} size={'5xl'}> | ||||||||||||||||||
| Conference Highlights | ||||||||||||||||||
| </Heading> | ||||||||||||||||||
| <SectionTemplate> */} | ||||||||||||||||||
| <Container fluid height={"100"} h={"100"} bgColor={"midnightblue"} color={"white"} > | ||||||||||||||||||
|
Comment on lines
+177
to
+180
|
||||||||||||||||||
| <Container fluid height={"100"} h={"100"} bgColor={"midnightblue"} color={"white"} > | |
| <Container fluid h={"100%"} bgColor={"midnightblue"} color={"white"} > |
Copilot
AI
Apr 21, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The highlights grid is now hard-coded to columns={2} with w="40vw". This removes the previous responsive behavior and will likely overflow or become unreadable on smaller viewports. Consider restoring responsive props (e.g., columns={{ base: 1, md: 2 }} and a responsive width) so the section remains usable on mobile/tablet.
| <SimpleGrid columns={2} gap="75px" mb="20" w="40vw" textStyle={"lg"} > | |
| <SimpleGrid | |
| columns={{ base: 1, md: 2 }} | |
| gap={{ base: "6", md: "10", lg: "75px" }} | |
| mb="20" | |
| w={{ base: "full", lg: "40vw" }} | |
| textStyle={"lg"} | |
| > |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This keeps the deprecated
typescript.tsdksetting (and an inline comment). Since the comment indicates the setting is deprecated, it’d be better to switch to the recommended key (js/ts.tsdk.path, per the warning) and remove the inline note so the file stays clean and future-proof.