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

Popup z tekstem komentarza jako opcja zamiast wypisywania całego tekstu komentarza #41

Closed
HyperDX opened this issue Jul 10, 2019 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@HyperDX
Copy link

HyperDX commented Jul 10, 2019

Proponuje skrócenie pola komentarz, zastępując komentarz ikonką (i) - po najechaniu na nią będzie wyświetlana treść komentarza tak jak ma to miejsce tu: https://konrad.fmdx.pl/?q=2&id=744
W związku z tym warto by było skrócić nazwę pola na "Koment.", "Kom." "Info".

  • Wykazy i stacje: X
@TomaszGasior
Copy link
Owner

Jakoś nie za bardzo podoba mi się takie rozwiązanie — wolę, gdy komentarz jest widoczny tak jak teraz.

Natomiast warto się zastanowić, do czego w ogóle użytkownicy używają pola komentarza. Jeśli do, hmm, komentarza, to ma to sens i nie trzeba wtedy niczego chować. Natomiast problemem jest to, że w wielu wykazach użytkownicy używają komentarza do wpisania zawartości, które powinny znaleźć się po prostu w oddzielnej kolumnie wykazu, ale danej kolumny RLv3 nie oferuje.

Pozostawię to zgłoszenie otwarte, ale dopóki userzy używają pola komentarza niezgodnie z przeznaczeniem (bo muszą przez braki RLv3) to nie widzę takiej potrzeby.

@TomaszGasior TomaszGasior added the question Further information is requested label Jul 10, 2019
@TomaszGasior TomaszGasior changed the title [Propozycja nowej funkcji] Skrócenie pola komentarz Skrócenie pola komentarz Jul 18, 2019
@HyperDX
Copy link
Author

HyperDX commented Oct 27, 2019

Zawsze można pójść na kompromis i np. w ustawieniach wykazu zrobić pole ukrywania treści pola komentarz, które użytkownik może zaznaczyć lub odznaczyć. Takie ukrywanie może też zmniejszyć szerokość wykazu, dzięki temu może stać się bardziej przejrzysty.

@TomaszGasior
Copy link
Owner

TomaszGasior commented Oct 27, 2019

Każda opcja konfiguracyjna ma swój koszt, bowiem aplikacja musi obsługiwać dwie formy wykonania tego samego zadania, a dodatkowo oferować użytkownikowi możliwość zmiany preferowanego zachowania. Innymi słowy: im mniej opcji konfiguracyjnych, tym łatwiej skupić się na jednym, z góry określonym, przewidywalnym zachowaniu. Oddzielną kwestią jest a11y: musiałbym zakodować ewentualny popup tak, żeby był dostępny z poziomu klawiatury dla osób niepełnosprawnych ruchowo.

Nie jestem zdecydowanie przeciwko (dlatego trzymam to zgłoszenie otwarte), natomiast nadal nie widzę sensu. Spróbuj proszę podać przykłady zastosowania takiej opcji, co taka opcja by wniosła, jakie problemy by rozwiązała, itp. Może wtedy będzie łatwiej znaleźć mi sens takiej personalizacji.

@HyperDX
Copy link
Author

HyperDX commented Oct 27, 2019

Przykład zastosowania na szybko. Długie komentarze, które mocno psują estetykę wykazu rozciągając jego wiersze np. tutaj https://radiolista.pl/wykaz/190 tu https://radiolista.pl/wykaz/220 lub tu https://radiolista.pl/wykaz/160 Malutka ikonka z pop-upem i problem może być rozwiązany, a wykaz bardziej czytelny

@TomaszGasior TomaszGasior added this to the 2. Users features milestone Jan 30, 2020
@TomaszGasior TomaszGasior changed the title Skrócenie pola komentarz Popup z tekstem komentarza jako opcja zamiast wypisywania całego tekstu komentarza Feb 2, 2020
@TomaszGasior TomaszGasior added enhancement New feature or request and removed question Further information is requested labels Feb 2, 2020
@TomaszGasior TomaszGasior self-assigned this Feb 2, 2020
@TomaszGasior
Copy link
Owner

@HyperDX Czy po zaimplementowaniu opcji #21 (własna szerokość wykazu) problem wciąż jest aktualny i nadal potrzeba funkcji wyświetlania komentarza jako popup? Jeżeli tak, proszę cię o potwierdzenie — w miarę możliwości postaram się zająć tym ficzerem w najbliższy weekend.

@HyperDX
Copy link
Author

HyperDX commented Apr 27, 2020

Sądzę, że tak, nadal jest potrzeba implementacji takiej funkcji.

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

No branches or pull requests

2 participants