fix(254): move invoice modal buttons to sticky footer

fix(254): modal scrollable with sticky footer
Merge pull request 'fix: remove public COPY — no public dir in repo' (#24 ) from fix/remove-public-copy into main
2026-04-01 07:45:02 +00:00 · 2026-04-01 07:44:53 +00:00 · 2026-04-01 05:03:32 +00:00 · 2026-04-01 05:03:16 +00:00 · 2026-04-01 04:56:36 +00:00 · 2026-04-01 04:56:21 +00:00
12 changed files with 92 additions and 12 deletions
--- a/.dockerignore
+++ b/.dockerignore
@@ -0,0 +1,9 @@
 .next
 node_modules
 .git
 .env.local
 .env.*.local
 npm-debug.log*
 *.log
 test-results
 playwright-report
--- a/21
+++ b/21
@@ -0,0 +1,21 @@
 FROM node:22-alpine AS builder
 WORKDIR /app
 ENV NODE_ENV=development
 COPY package*.json ./
 RUN npm ci
 COPY . .
 ENV NEXT_TELEMETRY_DISABLED=1
 RUN npm run build
 FROM node:22-alpine AS runner
 WORKDIR /app
 ENV NODE_ENV=production
 ENV NEXT_TELEMETRY_DISABLED=1
 COPY --from=builder /app/package*.json ./
 COPY --from=builder /app/.next ./.next
 COPY --from=builder /app/node_modules ./node_modules
 EXPOSE 3000
 ENV PORT=3000
 CMD ["node_modules/.bin/next", "start"]
--- a/app/(app)/accounting/reports/page.tsx
+++ b/app/(app)/accounting/reports/page.tsx
@@ -1,4 +1,7 @@
 "use client";
 export const dynamic = "force-dynamic";
 import { useState } from "react";
 import { useQuery } from "@tanstack/react-query";
--- a/app/(app)/dashboard/page.tsx
+++ b/app/(app)/dashboard/page.tsx
@@ -1,4 +1,7 @@
 "use client";
 export const dynamic = "force-dynamic";
 import { useQuery, useMutation } from "@tanstack/react-query";
 import { useRouter } from "next/navigation";
--- a/app/(app)/invoices/page.tsx
+++ b/app/(app)/invoices/page.tsx
@@ -154,7 +154,20 @@ export default function InvoicesPage() {
      </Card>
      {/* Invoice Detail Modal */}
-      <Modal isOpen={!!selected} onClose={() => setSelected(null)} title={`Invoice ${selected?.invoiceNumber ?? ""}`} className="max-w-lg">
+      <Modal
        isOpen={!!selected}
        onClose={() => setSelected(null)}
        title={`Invoice ${selected?.invoiceNumber ?? ""}`}
        className="max-w-lg"
        footer={selected ? (
          <>
            {selected.status !== "VOID" && selected.status !== "PAID" && (
              <Button variant="danger" size="sm" onClick={() => voidInvoice.mutate(selected.id)} isLoading={voidInvoice.isPending}>Void Invoice</Button>
            )}
            <Button variant="outline" size="sm" onClick={() => setSelected(null)}>Close</Button>
          </>
        ) : undefined}
      >
        {selected && (
          <div className="space-y-4">
            <div className="bg-gray-50 rounded-lg p-4 space-y-2 text-sm">
@@ -196,13 +209,6 @@ export default function InvoicesPage() {
                  disabled={!payForm.amount}>Record Payment</Button>
              </div>
            )}
            <div className="flex justify-between pt-1">
              {selected.status !== "VOID" && selected.status !== "PAID" && (
                <Button variant="danger" size="sm" onClick={() => voidInvoice.mutate(selected.id)} isLoading={voidInvoice.isPending}>Void Invoice</Button>
              )}
              <Button variant="outline" size="sm" onClick={() => setSelected(null)} className="ml-auto">Close</Button>
            </div>
          </div>
        )}
      </Modal>
--- a/app/(app)/layout.tsx
+++ b/app/(app)/layout.tsx
@@ -1,4 +1,5 @@
 'use client';
 export const dynamic = 'force-dynamic';
 import { useEffect, useState } from 'react';
 import { useRouter } from 'next/navigation';
--- a/app/(app)/reports/page.tsx
+++ b/app/(app)/reports/page.tsx
@@ -1,4 +1,7 @@
 "use client";
 export const dynamic = "force-dynamic";
 import { useState } from "react";
 import { useQuery } from "@tanstack/react-query";
--- a/app/layout.tsx
+++ b/app/layout.tsx
@@ -6,6 +6,8 @@ import Providers from '@/components/providers';
 // Fonts are loaded via CSS @import in globals.css (runtime CDN load).
 // CSS vars are set directly in globals.css :root — no JS injection needed.
 export const dynamic = 'force-dynamic';
 export const metadata: Metadata = {
  title: 'FiberOps Admin',
  description: 'ISP Management Platform',
--- a/pages/_document.tsx
+++ b/pages/_document.tsx
@@ -0,0 +1,12 @@
 import { Html, Head, Main, NextScript } from 'next/document';
 export default function Document() {
  return (
    <Html lang="en">
      <Head />
      <body>
        <Main />
        <NextScript />
      </body>
    </Html>
  );
 }
--- a/pages/_error.tsx
+++ b/pages/_error.tsx
@@ -0,0 +1,14 @@
 // Custom error page — prevents Html import issue in Next.js pages router
 export default function Error({ statusCode }: { statusCode?: number }) {
  return (
    <div style={{ padding: '2rem', fontFamily: 'sans-serif' }}>
      <h1>{statusCode || 'Error'}</h1>
      <p>{statusCode === 404 ? 'Page not found' : 'An error occurred'}</p>
    </div>
  );
 }
 Error.getInitialProps = ({ res, err }: any) => {
  const statusCode = res ? res.statusCode : err ? err.statusCode : 404;
  return { statusCode };
 };
--- a/public/.gitkeep
+++ b/public/.gitkeep
--- a/src/components/ui/Modal.tsx
+++ b/src/components/ui/Modal.tsx
@@ -9,10 +9,11 @@ interface ModalProps {
  onClose: () => void;
  title: string;
  children: React.ReactNode;
  footer?: React.ReactNode;
  className?: string;
 }
-export function Modal({ isOpen, onClose, title, children, className }: ModalProps) {
+export function Modal({ isOpen, onClose, title, children, footer, className }: ModalProps) {
  const overlayRef = useRef<HTMLDivElement>(null);
  useEffect(() => {
@@ -31,8 +32,8 @@ export function Modal({ isOpen, onClose, title, children, className }: ModalProp
      className="fixed inset-0 z-50 flex items-center justify-center bg-black/40 p-4"
      onClick={(e) => e.target === overlayRef.current && onClose()}
    >
-      <div className={cn("w-full max-w-lg rounded-xl bg-white shadow-xl", className)}>
+      <div className={cn("w-full max-w-lg rounded-xl bg-white shadow-xl flex flex-col max-h-[90vh]", className)}>
-        <div className="flex items-center justify-between border-b border-gray-100 px-6 py-4">
+        <div className="flex items-center justify-between border-b border-gray-100 px-6 py-4 flex-shrink-0">
          <h3 className="text-base font-semibold text-gray-900">{title}</h3>
          <button
            onClick={onClose}
@@ -41,7 +42,12 @@ export function Modal({ isOpen, onClose, title, children, className }: ModalProp
            <X className="h-5 w-5" />
          </button>
        </div>
-        <div className="px-6 py-4">{children}</div>
+        <div className="px-6 py-4 overflow-y-auto flex-1">{children}</div>
        {footer && (
          <div className="flex items-center justify-end gap-2 border-t border-gray-100 px-6 py-3 flex-shrink-0">
            {footer}
          </div>
        )}
      </div>
    </div>
  );
Author	SHA1	Message	Date
kibin	c5826a34d0	fix(254): move invoice modal buttons to sticky footer	2026-04-01 07:45:02 +00:00
kibin	7de6d899f1	fix(254): modal scrollable with sticky footer	2026-04-01 07:44:53 +00:00
kibin	e582eb1693	Merge pull request 'fix: remove public COPY — no public dir in repo' (#24 ) from fix/remove-public-copy into main	2026-04-01 05:03:32 +00:00
Nemo (Claude Code)	41a90ad76e	fix: remove public COPY from Dockerfile, add public/.gitkeep	2026-04-01 05:03:16 +00:00
kibin	b880137084	Merge pull request 'fix: pages/_error and _document for build' (#23 ) from fix/pages-error-handler into main	2026-04-01 04:56:36 +00:00
Nemo (Claude Code)	fca3194801	fix: add pages/_error and _document to fix Html import in build	2026-04-01 04:56:21 +00:00
kibin	22c1df67c1	Merge pull request 'fix: force-dynamic on layouts — skip SSR static gen' (#22 ) from fix/force-dynamic-all-pages into main	2026-04-01 04:49:51 +00:00
Nemo (Claude Code)	ef6b6a3ad4	fix: force-dynamic on app layout to skip SSR static gen for all pages	2026-04-01 04:49:20 +00:00
kibin	8156c1f207	Merge pull request 'fix: force-dynamic on recharts pages (FIBEROPS-249)' (#21 ) from fix/force-dynamic-recharts-pages into main	2026-04-01 04:39:57 +00:00
Nemo (Claude Code)	8a31ca0199	fix: force-dynamic on recharts pages to prevent SSR static export failure	2026-04-01 04:39:54 +00:00
kibin	d58b6bfd0b	Merge pull request 'fix: remove standalone output — run next start in Docker' (#20 ) from fix/no-standalone-output into main	2026-04-01 04:18:47 +00:00
Nemo (Claude Code)	e8b91468a1	fix: remove standalone output, run next start in Docker	2026-04-01 04:18:31 +00:00
kibin	ac60822134	Merge pull request 'fix: .dockerignore to exclude .next cache' (#19 ) from fix/dockerignore-next-cache into main	2026-04-01 04:09:30 +00:00
Nemo (Claude Code)	87e9fac4c1	fix: add .dockerignore to exclude stale .next cache from Docker build	2026-04-01 04:09:26 +00:00
kibin	c061e821c9	Merge pull request 'fix: Dockerfile dev deps for tailwindcss build' (#18 ) from fix/dockerfile-devdeps into main	2026-04-01 03:59:58 +00:00
Nemo (Claude Code)	63cce69634	fix: install devDeps in builder stage for tailwind/typescript (FIBEROPS-249)	2026-04-01 03:59:41 +00:00
kibin	ff73898dac	Merge pull request 'fix: add Dockerfile + standalone output (FIBEROPS-249)' (#17 ) from fix/FIBEROPS-249-add-dockerfile into main	2026-04-01 03:44:28 +00:00
Nemo (Claude Code)	0715e66a65	fix: add Dockerfile + standalone output for Coolify deploy (FIBEROPS-249)	2026-04-01 03:44:06 +00:00
kibin	205f0091dc	Merge pull request 'chore: remove portal from admin app (FIBEROPS-249)' (#16 ) from chore/FIBEROPS-249-remove-portal-from-web into main	2026-04-01 03:30:54 +00:00