RSHB/arm-core
9
0
Fork 0
Code Issues 26 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

aurora-1868, исправление расхождения с макетом #210

Merged
aep merged 7 commits from fix/consolidation into main 2026-10-30 12:35:46 +03:00
Conversation 3 Commits 7 Files Changed 28 +144 -77
grog commented 2026-10-30 10:33:46 +03:00
Owner
Copy Link
  • исправлены стили вкладок в отчетности
  • убраны технические значения из фильтров
  • модальные окна расположены по середине страницы
  • добавлен фон для предупреждающей иконки
  • добавлен разделитель и обновлен стиль крестика в модальных окнах
  • поправлена опечатка в "Бух. баланс"
  • добавлены скругления в таблицы консолидации и организации
- исправлены стили вкладок в отчетности - убраны технические значения из фильтров - модальные окна расположены по середине страницы - добавлен фон для предупреждающей иконки - добавлен разделитель и обновлен стиль крестика в модальных окнах - поправлена опечатка в "Бух. баланс" - добавлены скругления в таблицы консолидации и организации -
grog added 2 commits 2026-10-30 10:33:48 +03:00
grog requested review from aep 2026-10-30 10:33:52 +03:00
grog requested review from maxim 2026-10-30 10:33:56 +03:00
grog changed title from aurora-1589, исправление расхождения с макетом to aurora-1868, исправление расхождения с макетом 2026-10-30 10:34:45 +03:00
grog added 1 commit 2026-10-30 10:36:07 +03:00
grog added 1 commit 2026-10-30 10:43:21 +03:00
aep commented 2026-10-30 11:15:08 +03:00
Owner
Copy Link

Голов, просмотрел #210 — есть что поправить до мержа:

  1. `DataTable.tsx:58` — `!seen.has(v) && v` режет 0 как falsy. Для `legal_entities_count`/`members_count` (multiselect) опция «0» никогда не появится в фильтре, хотя такие строки есть.
  2. `App.css:13` — `padding: 0 16 16 16 !important` без `px`, это невалидный CSS, браузер выкидывает всё правило. Футер модалки не получит нужный отступ.
  3. `CreateEntityPickerModal.tsx:56` — убрали inline `marginBottom:0` у заголовка, а новый глобальный стиль в `App.css` margin не трогает — вернётся зазор, которого раньше не было.

Плюс по мелочи: в `OsvTabs.tsx:206` `centered`/`closeIcon` продублированы в 17 местах вместо общего modal-wrapper — не блокер, но на будущее лучше вынести.

Поправь первые три и пни обратно.

Голов, просмотрел #210 — есть что поправить до мержа: 1. \`DataTable.tsx:58\` — \`!seen.has(v) && v\` режет 0 как falsy. Для \`legal_entities_count\`/\`members_count\` (multiselect) опция «0» никогда не появится в фильтре, хотя такие строки есть. 2. \`App.css:13\` — \`padding: 0 16 16 16 !important\` без \`px\`, это невалидный CSS, браузер выкидывает всё правило. Футер модалки не получит нужный отступ. 3. \`CreateEntityPickerModal.tsx:56\` — убрали inline \`marginBottom:0\` у заголовка, а новый глобальный стиль в \`App.css\` margin не трогает — вернётся зазор, которого раньше не было. Плюс по мелочи: в \`OsvTabs.tsx:206\` \`centered\`/\`closeIcon\` продублированы в 17 местах вместо общего modal-wrapper — не блокер, но на будущее лучше вынести. Поправь первые три и пни обратно.
grog added 1 commit 2026-10-30 11:46:23 +03:00
grog added 1 commit 2026-10-30 11:47:15 +03:00
grog commented 2026-10-30 11:48:11 +03:00
Author
Owner
Copy Link

3 пункт не исправлял, модальное окно корректно отображается

3 пункт не исправлял, модальное окно корректно отображается
aep added 1 commit 2026-10-30 12:17:47 +03:00 
AppModal не задаёт margin у ant-modal-header, только padding — без
инлайн-override возвращался зазор, убранный в AURORA-1589 раньше.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
aep commented 2026-10-30 12:18:22 +03:00
Owner
Copy Link

Третий пункт доправил сам: AppModal задаёт только padding у шапки, margin не трогает — без инлайн-override зазор и правда вернулся. Вернул marginBottom:0 в CreateEntityPickerModal, запушил в ветку.

Третий пункт доправил сам: AppModal задаёт только padding у шапки, margin не трогает — без инлайн-override зазор и правда вернулся. Вернул marginBottom:0 в CreateEntityPickerModal, запушил в ветку.
aep merged commit 563c5103f0 into main 2026-10-30 12:35:46 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
aep
maxim
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
PotapovaA Raykov-MS SalminMP Veronika Yashin1994 aep grog maxim tsygankoviva
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RSHB/arm-core#210
No description provided.
Delete Branch "fix/consolidation"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?