Skip to content

[Audit] Sprawdzić wpływ globalnych stylów DOCX i dziedziczenia po PR #230 #236

Description

@OtisRed

Kontekst

W review PR #230 sprawdzaliśmy wpływ zmian w szablonie indywidualne na wspólne style DOCX.

Aktualny wniosek: PR może być poprawny merytorycznie dla samego wzoru, ale część zmian dotyka globalnych zasobów DOCX (commons/word/styles.xml, commons/word/numbering.xml), które są pakowane do każdego generowanego dokumentu przez DocumentGenerator. Warto zrobić osobny audyt stylów i dziedziczenia, żeby potwierdzić, że nic nie rozjeżdża się wizualnie w innych typach dokumentów.

Obawy do sprawdzenia

  1. Recommend jest stylem globalnym.

    • PR zmienia istniejący styl Recommend, m.in. rozmiar tekstu.
    • Recommend jest używany przez wspólny template_parts/recommendation.xml.
    • Ten partial trafia do kilku typów dokumentów: specjalne, opinia, rewalidacyjne, indywidualne_roczne.
    • Ryzyko: zalecenia w tych dokumentach mogą dostać inną czcionkę, odstępy albo numerację.
  2. Akapitzlist jest bazą dla innych stylów.

    • PR zmienia bazowy styl listowy Akapitzlist.
    • Dziedziczą po nim m.in. Recommend, RecommendText, DiagnosisList, teamMembers.
    • Ryzyko: zmiany spacingu/wcięć mogą pośrednio wpłynąć na bloki, których PR nie miał intencjonalnie zmieniać.
  3. teamMembers może być dotknięty pośrednio.

    • teamMembers dziedziczy po Akapitzlist.
    • Jest używany m.in. w protokol i zarzadzenie.
    • Ryzyko: listy członków zespołu mogą dostać inny układ/odstępy.
  4. Nowe style wyglądają raczej lokalnie, ale warto to potwierdzić.

    • DiagnosisList, diagnosis, RecommendText są używane głównie przez nowe części indywidualne.
    • Ryzyko mniejsze, ale warto potwierdzić, że nie są przypadkiem użyte w miejscach współdzielonych.
  5. Numeracja w recommendation_indywidualne.xml używa numId=1.

    • numId=1 mapuje na istniejącą numerację związaną z Reason-list.
    • W samym indywidualne prawdopodobnie działa poprawnie, bo wcześniej nie ma użycia Reason-list.
    • Ryzyko: semantycznie to kruche i może być trudne do utrzymania; warto rozważyć osobny numId dla zaleceń indywidualnych.

Proponowany zakres audytu

  • Wygenerować próbki wszystkich typów dokumentów po PR update indywidualne template #230.
  • Porównać wizualnie przynajmniej:
    • sekcje zaleceń,
    • diagnozę,
    • dodatkowe informacje/AAC,
    • listy członków zespołu w protokole i zarządzeniu,
    • numerację list wielopoziomowych.
  • Sprawdzić, czy zmiany w Recommend i Akapitzlist są celowo globalne, czy powinny zostać wydzielone do nowych stylów specyficznych dla indywidualne.

Kryteria akceptacji

  • Wiemy, które style są globalne, a które lokalne dla nowych wzorców.
  • Mamy potwierdzenie, że pozostałe dokumenty nie mają regresji wizualnych.
  • Jeśli są regresje, mamy decyzję: poprawić style globalne albo wydzielić style specyficzne dla indywidualne.

Powiązane

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    discoveryNie wiemy jak to zrobić — issue wymaga zbadania przed implementacjątemplatesSzablony dokumentów DOCX: style, wzorce MEN, porównanie wersji

    Type

    No type

    Projects

    • Status
      No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions