lms icon indicating copy to clipboard operation
lms copied to clipboard

Published 20 hours ago • verified publisher icon chilek

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

Open interduo opened this issue 3 years ago • 8 comments

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 avatar Jan 12 '22 13:01 interduo

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}?

interduo avatar Jan 13 '22 16:01 interduo

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

chilek avatar Jan 13 '22 17:01 chilek

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

interduo avatar Jan 13 '22 18:01 interduo

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 avatar Jan 17 '22 14:01 interduo

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 avatar Jan 17 '22 18:01 interduo

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 avatar Jan 17 '22 20:01 interduo

Trzeba fixa na takie zdarzenie:

  • wrzucam obrazek
  • wrzucam PDF oddzielnym kliknięciem przycisku "Wybierz pliki"
  • klikam w podgląd obrazka i dostaję 404 w oknie alertu.

interduo avatar Jan 25 '22 17:01 interduo

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).

interduo avatar Jan 25 '22 17:01 interduo