search

chilek / lms

Lan Management System (LMS) public GIT repo
http://lms.org.pl
126 stars 135 forks source link

{fileupload} nie działa przeglądanie wrzuconych plików na urządzeniach mobilnych przed zapisaniem zgłoszenia #2212

Open interduo opened 2 years ago

interduo commented 2 years ago

Odtworzenie problemu Kroki odtwarzające błędne zachowanie:

  1. Otwórz https://lms/?m=rtticketadd z telefonu kom. (przeglądarka chrome/firefox)
  2. Załącz plik.
  3. Zaobserwuj problem.

Oczekiwane zachowanie Ikonka (np. lms-ui-icon-search) obok ikonki kosza, kliknięcie w nią wyświetli załącznik inline w jakimś wybranym kontenerze <span> lub <div>. Kliknięcie w nazwę pliku nie powinno usuwać pliku (na to przygotowałem łatkę https://github.com/chilek/lms/pull/2188). Brakuje też popup'a z potwierdzeniem czynności.

interduo commented 2 years ago

Jest jedno miejsce w którym to jest zaimplementowane: formularz edycji dokumentu klienta.

image

@chilek dałbyś radę zrobić takie, żeby to oczko z podglądem pojawiało się wszędzie gdzie jest {fileuploads}?

chilek commented 2 years ago

W takim razie to jest zgłoszenie błędu czy nowej funkcji?

interduo commented 2 years ago

Nadal błędu ale jak się okazuje w jednym formularzu został on już załatany. Chciałem Ci to miejsce dodatkowo wskazać.

interduo commented 2 years ago

Te trzy rzeczy dotyczące kontrolki wrzucania plików: https://github.com/chilek/lms/pull/2192 https://github.com/chilek/lms/pull/2215 https://github.com/chilek/lms/pull/2216 https://github.com/chilek/lms/pull/2300

Rozwiązują to zgłoszenie.

Co do podglądu innych typów plików (xls, pdf) - również je dorobię ale muszę najpierw najpierw znaleźć sposób.

interduo commented 2 years ago

By był komplet to przydałoby się dać możliwość podglądu plików pdf w dialogu lub wszystkich plików w jakimś określonym kontenerze np. {fileuploads inline_view_container="idkontenerapodgląduinline"}. O tyle z konkretnego linka na serwerze umiem to zrobić o tyle z obiektu nie umiem tego wyświetlić.

interduo commented 2 years ago

Nie można obejrzeć PDF nie przekazując do przeglądarki linka gdzie leży - co znaczy, że trzeba go pobrać ponownie z serwera. Trochę to słabe.

interduo commented 2 years ago

Trzeba fixa na takie zdarzenie:

interduo commented 2 years ago

Gdyby udało się jeszcze inline wyświetlać dokumenty HTML to byłby komplet i można by było zaimplementować to w wielu miejscach w LMS - tak by nie skakać po zakładkach.

No i tak patrzę na kod {documentview} - to też może korzystać z lmsFileView.

No i podglądu inline w jakimś view_selector nie ma w Twoim commicie (https://github.com/chilek/lms/pull/2300).