Zadanie #1892
Zadanie #1885: uwagi ze spotkania
ekran szablonów numeracji się zepsuł
Status: | Zamknięty | Start date: | 2017-12-13 | |
---|---|---|---|---|
Priority: | Normalny | Due date: | ||
Assignee: | Łukasz Waśkiewicz | % Done: | 100% | |
Category: | ui | |||
Target version: | 0.3 | |||
Developer: | Michał Komar | Tester: | Jarosław Bąbel |
Description
screen
Associated revisions
refs #1892: Poprawienie wyświetlania kontrolek w journal-numbering
refs #1892 dodana onformacja o braku szablonów
refs #1892 dodanie opcji wyczyszczenia wyboru w szablonach numeracji
czasopisma
History
#1 Updated by Łukasz Waśkiewicz about 7 years ago
- Parent task set to #1885
#2 Updated by Michał Komar about 7 years ago
- Category set to ui
- Assignee set to Michał Komar
- Target version set to 0.3
- Developer Michał Komar added
- Tester Jarosław Bąbel added
#3 Updated by Michał Komar about 7 years ago
- Status changed from Nowy to W toku
#4 Updated by Michał Komar about 7 years ago
- Status changed from W toku to Testowanie
- Assignee changed from Michał Komar to Jarosław Bąbel
- % Done changed from 0 to 100
Poprawiłem
#5 Updated by Jarosław Bąbel about 7 years ago
- File Screenshot-2018-1-18 Jupiter editor document bwmeta1 element issn-1691-5771 numbering.png added
- Status changed from Testowanie to Odpowiedź
- Assignee changed from Jarosław Bąbel to Łukasz Waśkiewicz
Ekran wygląda dobrze (screen w załączniku).
Z selectów nie można nic wybrać. Nie znam tej funkcjonalności, więc nie wiem, czy to oczekiwane zachowanie.
Testowałem w rev. d609af2
#6 Updated by Łukasz Waśkiewicz about 7 years ago
- Status changed from Odpowiedź to Testowanie
- Assignee changed from Łukasz Waśkiewicz to Jarosław Bąbel
To oczekiwane zachowanie. Ale aby było klarowniej dodałem informacje o braku szablonów i ustawiłem select disabled jeśli nie ma nic do wybrania
zlokalizowałem też formularz bo teksty były na sztywno w html
#7 Updated by Jarosław Bąbel about 7 years ago
- File BrakOpcjiBezNumeracji.png added
- File TakMogloByWygladacPoOtwarciu.png added
- File PoOtwarciu.png added
#8 Updated by Jarosław Bąbel about 7 years ago
- Status changed from Testowanie to Odpowiedź
- Assignee changed from Jarosław Bąbel to Łukasz Waśkiewicz
#9 Updated by Łukasz Waśkiewicz about 7 years ago
- Status changed from Odpowiedź to Testowanie
- Assignee changed from Łukasz Waśkiewicz to Jarosław Bąbel
zachowania z punktu 1 nie udało mi się powtórzyć
punkt 2 dodałem opcję Brak (czyszczącą wybór)
#10 Updated by Jarosław Bąbel about 7 years ago
- Status changed from Testowanie to Odpowiedź
- Assignee changed from Jarosław Bąbel to Łukasz Waśkiewicz
Zachowanie 1 pojawia się u mnie jeżeli mam tylko jeden szablon dla jednego poziomu i dostatecznie szybko klikam. Z drugiej strony jak się doda taki "opóźniacz" (log na konsoli) w journal-numberings.component.ts:
private initialize(): void { this.hierarchyService.getHierarchyByJournalId( this._id ).switchMap(( hierarchy ) => { console.log("OPOZNIACZ"); let numberings: Numbering[] = []; ...
To problem znika.
Nie jest to istotny błąd, ale wydaje się, że strona jest wyświetlona czasem zanim konstruktor się zakończy.
Można to zadanie pewnie zamknąć, ale coś tam siedzi i może kiedyś mieć znaczenie w innych miejscach.
#11 Updated by Łukasz Waśkiewicz about 7 years ago
- Assignee changed from Łukasz Waśkiewicz to Jarosław Bąbel
a gdzie musisz szybko klikać bo ja wciąż nie mogę tego wywołać
otwierając stronę bezpośrednio na http://localhost:4200/editor/document/bwmeta1.element.issn-2083-4594/numbering
albo wchodząc od http://localhost:4200/editor/search i w obu przypadkach jeszcze klikając pomiędzy czasopismami które mają wybrany szablon lub nie
ten komponent rozszerza BusyComponent i do momentu załadowania wszystkiego jest przesłonka, może to problem z przesłonką, ale u mnie ona znika dopiero po załadowaniu danych i zawsze jest wszystko OK
#12 Updated by Jarosław Bąbel about 7 years ago
- Assignee changed from Jarosław Bąbel to Łukasz Waśkiewicz
W administracja -> szablony numeracji mam zdefiniowany tylko jeden szablon dla numerów. Po dodaniu drugiego czasem jedna opcja wyświetla się prawidłowo, a druga już jako "niezdefiniowana". Ale lepiej to widać jak jest tylko jeden szablon.
Przez szybkie klikanie rozumiem:- wejdź w edycję artykułu
- po wczytaniu edytora (zaraz po zniknięciu maski) szybko przełącz na zakładkę "szablony numeracji"
- otwórz edytor
- przełącz się na numerację
- przełącz się na edycję i jak tylko się wczyta kliknij znowu numerację
Wejście prosto z linku "numbering" też ten efekt daje, ale rzadziej.
#13 Updated by Łukasz Waśkiewicz about 7 years ago
- Status changed from Odpowiedź to Testowanie
- Assignee changed from Łukasz Waśkiewicz to Jarosław Bąbel
- poprawiłem obsługę disabled ( w konsoli JS był warning )
- zlokalizowałem problem z odpowiedzi. Było nim tworzenie obiektu Numbering, które w konstruktorze miało wołanie do backendu o listę opcji. Po zakończeniu inicjalizacji przy generowaniu formularza options (opcje do wyboru) mogły jeszcze nie być zapisane wiec otrzymywaliśmy informacje o braku, ale chwile później opcje już były (po naprawianiu WARN z disabled skutkowało to tym ze selecty były na stałe wyłączone)
przerobiłem aby inicjalizacja zwracała Promise<Numbering> i dopiero po zakończeniu wszystkich kontynuowało inicjalizację
#14 Updated by Jarosław Bąbel almost 7 years ago
- Status changed from Testowanie to Rozwiązany
- Assignee changed from Jarosław Bąbel to Łukasz Waśkiewicz
#15 Updated by Łukasz Waśkiewicz almost 7 years ago
- Status changed from Rozwiązany to Zamknięty