Browse Source

feat(test detail): regroup sections, copy, and editor affordances

- AccSection: optional subtitle; disclosure body top padding
- Renames: О тесте, Вопросы, История (Версии + Прохождения), Показ в каталоге
- Move import into Вопросы as «Документ в вопросы»; remove duplicate import section
- Merge publication + assignment with subheads Видимость / Кому выдать; non-full-width actions
- Assign: «Выбрать всех (N)»; search row height tweaks
- Editor: крестик удаления варианта; единый футер + вариант/удалить вопрос; + вопрос в отдельной полосе
- AI block: test-detail-ai-panel (lighter, aligned with rest)
- History: empty state for runs; attempts error only in section (remove hero duplicate)

Made-with: Cursor
dev
Константин Лебединский 2 weeks ago
parent
commit
f1f5223076
  1. 334
      frontend/src/pages/TestDetail.jsx
  2. 195
      frontend/src/styles/cabinet-theme.css

334
frontend/src/pages/TestDetail.jsx

@ -34,13 +34,18 @@ function createEmptyQuestion() {
} }
/** /**
* @param {{ title: string, defaultOpen?: boolean, id?: string, children: import('react').ReactNode }} p * @param {{ title: string, subtitle?: string, defaultOpen?: boolean, id?: string, children: import('react').ReactNode }} p
*/ */
function AccSection({ title, defaultOpen, id, children }) { function AccSection({ title, subtitle, defaultOpen, id, children }) {
return ( return (
<div className="cabinet-brick"> <div className="cabinet-brick">
<details className="cabinet-disclosure" defaultOpen={defaultOpen} id={id || undefined}> <details className="cabinet-disclosure" defaultOpen={defaultOpen} id={id || undefined}>
<summary className="cabinet-disclosure__summary font-headline">{title}</summary> <summary className="cabinet-disclosure__summary">
<span className="cabinet-disclosure__summary-text">
<span className="cabinet-disclosure__summary-title font-headline">{title}</span>
{subtitle ? <span className="cabinet-disclosure__summary-sub">{subtitle}</span> : null}
</span>
</summary>
<div className="cabinet-disclosure__body">{children}</div> <div className="cabinet-disclosure__body">{children}</div>
</details> </details>
</div> </div>
@ -235,6 +240,16 @@ export default function TestDetail() {
}); });
} }
function selectAllVisible() {
setAssignSelected((prev) => {
const next = new Set(prev);
for (const p of assignPeople) {
next.add(assignPersonKey(p));
}
return next;
});
}
async function postAssign() { async function postAssign() {
const selectedRows = assignPeople.filter((p) => const selectedRows = assignPeople.filter((p) =>
assignSelected.has(assignPersonKey(p)) assignSelected.has(assignPersonKey(p))
@ -657,14 +672,13 @@ export default function TestDetail() {
При сохранении будет создана новая версия теста. При сохранении будет создана новая версия теста.
</div> </div>
)} )}
{data && attemptsErr && (
<p className="error-text" style={{ marginTop: '0.75rem', marginBottom: 0 }} role="alert">
{attemptsErr}
</p>
)}
</div> </div>
<AccSection title="Метаинформация" defaultOpen> <AccSection
title="О тесте"
subtitle="Название, описание и порог зачёта"
defaultOpen
>
<div className="draft-block"> <div className="draft-block">
<label className="form-label" htmlFor="draft-title"> <label className="form-label" htmlFor="draft-title">
Название Название
@ -702,17 +716,13 @@ export default function TestDetail() {
</div> </div>
</AccSection> </AccSection>
<AccSection title="Содержание" defaultOpen> <AccSection
<div className="draft-block"> title="Вопросы"
<div subtitle="Тексты, варианты и при необходимости загрузка из файла"
className="surface-card" defaultOpen
style={{
padding: '1rem 1.1rem',
marginBottom: '1.25rem',
borderLeft: '3px solid color-mix(in srgb, var(--primary) 45%, transparent)',
}}
aria-label="Параметры генерации теста с ИИ"
> >
<div className="draft-block">
<div className="test-detail-ai-panel" aria-label="Параметры генерации теста с ИИ">
<p <p
className="font-headline" className="font-headline"
style={{ fontSize: '0.95rem', marginTop: 0, marginBottom: '0.65rem' }} style={{ fontSize: '0.95rem', marginTop: 0, marginBottom: '0.65rem' }}
@ -792,12 +802,7 @@ export default function TestDetail() {
{draftQuestions.map((q, qi) => ( {draftQuestions.map((q, qi) => (
<div <div
key={q.key} key={q.key}
className="question-editor-block" className={`question-editor-block${qi === 0 ? ' question-editor-block--first' : ''}`}
style={{
borderTop: '1px solid var(--border-subtle, #2a2a2a)',
marginTop: '1rem',
paddingTop: '1rem',
}}
> >
<div className="question-editor-block__header"> <div className="question-editor-block__header">
<label <label
@ -848,11 +853,7 @@ export default function TestDetail() {
</p> </p>
<ul className="question-options-list" style={{ listStyle: 'none', padding: 0, margin: 0 }}> <ul className="question-options-list" style={{ listStyle: 'none', padding: 0, margin: 0 }}>
{q.options.map((o, oi) => ( {q.options.map((o, oi) => (
<li <li key={o.key} className="question-option-row">
key={o.key}
className="question-option-row"
style={{ display: 'flex', flexWrap: 'wrap', alignItems: 'center', gap: 8, marginBottom: 8 }}
>
<input <input
key={`${o.key}-${q.hasMultipleAnswers ? 'm' : 's'}`} key={`${o.key}-${q.hasMultipleAnswers ? 'm' : 's'}`}
className="question-option-row__mark" className="question-option-row__mark"
@ -886,8 +887,7 @@ export default function TestDetail() {
aria-label="Пометить как верный" aria-label="Пометить как верный"
/> />
<input <input
className="form-input" className="form-input question-option-row__text"
style={{ flex: '1 1 200px' }}
value={o.text} value={o.text}
onChange={(e) => { onChange={(e) => {
const v = e.target.value; const v = e.target.value;
@ -908,17 +908,25 @@ export default function TestDetail() {
{q.options.length > 1 && ( {q.options.length > 1 && (
<button <button
type="button" type="button"
className="btn btn-ghost btn--sm" className="question-option-remove"
onClick={() => removeOption(qi, oi)} onClick={() => removeOption(qi, oi)}
title="Удалить вариант"
aria-label="Удалить вариант"
> >
убрать <span className="material-symbols-outlined" aria-hidden>
close
</span>
</button> </button>
)} )}
</li> </li>
))} ))}
</ul> </ul>
<div className="inline-actions" style={{ marginTop: 4 }}> <div className="question-editor__footer">
<button type="button" className="btn btn-ghost btn--sm" onClick={() => addOption(qi)}> <button
type="button"
className="btn btn-ghost btn--sm"
onClick={() => addOption(qi)}
>
+ вариант + вариант
</button> </button>
{draftQuestions.length > 1 && ( {draftQuestions.length > 1 && (
@ -933,11 +941,92 @@ export default function TestDetail() {
</div> </div>
</div> </div>
))} ))}
<div className="inline-actions" style={{ marginTop: '0.75rem' }}> <div className="test-detail-add-question">
<button type="button" className="btn btn-ghost" onClick={addQuestion}> <button
type="button"
className="btn btn-ghost question-editor__add-question"
onClick={addQuestion}
>
+ вопрос + вопрос
</button> </button>
</div> </div>
<div className="test-detail-subsection test-detail-subsection--import">
<h3 className="test-detail-subsection__title">Документ в вопросы</h3>
<p className="muted test-detail-hint" style={{ marginTop: 0 }}>
PDF, Word или текст вставьте в черновик вопросов.
</p>
<div
className="import-file-row import-file-row--block"
style={{ marginBottom: importPreview || importErr ? '0.5rem' : 0, marginTop: 0 }}
>
<input
id="import-test-file"
className="import-file-input"
type="file"
accept=".pdf,.docx,.txt,.md,application/pdf"
onChange={onImportFile}
disabled={importBusy}
aria-label="Выбрать файл для импорта"
/>
<label
htmlFor="import-test-file"
className={`btn btn-ghost import-file-label${importBusy ? ' is-busy' : ''}`}
style={importBusy ? { pointerEvents: 'none' } : undefined}
>
{importBusy ? 'Обработка…' : 'Выбрать файл'}
</label>
</div>
{importErr && (
<p className="error-text" role="alert">
{importErr}
</p>
)}
{importPreview && (
<div className="draft-block" style={{ marginTop: '0.65rem', marginBottom: 0 }}>
<p className="muted" style={{ marginTop: 0, marginBottom: 0 }}>
{importPreview.generation?.message}
</p>
{importPreview.generation?.textPreview && !importPreview.generation?.available && (
<pre
className="form-input"
style={{
maxHeight: 180,
overflow: 'auto',
fontSize: 13,
marginTop: 8,
whiteSpace: 'pre-wrap',
}}
>
{importPreview.generation.textPreview}
{importPreview.textLength > 4000 ? '…' : ''}
</pre>
)}
{importPreview.generation?.draft && (
<div className="inline-actions" style={{ marginTop: 8, marginBottom: 0 }}>
<button
type="button"
className="btn btn-ghost"
onClick={applyGeneratedDraft}
>
Применить сгенерированный черновик
</button>
</div>
)}
{importPreview.extractedText && (
<div className="inline-actions" style={{ marginTop: 8, marginBottom: 0 }}>
<button
type="button"
className="btn btn-ghost"
onClick={applyExtractedToQuestion}
>
Вставить в первый вопрос (до 2000 симв.)
</button>
</div>
)}
</div>
)}
</div>
</div> </div>
</AccSection> </AccSection>
@ -959,7 +1048,13 @@ export default function TestDetail() {
</p> </p>
)} )}
</div> </div>
<AccSection title="История версий" defaultOpen={false}> <AccSection
title="История"
subtitle="Версии теста и кто проходил"
defaultOpen={false}
>
<div className="test-detail-subsection test-detail-subsection--tight">
<h3 className="test-detail-subsection__title">Версии</h3>
<ul className="version-card-list" aria-label="Список версий теста"> <ul className="version-card-list" aria-label="Список версий теста">
{versions.map((r) => ( {versions.map((r) => (
<li key={r.id} className="surface-card version-card-list__item"> <li key={r.id} className="surface-card version-card-list__item">
@ -995,15 +1090,20 @@ export default function TestDetail() {
</li> </li>
))} ))}
</ul> </ul>
</AccSection> </div>
{attemptsList !== undefined && (
{attemptsList != null && attemptsList.length > 0 && ( <div className="test-detail-subsection">
<AccSection title="Прогоны и разбор" defaultOpen={false}> <h3 className="test-detail-subsection__title">Прохождения</h3>
{attemptsErr && ( {attemptsList == null && (
<p className="error-text" role="alert"> <p
{attemptsErr} className={attemptsErr ? 'error-text' : 'text-muted'}
style={{ margin: 0 }}
role={attemptsErr ? 'alert' : undefined}
>
{attemptsErr || 'Список прогонов сейчас недоступен.'}
</p> </p>
)} )}
{attemptsList && attemptsList.length > 0 && (
<ul className="attempts-card-list" aria-label="Список прогонов"> <ul className="attempts-card-list" aria-label="Список прогонов">
{attemptsList.map((a) => { {attemptsList.map((a) => {
const when = a.completedAt const when = a.completedAt
@ -1024,10 +1124,16 @@ export default function TestDetail() {
<li key={a.id} className="surface-card attempts-card-list__item"> <li key={a.id} className="surface-card attempts-card-list__item">
<div className="attempts-card-list__row"> <div className="attempts-card-list__row">
<div className="attempts-card-list__main"> <div className="attempts-card-list__main">
<p className="muted attempts-card-list__when mono" style={{ margin: 0, fontSize: '0.8rem' }}> <p
className="muted attempts-card-list__when mono"
style={{ margin: 0, fontSize: '0.8rem' }}
>
{when} {when}
</p> </p>
<p className="attempts-card-list__who" style={{ margin: '0.35rem 0 0', fontWeight: 600 }}> <p
className="attempts-card-list__who"
style={{ margin: '0.35rem 0 0', fontWeight: 600 }}
>
{a.attempterName || '—'} {a.attempterName || '—'}
{a.attempterLogin && ( {a.attempterLogin && (
<span className="code-inline" style={{ fontSize: 11, marginLeft: 6 }}> <span className="code-inline" style={{ fontSize: 11, marginLeft: 6 }}>
@ -1053,14 +1159,31 @@ export default function TestDetail() {
); );
})} })}
</ul> </ul>
</AccSection>
)} )}
{attemptsList && attemptsList.length === 0 && (
<p className="text-muted" style={{ margin: 0 }}>
Пока нет зарегистрированных прогонов.
</p>
)}
</div>
)}
</AccSection>
<AccSection title="Публикация (видимость в списке)" defaultOpen={false}> <AccSection
<div title="Показ в каталоге"
className="inline-actions inline-actions--block-mobile" subtitle={
style={{ marginTop: '0.5rem' }} assignmentUi && data
? 'Видимость в списке и выдача сотрудникам'
: 'Показать или скрыть в общем списке'
}
defaultOpen={false}
> >
<div className="test-detail-subsection test-detail-subsection--tight">
<h3 className="test-detail-subsection__title">Видимость</h3>
<p className="test-detail-hint" style={{ marginTop: 0 }}>
Скрытые тесты в общем списке не показываются; ссылку на тест по-прежнему можно открыть.
</p>
<div className="publication-visibility__actions" style={{ marginTop: 4 }}>
{test?.chainActive !== false ? ( {test?.chainActive !== false ? (
<button <button
type="button" type="button"
@ -1081,82 +1204,15 @@ export default function TestDetail() {
</button> </button>
)} )}
</div> </div>
</AccSection>
<AccSection title="Импорт из файла" defaultOpen={false}>
<div className="import-file-row" style={{ marginBottom: '0.5rem' }}>
<input
id="import-test-file"
className="import-file-input"
type="file"
accept=".pdf,.docx,.txt,.md,application/pdf"
onChange={onImportFile}
disabled={importBusy}
aria-label="Выбрать файл для импорта"
/>
<label
htmlFor="import-test-file"
className={`btn btn-ghost import-file-label${importBusy ? ' is-busy' : ''}`}
style={importBusy ? { pointerEvents: 'none' } : undefined}
>
{importBusy ? 'Обработка…' : 'Выбрать файл'}
</label>
</div> </div>
{importErr && (
<p className="error-text" role="alert">
{importErr}
</p>
)}
{importPreview && (
<div className="draft-block" style={{ marginBottom: '1rem' }}>
<p className="muted" style={{ marginTop: 0 }}>
{importPreview.generation?.message}
</p>
{importPreview.generation?.textPreview && !importPreview.generation?.available && (
<pre
className="form-input"
style={{
maxHeight: 180,
overflow: 'auto',
fontSize: 13,
marginTop: 8,
whiteSpace: 'pre-wrap',
}}
>
{importPreview.generation.textPreview}
{importPreview.textLength > 4000 ? '…' : ''}
</pre>
)}
{importPreview.generation?.draft && (
<div className="inline-actions" style={{ marginTop: 8 }}>
<button
type="button"
className="btn btn-ghost"
onClick={applyGeneratedDraft}
>
Применить сгенерированный черновик
</button>
</div>
)}
{importPreview.extractedText && (
<div className="inline-actions" style={{ marginTop: 8 }}>
<button
type="button"
className="btn btn-ghost"
onClick={applyExtractedToQuestion}
>
Вставить в первый вопрос (до 2000 симв.)
</button>
</div>
)}
</div>
)}
</AccSection>
{assignmentUi && data && ( {assignmentUi && data && (
<AccSection title="Назначение сотрудникам" defaultOpen={false}> <div className="test-detail-subsection">
<h3 className="test-detail-subsection__title">Кому выдать</h3>
<p className="test-detail-hint" style={{ marginTop: 0 }}>
Список с учётом поиска и фильтров; можно отметить всех на экране.
</p>
{assignErr && ( {assignErr && (
<p className="error-text" role="alert"> <p className="error-text" role="alert" style={{ marginTop: 0, marginBottom: 8 }}>
{assignErr} {assignErr}
</p> </p>
)} )}
@ -1194,6 +1250,21 @@ export default function TestDetail() {
</select> </select>
{assignLoadBusy && <span className="text-muted">Загрузка</span>} {assignLoadBusy && <span className="text-muted">Загрузка</span>}
</div> </div>
{assignPeople.length > 0 && (
<div
className="publication-visibility__actions"
style={{ marginTop: '0.45rem', marginBottom: '0.25rem' }}
>
<button
type="button"
className="btn btn-ghost btn--sm"
onClick={selectAllVisible}
disabled={assignLoadBusy}
>
Выбрать всех ({assignPeople.length})
</button>
</div>
)}
{assignPeople.length > 0 ? ( {assignPeople.length > 0 ? (
<div className="assign-list" role="group" aria-label="Список сотрудников"> <div className="assign-list" role="group" aria-label="Список сотрудников">
{assignPeople.map((p) => { {assignPeople.map((p) => {
@ -1211,9 +1282,7 @@ export default function TestDetail() {
/> />
<span className="assign-row__text"> <span className="assign-row__text">
<span className="assign-row__fio">{p.fio}</span> <span className="assign-row__fio">{p.fio}</span>
{p.webLogin && ( {p.webLogin && <span className="assign-row__login">{p.webLogin}</span>}
<span className="assign-row__login">{p.webLogin}</span>
)}
<span className="assign-row__meta"> <span className="assign-row__meta">
{p.departments || '—'} {p.departments || '—'}
{p.clinicUserId {p.clinicUserId
@ -1232,7 +1301,7 @@ export default function TestDetail() {
</p> </p>
) )
)} )}
<div className="inline-actions" style={{ marginTop: '0.75rem' }}> <div className="publication-visibility__actions" style={{ marginTop: '0.75rem' }}>
<button <button
type="button" type="button"
className="btn btn-ghost" className="btn btn-ghost"
@ -1245,9 +1314,10 @@ export default function TestDetail() {
: ''} : ''}
</button> </button>
</div> </div>
{assignMsg && <p className="text-muted">{assignMsg}</p>} {assignMsg && <p className="text-muted" style={{ marginTop: 8, marginBottom: 0 }}>{assignMsg}</p>}
</AccSection> </div>
)} )}
</AccSection>
<div <div
className="test-detail-fixed-actions" className="test-detail-fixed-actions"

195
frontend/src/styles/cabinet-theme.css

@ -605,6 +605,29 @@ code,
min-height: 2.75rem; min-height: 2.75rem;
display: flex; display: flex;
align-items: center; align-items: center;
gap: 0.5rem;
}
.cabinet-disclosure__summary-text {
display: flex;
flex-direction: column;
align-items: flex-start;
gap: 0.15rem;
min-width: 0;
text-align: left;
}
.cabinet-disclosure__summary-title {
font-size: 1.05rem;
line-height: 1.25;
}
.cabinet-disclosure__summary-sub {
display: block;
font-size: 0.8rem;
font-weight: 400;
line-height: 1.3;
color: var(--secondary);
} }
.cabinet-disclosure__summary::-webkit-details-marker { .cabinet-disclosure__summary::-webkit-details-marker {
@ -626,7 +649,7 @@ code,
} }
.cabinet-disclosure__body { .cabinet-disclosure__body {
padding: 0 1rem 1.05rem; padding: 0.7rem 1rem 1.05rem;
border-top: 1px solid color-mix(in srgb, var(--outline-variant) 35%, transparent); border-top: 1px solid color-mix(in srgb, var(--outline-variant) 35%, transparent);
} }
@ -657,19 +680,160 @@ code,
.assign-toolbar__search { .assign-toolbar__search {
flex: 1 1 200px; flex: 1 1 200px;
min-height: 2.5rem;
max-height: 2.75rem;
padding-top: 6px;
padding-bottom: 6px;
line-height: 1.25;
box-sizing: border-box;
} }
/* Назначение: компактнее поля на мобилке; 16px — без авто-зума iOS в поле */ /* Назначение: селекты — как раньше; поиск — одна «строка», не «плитка» */
@media (max-width: 640px) { @media (max-width: 640px) {
.assign-toolbar { .assign-toolbar {
gap: 0.4rem; gap: 0.4rem;
} }
.assign-toolbar .form-input { .assign-toolbar .form-input {
padding: 8px 12px;
font-size: 16px; font-size: 16px;
line-height: 1.3; line-height: 1.3;
} }
.assign-toolbar__search {
font-size: 16px;
line-height: 1.3;
}
.assign-toolbar .form-input:not(.assign-toolbar__search) {
padding: 8px 12px;
}
}
/* Подсекции и подсказки (карточка теста) */
.test-detail-subsection {
margin-top: 1.25rem;
padding-top: 1.15rem;
border-top: 1px solid color-mix(in srgb, var(--outline-variant) 32%, transparent);
}
.test-detail-subsection:first-of-type,
.test-detail-subsection--tight {
margin-top: 0;
padding-top: 0;
border-top: none;
}
.test-detail-subsection--import {
margin-top: 1.35rem;
padding-top: 1.2rem;
border-top: 1px solid color-mix(in srgb, var(--outline-variant) 32%, transparent);
}
.test-detail-subsection__title {
margin: 0 0 0.35rem;
font-size: 0.95rem;
font-weight: 600;
font-family: inherit;
color: var(--on-surface);
letter-spacing: -0.01em;
}
.test-detail-hint {
margin: 0 0 0.6rem;
font-size: 0.8rem;
line-height: 1.4;
color: var(--secondary);
}
/* Панель «ИИ» — в тон остальным disclosure, без лишнего «модальности» */
.test-detail-ai-panel {
padding: 0.9rem 1rem;
margin-bottom: 1.15rem;
background: var(--surface-container-low);
border: 1px solid color-mix(in srgb, var(--outline-variant) 32%, transparent);
border-radius: 0.85rem;
box-shadow: none;
}
/* Кнопка публикации — не тянем на 100% ширины, если одна */
.publication-visibility__actions {
display: flex;
flex-wrap: wrap;
align-items: center;
gap: 0.5rem;
margin: 0;
}
.publication-visibility__actions .btn {
width: auto;
min-width: min(100%, 11rem);
max-width: 100%;
box-sizing: border-box;
}
/* Убрать вариант: иконка */
.question-option-remove {
display: inline-flex;
align-items: center;
justify-content: center;
width: 2.25rem;
min-width: 2.25rem;
height: 2.25rem;
margin: 0;
padding: 0;
border: none;
border-radius: 0.65rem;
background: transparent;
color: var(--secondary);
cursor: pointer;
transition: background 0.12s, color 0.12s;
}
.question-option-remove:hover {
background: color-mix(in srgb, var(--error) 8%, var(--surface));
color: var(--error, #b91c1c);
}
.question-option-remove .material-symbols-outlined {
font-size: 1.2rem;
line-height: 1;
}
/* Футер вопроса: + вариант, удалить вопрос */
.question-editor__footer {
display: flex;
flex-wrap: wrap;
align-items: center;
gap: 0.4rem 0.65rem;
margin-top: 0.4rem;
}
.question-editor__footer .btn--sm,
.question-editor__add-question {
font-size: 0.8rem;
}
/* Добавить вопрос — в конец блока «Вопросы» */
.test-detail-add-question {
display: flex;
justify-content: flex-start;
margin-top: 1.1rem;
padding-top: 0.85rem;
border-top: 1px solid color-mix(in srgb, var(--outline-variant) 28%, transparent);
}
/* Импорт: кнопка на всю ширину в колонку; ритм отступов */
@media (max-width: 640px) {
.import-file-row--block {
flex-direction: column;
align-items: stretch;
}
.import-file-label {
display: block;
width: 100%;
text-align: center;
}
} }
.assign-list { .assign-list {
@ -986,6 +1150,31 @@ code,
/* --- Спринт 2: редактор вопроса, прогоны, импорт, фикс. футер --- */ /* --- Спринт 2: редактор вопроса, прогоны, импорт, фикс. футер --- */
.question-editor-block {
border-top: 1px solid color-mix(in srgb, var(--outline-variant) 32%, transparent);
margin-top: 1.05rem;
padding-top: 1.05rem;
}
.question-editor-block--first {
border-top: none;
margin-top: 0;
padding-top: 0;
}
.question-option-row {
display: flex;
flex-wrap: nowrap;
align-items: center;
gap: 0.5rem;
margin-bottom: 0.5rem;
}
.question-option-row__text {
flex: 1 1 10rem;
min-width: 0;
}
.question-editor-block__header { .question-editor-block__header {
display: flex; display: flex;
flex-direction: column; flex-direction: column;

Loading…
Cancel
Save