-
-
Notifications
You must be signed in to change notification settings - Fork 250
Fix homepage container flexibility #2100
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
Changes from 16 commits
5e24a6d
caea551
7e7c88e
026ee9a
305d246
8a73963
8917f7c
3789e3d
5e0536e
f565947
37fa65b
4b93f3e
adb2e32
64ffd22
058cdd7
41d79c1
fdd40ea
3419a80
2e76459
891dd58
2cd43f6
d87feba
03f7dcc
b952945
ec0d606
23facd8
558aa6f
48f79f1
5892607
7352a18
df7aff0
c425484
6ee3cd5
dc5ed1d
690fe21
f4d49ea
10082d5
f17b2e7
e483c63
e625c12
8b5e522
1117ef3
2f37ee0
0bdde86
ba63156
c5e0993
b083a8a
4bbc988
b0627fe
a735bd5
2f6d0e7
3fcd9f9
dc248c0
9641262
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 | ||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -3,6 +3,7 @@ | |||||||||||||||||||||||||||||||||||||||
| "version": "0.1.0", | ||||||||||||||||||||||||||||||||||||||||
| "private": true, | ||||||||||||||||||||||||||||||||||||||||
| "scripts": { | ||||||||||||||||||||||||||||||||||||||||
| "build:tailwind": "ts-node --project tsconfig.tsnode.json scripts/generate-tailwind-config.ts", | ||||||||||||||||||||||||||||||||||||||||
| "build": "next build", | ||||||||||||||||||||||||||||||||||||||||
| "build:turbo": "next build --turbo", | ||||||||||||||||||||||||||||||||||||||||
| "dev": "next dev --port 3000", | ||||||||||||||||||||||||||||||||||||||||
|
|
@@ -11,11 +12,12 @@ | |||||||||||||||||||||||||||||||||||||||
| "lint": "eslint . --config eslint.config.mjs --fix --max-warnings=0", | ||||||||||||||||||||||||||||||||||||||||
| "lint:check": "eslint . --config eslint.config.mjs --max-warnings=0", | ||||||||||||||||||||||||||||||||||||||||
| "start": "next start", | ||||||||||||||||||||||||||||||||||||||||
| "test:e2e": "npx playwright test", | ||||||||||||||||||||||||||||||||||||||||
| "test:unit": "tsc --noEmit && NODE_OPTIONS='--experimental-vm-modules --no-warnings=DEP0040' jest" | ||||||||||||||||||||||||||||||||||||||||
| "test": "pnpm test:unit", | ||||||||||||||||||||||||||||||||||||||||
| "test:unit": "tsc --noEmit && cross-env NODE_OPTIONS=--experimental-vm-modules --no-warnings=DEP0040 jest", | ||||||||||||||||||||||||||||||||||||||||
| "test:e2e": "playwright test", | ||||||||||||||||||||||||||||||||||||||||
| "test:all": "pnpm test:unit && pnpm test:e2e" | ||||||||||||||||||||||||||||||||||||||||
coderabbitai[bot] marked this conversation as resolved.
Show resolved
Hide resolved
Comment on lines
+17
to
+20
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. cross-env is used in scripts but not declared — tests will fail.
Add "devDependencies": {
+ "cross-env": "^7.0.3",📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||
| }, | ||||||||||||||||||||||||||||||||||||||||
| "dependencies": { | ||||||||||||||||||||||||||||||||||||||||
| "@apollo/client": "^3.13.9", | ||||||||||||||||||||||||||||||||||||||||
| "@fortawesome/fontawesome-svg-core": "^6.7.2", | ||||||||||||||||||||||||||||||||||||||||
| "@fortawesome/free-brands-svg-icons": "^6.7.2", | ||||||||||||||||||||||||||||||||||||||||
| "@fortawesome/free-regular-svg-icons": "^6.7.2", | ||||||||||||||||||||||||||||||||||||||||
|
|
@@ -45,7 +47,6 @@ | |||||||||||||||||||||||||||||||||||||||
| "dompurify": "^3.2.6", | ||||||||||||||||||||||||||||||||||||||||
| "eslint-plugin-import": "^2.32.0", | ||||||||||||||||||||||||||||||||||||||||
| "framer-motion": "^12.23.12", | ||||||||||||||||||||||||||||||||||||||||
| "graphql": "^16.11.0", | ||||||||||||||||||||||||||||||||||||||||
| "leaflet": "^1.9.4", | ||||||||||||||||||||||||||||||||||||||||
| "leaflet.markercluster": "^1.5.3", | ||||||||||||||||||||||||||||||||||||||||
| "lodash": "^4.17.21", | ||||||||||||||||||||||||||||||||||||||||
|
|
@@ -58,9 +59,12 @@ | |||||||||||||||||||||||||||||||||||||||
| "react-apexcharts": "^1.7.0", | ||||||||||||||||||||||||||||||||||||||||
| "react-dom": "^19.1.1", | ||||||||||||||||||||||||||||||||||||||||
| "react-icons": "^5.5.0", | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| "react-router-dom": "^7.8.1", | ||||||||||||||||||||||||||||||||||||||||
| "tailwind-merge": "^3.3.1", | ||||||||||||||||||||||||||||||||||||||||
| "tailwindcss-animate": "^1.0.7" | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| }, | ||||||||||||||||||||||||||||||||||||||||
| "devDependencies": { | ||||||||||||||||||||||||||||||||||||||||
| "@axe-core/react": "^4.10.2", | ||||||||||||||||||||||||||||||||||||||||
|
|
@@ -70,7 +74,11 @@ | |||||||||||||||||||||||||||||||||||||||
| "@swc/core": "^1.13.3", | ||||||||||||||||||||||||||||||||||||||||
| "@swc/jest": "^0.2.39", | ||||||||||||||||||||||||||||||||||||||||
| "@tailwindcss/postcss": "^4.1.12", | ||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
79
to
80
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainPick a single Jest transformer. Both @swc/jest and ts-jest are present; using both can conflict. Prefer @swc/jest for speed, or keep ts-jest — but not both in config. Run to see what your config uses: If only one is referenced, remove the unused package. Also applies to: 117-117 🏁 Script executed: #!/bin/bash
rg -nP -C2 '(jest\.config|setupFiles|setupFilesAfterEnv|jest":\s*{)'
rg -nP -C3 '"transform"\s*:\s*{[^}]*}'
rg -nP -C2 '(ts-jest|@swc/jest)'Length of output: 5414 Remove unused ts-jest dependency transform in frontend/jest.config.ts only references @swc/jest, so ts-jest in package.json (line 117) isn’t used and should be removed. 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| "@testing-library/jest-dom": "^6.7.0", | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| "@testing-library/react": "^16.3.0", | ||||||||||||||||||||||||||||||||||||||||
| "@types/jest": "^29.5.14", | ||||||||||||||||||||||||||||||||||||||||
| "@types/leaflet": "^1.9.20", | ||||||||||||||||||||||||||||||||||||||||
|
|
@@ -83,6 +91,7 @@ | |||||||||||||||||||||||||||||||||||||||
| "@typescript-eslint/eslint-plugin": "^8.40.0", | ||||||||||||||||||||||||||||||||||||||||
| "@typescript-eslint/parser": "^8.40.0", | ||||||||||||||||||||||||||||||||||||||||
| "autoprefixer": "^10.4.21", | ||||||||||||||||||||||||||||||||||||||||
| "cross-env": "^10.0.0", | ||||||||||||||||||||||||||||||||||||||||
| "eslint": "^9.33.0", | ||||||||||||||||||||||||||||||||||||||||
| "eslint-config-next": "^15.4.7", | ||||||||||||||||||||||||||||||||||||||||
| "eslint-config-prettier": "^10.1.8", | ||||||||||||||||||||||||||||||||||||||||
|
|
@@ -103,7 +112,8 @@ | |||||||||||||||||||||||||||||||||||||||
| "prettier": "^3.6.2", | ||||||||||||||||||||||||||||||||||||||||
| "prettier-plugin-tailwindcss": "^0.6.14", | ||||||||||||||||||||||||||||||||||||||||
| "require-in-the-middle": "^7.5.2", | ||||||||||||||||||||||||||||||||||||||||
| "tailwindcss": "^3.4.17", | ||||||||||||||||||||||||||||||||||||||||
| "tailwindcss": "^4.1.12", | ||||||||||||||||||||||||||||||||||||||||
| "tailwindcss-animate": "^1.0.7", | ||||||||||||||||||||||||||||||||||||||||
| "ts-jest": "^29.4.1", | ||||||||||||||||||||||||||||||||||||||||
coderabbitai[bot] marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||||
| "ts-node": "^10.9.2", | ||||||||||||||||||||||||||||||||||||||||
| "typescript": "~5.8.3", | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| ignoredBuiltDependencies: | ||
| - '@heroui/shared-utils' | ||
| - '@sentry/cli' | ||
| - core-js | ||
| - unrs-resolver | ||
| onlyBuiltDependencies: | ||
| - '@swc/core' | ||
| - '@tailwindcss/oxide' | ||
| - sharp | ||
OUMIMANDAL marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| module.exports = { | ||
| plugins: { | ||
| "@tailwindcss/postcss": {}, | ||
| autoprefixer: {}, | ||
| tailwindcss: {}, | ||
| }, | ||
| } |
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,22 @@ | ||||||||
|
|
||||||||
| // scripts/generate-tailwind-config.ts | ||||||||
| import fs from 'fs'; | ||||||||
| import path from 'path'; | ||||||||
| import tailwindConfig from '../tailwind.config.js'; // ← now .js | ||||||||
|
||||||||
| import tailwindConfig from '../tailwind.config.js'; // ← now .js | |
| // Import the TS config directly; ts-node project should enable allowImportingTsExtensions | |
| import tailwindConfig from '../tailwind.config.ts'; |
🤖 Prompt for AI Agents
In frontend/scripts/generate-tailwind-config.ts around line 5, the script
imports ../tailwind.config.js but that file contains TypeScript-only syntax and
ESM export which will throw under ts-node/CommonJS; update the import to point
to the TypeScript source (../tailwind.config.ts) or otherwise load the config
via a TS-aware loader (e.g., register ts-node or compile the config to JS) so
the runtime can parse the TS/ESM syntax — ensure the import path, runtime
loader, and any build scripts are adjusted consistently.
OUMIMANDAL marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,3 @@ | ||
| import { Metadata } from 'next' | ||
| import React from 'react' | ||
| import { getStaticMetadata } from 'utils/metaconfig' | ||
|
|
||
| export const metadata: Metadata = getStaticMetadata('about', '/about') | ||
|
|
||
| export default function AboutLayout({ children }: { children: React.ReactNode }) { | ||
| return children | ||
| } | ||
| return children; | ||
| } |
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.
💡 Verification agent
🧩 Analysis chain
Verify referenced generator files exist.
Ensure tsconfig.tsnode.json and scripts/generate-tailwind-config.ts are present.
🏁 Script executed:
Length of output: 131
Add or fix generate-tailwind-config.ts path The file
frontend/scripts/generate-tailwind-config.tsis missing; ensure it’s added or update the"build:tailwind"command to point to the correct script path.🤖 Prompt for AI Agents