Merge branch 'fix/qa-report-sweep' into 'main'

fix(admin-ui): QA report sweep — modal/table layout, i18n, sidebar polish See merge request 2-6/2-6-4/terravault/ordinis!239
2026-05-17 12:37:29 +00:00
parent dd84e57030 239cecd242
commit 37c9ae7040
5 changed files with 101 additions and 44 deletions
@@ -2,7 +2,6 @@ import { useState } from 'react'
 import { Link, useLocation } from '@tanstack/react-router'
 import { useTranslation } from 'react-i18next'
 import {
-  Home,
  Database,
  FileText,
  ClipboardList,
@@ -112,12 +111,14 @@ export function Sidebar() {
    webhooks.data?.filter((w) => w.active).length ?? 0

  // Flat list per redesign/compact.html — no section labels, /graph removed
-  // (now reachable via catalog toolbar "Граф ⇄" button). Order: Главная →
-  // Справочники → Мои черновики → На ревью → Outbox → Webhooks → Аудит → Поиск.
+  // (now reachable via catalog toolbar "Граф ⇄" button). QA bug #8: «Главная»
+  // удалён — он вёл на `/` который сразу redirect'нет на /dictionaries; в
+  // сайдбаре подсветка слетала (active state на /dictionaries, не на /).
+  // Logo сверху уже даёт home affordance — duplicate nav item только сбивал.
+  // Order: Справочники → Мои черновики → На ревью → Outbox → Webhooks → Аудит → Поиск.
  const sections: NavSection[] = [
    {
      items: [
-        { to: '/', label: t('nav.home'), icon: Home },
        {
          to: '/dictionaries',
          label: t('nav.dictionaries'),
@@ -484,11 +485,11 @@ export function MobileSidebar({ open, onClose }: { open: boolean; onClose: () =>
  const webhooksActive =
    webhooks.data?.filter((w) => w.active).length ?? 0

-  // Flat list per redesign — same as desktop Sidebar above.
+  // Flat list per redesign — same as desktop Sidebar above. QA bug #8:
+  // «Главная» удалён (redirect → /dictionaries сбивал active state).
  const sections: NavSection[] = [
    {
      items: [
-        { to: '/', label: t('nav.home'), icon: Home },
        {
          to: '/dictionaries',
          label: t('nav.dictionaries'),
@@ -352,8 +352,19 @@ i18n
          'dict.list.col.version': 'Версия',
          'dict.list.col.locales': 'Локали',
          'dict.list.col.records': 'Записи',
+          // Compact row table (catalog) — column headers per redesign/compact.html.
+          // QA bug #5: defaultValue inline в коде падал на ru-text при EN locale;
+          // выносим в i18n чтобы переключение языка реально влияло на заголовки.
+          'dict.col.title': 'Название',
+          'dict.col.id': 'id',
+          'dict.col.bundle': 'bundle',
+          'dict.col.records': 'записей',
+          'dict.col.updated': 'изменён',
+          'dict.col.outgoingFk': 'Ссылается (исходящие FK)',
+          'dict.col.incomingFk': 'Используют (входящие FK)',
          // List view (handoff D)
          'dict.list.bundle.all': 'все',
+          'dict.list.graph': 'Граф',
          'dict.list.deps.only': 'Только со связями',
          'dict.list.records.short': 'справочн.',
          'dict.list.fk.references': '→ ссылается',
@@ -485,7 +496,7 @@ i18n
          'schema.bundle': 'Bundle',
          'schema.bundleHint': 'Обычно cuod',
          'schema.template.label': 'Создать по шаблону',
-          'schema.template.hint': 'Скопирует схему, scope и locale выбранного словаря. Имя оставьте уникальное; версия сбросится на 1.0.0.',
+          'schema.template.hint': 'Скопирует схему, scope и локали выбранного словаря. Укажите уникальное имя — версия сбросится на 1.0.0.',
          'schema.template.placeholder': '— без шаблона —',
          'schema.version': 'Версия схемы',
          'schema.supportedLocales': 'Поддерживаемые локали',
@@ -1209,6 +1220,15 @@ i18n
          'dict.list.col.version': 'Version',
          'dict.list.col.locales': 'Locales',
          'dict.list.col.records': 'Records',
+          // Compact row table (catalog) — column headers EN.
+          'dict.col.title': 'Name',
+          'dict.col.id': 'id',
+          'dict.col.bundle': 'bundle',
+          'dict.col.records': 'records',
+          'dict.col.updated': 'updated',
+          'dict.col.outgoingFk': 'References (outgoing FK)',
+          'dict.col.incomingFk': 'Used by (incoming FK)',
+          'dict.list.graph': 'Graph',
          // List view (handoff D)
          'dict.list.bundle.all': 'all',
          'dict.list.deps.only': 'Only with links',
@@ -800,8 +800,14 @@ function DictionaryDetail() {
          Info panel — sticky слева с dict metadata + relations.
          На <lg viewport schedule переключается в stacked (info above content). */}
      {/* Grid items-stretch (default) — InfoPanel + main panel равной высоты.
-        * gap-3 — closer panels per user feedback ("ближе расположены"). */}
-      <div className="grid grid-cols-1 lg:grid-cols-[220px_1fr] gap-3 mt-2">
+        * gap-3 — closer panels per user feedback ("ближе расположены").
+        * QA bug #1: 1fr → minmax(0,1fr) — без minmax(0,…) grid track имеет
+        *   implicit min-content (auto), что позволяет records table вырасти
+        *   шире viewport. После закрытия Radix Dialog снимается body padding-
+        *   right (~15px), 1fr пересчитывается и ломает layout (исчезает
+        *   первая колонка под overflow-hidden card). minmax(0,1fr) фиксирует
+        *   нижнюю границу 0 → overflow-x-auto в Table primitive работает. */}
+      <div className="grid grid-cols-1 lg:grid-cols-[220px_minmax(0,1fr)] gap-3 mt-2">
        {detailQuery.data && (
          <EditorInfoPanel
            detail={detailQuery.data}
@@ -835,8 +841,12 @@ function DictionaryDetail() {

          {/* Main editor panel — tab bar + toolbar + tab content в bordered card
            * per redesign prototype. Visual containment отделяет editor от
-            * page chrome. */}
-          <div className="rounded-lg border border-line bg-surface overflow-hidden">
+            * page chrome.
+            * QA bug #1: overflow-hidden убран — клиппил records table при ширине
+            *   контента больше колонки. Table primitive сам имеет overflow-x-auto;
+            *   tabs ниже получают свой scroll контейнер. Card rounding продолжает
+            *   работать через border-radius на самом div'е. */}
+          <div className="rounded-lg border border-line bg-surface">

          {/* Editor tabs per handoff Stage 3.4 + redesign:
            * - counts в labels (Записи 127, Связи 6, Поля 12)
@@ -1189,7 +1199,7 @@ function DictionaryDetail() {
                      ) : null}
                    </div>
                  </TableCell>
-                  <TableCell hideBelow="md">
+                  <TableCell hideBelow="md" className="min-w-[180px] max-w-[320px] truncate">
                    {recordDisplayName(
                      r.data,
                      detailQuery.data?.defaultLocale ?? 'ru-RU',
@@ -1981,32 +1991,42 @@ function EditorTabBar({
    { id: 'events', label: t('editor.tab.events', { defaultValue: 'События' }) },
    { id: 'history', label: t('editor.tab.history', { defaultValue: 'История' }) },
  ]
+  // QA bug #3: на 1075px tabs (6 шт.) + actions (3 кнопки) переполняли flex
+  //   row → "+ Создать" обрезалось до "+ Соз". Решение: actions выносим в
+  //   отдельный shrink-0 контейнер, tabs кладём в overflow-x-auto wrapper
+  //   с min-w-0 → tabs скроллятся горизонтально если cramped, actions всегда
+  //   видны полностью. whitespace-nowrap на tab buttons чтобы текст не
+  //   переносился в кружок счётчика.
  return (
-    <div className="flex items-center gap-0 border-b border-line w-full">
-      {tabs.map((tab) => {
-        const isActive = active === tab.id
-        const count = counts?.[tab.id]
-        return (
-          <button
-            key={tab.id}
-            type="button"
-            onClick={() => onChange(tab.id)}
-            aria-pressed={isActive}
-            className={[
-              'px-3.5 h-10 text-body transition-colors -mb-px border-b-2 inline-flex items-center gap-1.5',
-              isActive
-                ? 'border-accent text-ink font-semibold'
-                : 'border-transparent text-ink-2 hover:text-ink',
-            ].join(' ')}
-          >
-            {tab.label}
-            {typeof count === 'number' && count > 0 && (
-              <span className="text-mono text-mute">{count}</span>
-            )}
-          </button>
-        )
-      })}
-      {actions && <div className="ml-auto flex items-center gap-2 pl-2">{actions}</div>}
+    <div className="flex items-center border-b border-line w-full">
+      <div className="flex items-center gap-0 overflow-x-auto flex-1 min-w-0 [scrollbar-width:none] [&::-webkit-scrollbar]:hidden">
+        {tabs.map((tab) => {
+          const isActive = active === tab.id
+          const count = counts?.[tab.id]
+          return (
+            <button
+              key={tab.id}
+              type="button"
+              onClick={() => onChange(tab.id)}
+              aria-pressed={isActive}
+              className={[
+                'px-3.5 h-10 text-body transition-colors -mb-px border-b-2 inline-flex items-center gap-1.5 shrink-0 whitespace-nowrap',
+                isActive
+                  ? 'border-accent text-ink font-semibold'
+                  : 'border-transparent text-ink-2 hover:text-ink',
+              ].join(' ')}
+            >
+              {tab.label}
+              {typeof count === 'number' && count > 0 && (
+                <span className="text-mono text-mute">{count}</span>
+              )}
+            </button>
+          )
+        })}
+      </div>
+      {actions && (
+        <div className="flex items-center gap-2 pl-2 pr-2 shrink-0">{actions}</div>
+      )}
    </div>
  )
 }
@@ -483,9 +483,12 @@ function DictionaryListTable({ rows }: { rows: DictionaryDefinition[] }) {
    sort.key === key ? (sort.dir === 'asc' ? ' ↑' : ' ↓') : ''

  return (
-    // No outer rounded/border wrapper — outer panel handles that (toolbar + table
-    // в одной card). Just table with white header per user feedback.
-    <table className="w-full bg-surface">
+    // QA bug #4: оборачиваем в overflow-x-auto wrapper чтобы при PUBLIC+INTERNAL
+    //   с extra columns table не клипал name'ы под parent overflow-hidden card.
+    //   table:min-w[760px] — нижняя граница ширины (sum min-content всех колонок),
+    //   ниже неё включается horizontal scroll.
+    <div className="w-full overflow-x-auto">
+    <table className="w-full min-w-[760px] bg-surface">
        <thead className="bg-surface border-b border-line">
          <tr>
            <SortableHeader onClick={() => toggleSort('title')} active={sort.key === 'title'}>
@@ -530,6 +533,7 @@ function DictionaryListTable({ rows }: { rows: DictionaryDefinition[] }) {
          ))}
        </tbody>
    </table>
+    </div>
  )
 }

@@ -623,8 +627,10 @@ function DictRow({ d, t }: { d: DictionaryDefinition; t: TFunc }) {
      onKeyDown={handleKey}
      className="relative border-b border-line-2 last:border-b-0 cursor-pointer bg-surface hover:bg-surface-2 focus-visible:outline-none focus-visible:bg-accent-bg/30 transition-colors"
    >
-      {/* name + subtitle (description short) + scope-color left stripe */}
-      <td className="relative px-3 py-2 align-top min-w-0">
+      {/* name + subtitle (description short) + scope-color left stripe.
+        * QA bug #4: min-w-[220px] — обеспечивает что name column не схлопывается
+        * до букв. При недостатке ширины table получает horizontal scroll. */}
+      <td className="relative px-3 py-2 align-top min-w-[220px]">
        {/* 3px scope stripe — absolutely positioned на левом краю row.
          * td имеет relative чтобы before работало в table context. */}
        <span
@@ -39,7 +39,12 @@ export const DialogOverlay = React.forwardRef<
    <DialogPrimitive.Overlay
      ref={ref}
      className={cn(
-        'fixed inset-0 z-50 bg-black/40 backdrop-blur-[1px]',
+        // QA bug #6: bg-black/40 + 1px blur был "почти невидимым" — backdrop
+        // не фокусировал внимание на modal'е. Bumped до /55 (заметно темнее
+        // при сохранении читабельности фона) + 2px blur. Visual reference:
+        // Radix docs/shadcn используют bg-black/80 по умолчанию, /55 —
+        // компромисс между нашим тёплым палитром и стандартом.
+        'fixed inset-0 z-50 bg-black/55 backdrop-blur-[2px]',
        'data-[state=open]:animate-in data-[state=closed]:animate-out',
        'data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0',
        className,
@@ -69,13 +74,18 @@ export const DialogContent = React.forwardRef<
  React.ElementRef<typeof DialogPrimitive.Content>,
  DialogContentProps
 >(function DialogContent({ className, children, size = 'md', hideClose, ...props }, ref) {
+  // duration-150 (вместо 200) — анимация закрытия укладывается в один frame
+  // на 60Hz без видимого "застывания" между state=closed и unmount. QA bug #2:
+  // "модал становится полупрозрачным, но остаётся отрисованным в течение
+  // секунд" — на slow CPU 200ms ощущается как несколько кадров; cutoff 150ms
+  // совпадает с tw-animate-css дефолтами и убирает рассинхрон визуала/DOM.
  return (
    <DialogPortal>
      <DialogOverlay />
      <DialogPrimitive.Content
        ref={ref}
        className={cn(
-          'fixed z-50 grid gap-4 border border-line bg-surface p-6 shadow-2xl duration-200',
+          'fixed z-50 grid gap-4 border border-line bg-surface p-6 shadow-2xl duration-150',
          // Centered modal — left/top 50% + translate.  В <880px mode пользуем
          // max-width 95vw чтобы modal не упирался в edges (вместо fullscreen
          // inset-0 который конфликтовал с left-1/2 → modal off-screen).