Ticket #116 (closed Task: fixed)

Opened 4 years ago

Last modified 4 years ago

Add a feeback modal window in the user interface to send an email to the piwik team

Reported by: matt Owned by:
Priority: major Milestone: Feature requests
Component: Core Keywords:
Cc: Sensitive:

Description

See an example (second example) on  http://www.ericmmartin.com/simplemodal/

Attachments

feedback.patch Download (22.4 KB) - added by mauser 4 years ago.

Change History

Changed 4 years ago by mauser

Working on this.

Apart from modal I will add a Piwik_Mailer class which is missing (I need it to solve #65 too).

Changed 4 years ago by matt

(In [447]) [mauser]- refs #62 commited huge work by Maciej Zawadzi?\197?\132ski: thanks! still some work to do + QA but looks really good :) [mauser]- refs #116 commited partial fix [mauser]- refs #65 commited partial fix - fixed broken tests

Changed 4 years ago by matt

feedback:

  • similar to #65 we should test to make sure the email is sent.

In this case, if the email cannot be sent for some reasons, we should show the message the user just typed and ask him to manually email us at hello@..

  • the styles seem not defined, the input boxes are larger than the modal on my FF?
  • I suggest we have only the fields: Your email, Message. All these extra fields are a bit overkill in this case.
  • remove the "or esc Key" text
  • change the close text with an icon (for example  http://www.iconfinder.net/iconlib/bnw/16x16/actions/stop.png)
  • make sure the font is georgia to be consisten with the rest of piwik
  • good job! :)

Changed 4 years ago by mauser

Changed 4 years ago by mauser

  • close icon
  • check if email was sent
  • defined styles

Changed 4 years ago by matt

  • status changed from new to closed
  • resolution set to fixed

(In [454]) [mauser] - fixes #116, font consistent, escape key now works [mauser] - fixes #65, now catching error, printing friendly message, refactoring template code [mauser] - fixes #62, fixing JS translations api, Piwik is now ready to be translated!!! :)

Note: See TracTickets for help on using tickets.