Browse Source

Remove non-working code and explain why.

I prefer to remove this code than leaving it there when it does not
work. The removal reason and the actual problem are better documented in
the Known Issues section of the README so other developers get it easier
to work on it.
pull/44/head
Jairo Llopis 9 years ago
parent
commit
2c04dae9eb
  1. 11
      website_mail_snippet_responsive/README.rst
  2. 5
      website_mail_snippet_responsive/views/templates.xml

11
website_mail_snippet_responsive/README.rst

@ -51,6 +51,17 @@ Known issues / Roadmap
will insert a ``<br type="_moz"/>`` in Firefox, that will make it seem like
its height is at least like a caret, even if you set a lower value. Do not
worry, it goes away when you press *Save*.
* Some elements do not render with the proper width in MS Outlook. This should
be added inside each snippet to make them work::
<!--[if gte mso]>
<style type="text/css">
.fluid { width: 600px !important; }
</style>
<![endif]-->
But right now the view parser eats comments, and if you avoid that then
CKEditor will eat them, so there's not an easy solution for now.
Bug Tracker
===========

5
website_mail_snippet_responsive/views/templates.xml

@ -18,11 +18,6 @@
<div
class="oe_snippet_body"
style="padding:0px; width:100%; background-color:#ececec; color:rgb(0,0,0); line-height:20px; font-family:Arial,sans-serif; font-size:9pt">
<!--[if gte mso]>
<style type="text/css">
.fluid { width: 600px !important; }
</style>
<![endif]-->
<table class="fluid" align="center" cellpadding="0" cellspacing="0" style="max-width: 600px; border-collapse: collapse; background: inherit; color: inherit;">
<tbody>
<tr>

Loading…
Cancel
Save