Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
interduo opened this issue Jan 12, 2022 · 8 comments
Labels

Comments

@interduo
Copy link
Collaborator

interduo commented Jan 12, 2022

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ę #2188).
Brakuje też popup'a z potwierdzeniem czynności.

@interduo interduo added the BUG label Jan 12, 2022
@interduo
Copy link
Collaborator Author

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
Copy link
Owner

chilek commented Jan 13, 2022

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

@interduo
Copy link
Collaborator Author

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

@interduo
Copy link
Collaborator Author

interduo commented Jan 17, 2022

Te trzy rzeczy dotyczące kontrolki wrzucania plików:
#2192
#2215
#2216
#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
Copy link
Collaborator Author

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

This comment has been minimized.

@interduo
Copy link
Collaborator Author

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
Copy link
Collaborator Author

interduo commented Jan 25, 2022

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 (#2300).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants