Ticket #116 (closed Task: fixed)

Opened 5 months ago

Last modified 2 months ago

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

Reported by: matt Assigned to:
Priority: major Milestone: Future features
Component: Core Version:
Keywords: Cc:

Description

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

Attachments

feedback.patch (22.4 kB) - added by mauser on 04/22/08 11:00:14.

Change History

04/09/08 00:00:33 changed 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).

04/17/08 00:28:04 changed 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

04/17/08 00:39:25 changed 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! :)

04/22/08 11:00:14 changed by mauser

  • attachment feedback.patch added.

04/22/08 11:00:58 changed by mauser

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

04/24/08 01:00:32 changed 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!!! :)