-
Notifications
You must be signed in to change notification settings - Fork 7
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
Wyciszenie powiadomień w obrębie wątku #324
base: master
Are you sure you want to change the base?
Conversation
$this->addCustomButtons( | ||
$a_item, | ||
'a' . $a_item['raw']['postid'], | ||
' onclick="return qa_answer_click('.qa_js($answerId).', '.qa_js($questionId).', this);"' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Czy możemy obsłużyć klik z poziomu JS, zamiast HTML? Skrypt /qa-plugin/q2apro-on-site-notifications/script.js powinien się do tego nadać.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wszystko możemy 😃 Wrzuciłem na razie rozwiązanie na szybko, ale jak najbardziej w wolnej chwili chętnie poprawię, dzięki 😄
$this->addCustomButtons( | ||
$q_view, | ||
'q', | ||
' onclick="qa_show_waiting_after(this, false);"' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jak wyżej.
Chyba coś mi nie działa albo coś źle robię, pomimo tego że mam pytanie czy odpowiedź jako odsubskrybowaną, to po dodaniu odpowiedzi/komentarza dostaję powiadomienie. Przykładowo mam w jednym oknie zalogowanego użytkownika admin, w drugi użytkownika user. Jako user dodaję pytanie, domyślnie są powiadomienia włączone, klikam aby wyłączyć, w tabeli w bazie pojawia się wpis że wyciszone. Jako admin dodaję komentarz do pytania oraz odpowiedź i na koncie user dostaję dwa powiadomienia. Tak użytkowo - mam pytanie użytkownika admin z jedną odpowiedzią użytkownika admin. Wchodząc na nie jako user nie mogę włączyć ani wyciszyć powiadomień dla pytania ani odpowiedzi. Dodaję odpowiedź jako user i teraz dla tej mojej odpowiedzi mogę włączyć i wyłączyć wyciszenie oraz dla pytania także pojawił się przycisk, nie kumam czemu raz jest a raz go nie ma. Przycisku nie ma także na tej jednej odpowiedzi, która tam była, dodaje do niej komentarz jako user i przycisk się pojawia, zostaje już niezależnie czy włączę czy wyłączę wyciszenie. Rozumiem, że to ma działać tak na zasadzie tego, gdzie użytkownik aktualnie może dostać powiadomienie i w zależności od tego pokazuje przycisk, ale w takim razie czemu po dodaniu odpowiedzi pokazał mi się przy pytaniu, tego nie wiem. Tylko jak tak klikam to czy tutaj nie lepszym rozwiązaniem byłoby to, aby udostępnić ten przycisk zawsze, do wszystkich pytań i odpowiedzi, nawet jeśli ktoś nie jest autorem danego posta? Bo nie wiem czy taki był zamysł, ale wydaje mi się że tak byłoby najlepiej, najbardziej intuicyjnie, że ten przycisk tam po prostu zawsze jest w tym samym miejscu. Wtedy byłaby też konieczność dorobienia wysyłania dodatkowych powiadomień, nawet jeśli ktoś nie jest uczestnikiem posta. Miałoby to funkcję na zasadzie takiej, że jeśli ktoś zobaczy ciekawy temat o którym chciałby dostawać powiadomienia, no to mógłby sobie to włączyć wtedy. Obecnie to ja się sam trochę zgubiłem czemu to raz działa a raz nie, dlatego boję się czy takiego samego problemu nie będzie miał użytkownik. Poza tym na ukrytej odpowiedzi można nadal przełączać opcję wyciszenia, to ma sens? Podczas ukrycia i tak nie można nic dodać ani powiadomienie nie przyjdzie, ewentualnie gdyby dany post został przywrócony to mogłoby działać. Zastanawiam się też czy tak użytkowo nie warto byłoby odwrócić ikonek względem tego co jest obecnie? To w sumie zależy od podejścia, dlatego się zastanawiam jak lepiej zrobić, aby to było intuicyjne. Teraz przekreślony dzwonek wyświetla się w sytuacji, gdy post tak naprawdę nie jest wyciszony - dopiero kliknięcie go spowoduje wyciszenie, no i to ma sens, tylko czy ktoś widząc przekreślony dzwonek nie uzna, że powiadomienia są wyciszone już teraz? Może ikonka powinna pokazywać stan aktualny? |
Umieszczam prototyp rozwiązania, zostanie ono później udokumentowane, a kod posprzątany 😄