Page MenuHomePhabricator

Create donation comment functionality
Closed, ResolvedPublic5 Estimated Story Points

Assigned To
None
Authored By
Tim_WMDE
Aug 12 2019, 12:00 PM
Referenced Files
F30166491: image.png
Aug 30 2019, 9:47 AM
F30036786: image.png
Aug 15 2019, 7:36 AM
F30036781: image.png
Aug 15 2019, 7:35 AM
F30036740: image.png
Aug 15 2019, 7:34 AM
F30036699: image.png
Aug 15 2019, 7:34 AM
F30036665: image.png
Aug 15 2019, 7:34 AM
F30036768: image.png
Aug 15 2019, 7:34 AM
F30036653: image.png
Aug 15 2019, 7:34 AM

Description

After making a donation, users are provided a link which allows them to write a donation comment. The link is not shown for bank transfer payments (the logic for this has already been implemented). In previous skins, the comment form was shown on a separate page, however, on Laika this is supposed to be implemented as a pop-up to ensure users remain on the donation confirmation page even if they would like to write a comment.

Popup component mockup

grid/spec:

image.png (735×604 px, 70 KB)

image.png (688×343 px, 44 KB)

visual:

image.png (598×447 px, 31 KB)

image.png (692×355 px, 34 KB)

in context of the page

image.png (580×511 px, 44 KB)

in context of the page, mobile

image.png (746×320 px, 39 KB)

After sending the Spendenkommentar

image.png (217×346 px, 9 KB)

Event Timeline

The disabling of the "Write a comment button" would probably not retain its disabled state after reloading the page, we also do not check for that in the old skins and only show an error on submit when an existing comment is found.

@Jan_Dittrich following up on our discussion, we agreed we should replace the dropdowns with checkboxes. Can you please provide labels for them?

it should be

  • Ihren Kommentar öffentlich teilen (default: selected)
  • Ihren Namen am Kommentar nennen (default: not delected)

image.png (180×461 px, 8 KB)

cat17 and 10h16 display the success messages that come from the server, either

Herzlichen Dank für Ihren Kommentar. Falls Sie den Kommentar öffentlicht gemacht haben, finden Sie ihn nun auf der <a href=\"/list-comments.html\">Liste der neuesten Spendenkommentare</a>

or

Herzlichen Dank für Ihren Kommentar. Irgendein Wort im Kommentar scheint problematisch zu sein, weshalb der Kommentar nicht unmittelbar veröffentlicht werden kann. Wir schauen uns das an und schalten den Kommentar gegebenenfalls frei. Wenn Sie Fragen haben, können Sie sich über das <a href=\"/contact/get-in-touch\">Kontaktformular</a> aber auch an uns direkt wenden.

What is your recommendation, @Jan_Dittrich ? Keep the short "Danke für Ihren Spendenkommentar!" or display the message?

Do we know if people made their Kommentar öffentlich? If yes, we could just have

Danke für Ihren Spendenkommentar!

and link the word "Spendenkommentar" to the list.

as for

Herzlichen Dank für Ihren Kommentar. Irgendein Wort im Kommentar scheint problematisch zu sein, weshalb der Kommentar nicht unmittelbar veröffentlicht werden kann. Wir schauen uns das an und schalten den Kommentar gegebenenfalls frei. Wenn Sie Fragen haben, können Sie sich über das <a href=\"/contact/get-in-touch\">Kontaktformular</a> aber auch an uns direkt wenden.

– can we show this in a popup, basically by swapping the content of the comment popup or so?

I would nevertheless shorten it, it is too long to understand easily.

What should be implemented:
In case of server response OK and message being "we have to moderate/review your comment first.." we will swap the contents of the popup and display that message with the only possibility of the popup to be closed.
In case of server response OK and message being something else we just display "Thanks for you comment" instead of the link.
In case of server response ERR we display a generic error message under the text field.