solomax commented on code in PR #189:
URL: https://github.com/apache/openmeetings/pull/189#discussion_r1194525863


##########
openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/InvitationTemplate.html:
##########
@@ -16,20 +16,172 @@
   KIND, either express or implied.  See the License for the
   specific language governing permissions and limitations
   under the License.
-
 -->
 <!DOCTYPE html>
 <html xmlns:wicket="http://wicket.apache.org/dtds.data/wicket-9.xsd";>
+
+<head>
+  <meta name="viewport" content="width=device-width, initial-scale=1.0">
+  <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
+  <style>
+    @import 
url('https://fonts.googleapis.com/css2?family=Montserrat:wght@400;500&display=swap');
+
+    @media only screen and (max-width: 620px) {
+      table.body h1 {
+        font-size: 28px !important;
+        margin-bottom: 10px !important;
+      }
+
+      table.body p,
+      table.body td,
+      table.body span,
+      table.body a {
+        font-size: 16px !important;
+      }
+
+      table.body .wrapper,
+      table.body .article {
+        padding: 10px !important;
+      }
+
+      table.body .content {
+        padding: 0 !important;
+      }
+
+      table.body .container {
+        padding: 0 !important;
+        width: 100% !important;
+      }
+
+      table.body .main {
+        border-left-width: 0 !important;
+        border-radius: 0 !important;
+        border-right-width: 0 !important;
+      }
+
+      table.body .btn table {
+        width: 100% !important;
+      }
+
+      table.body .btn a {
+        width: 100% !important;
+      }
+
+    }
+
+    @mwedia all {
+      body {
+        background-color: #f6f6f6;
+        font-family: sans-serif;
+        -webkit-font-smoothing: antialiased;
+        font-size: 14px;
+        line-height: 1.4;
+        margin: 0;
+        padding: 0;
+        -ms-text-size-adjust: 100%;
+        -webkit-text-size-adjust: 100%;
+        font-family: 'Montserrat', sans-serif;
+        font-size: 14px;
+      }
+
+    }
+  </style>
+</head>
+
 <wicket:extend>
-       <strong><span wicket:id="titleLbl"></span></strong><br />
-       <span wicket:id="userLbl"></span> <span wicket:id="user"></span><br/>
-       <span wicket:id="messageLbl"></span> <span 
wicket:id="message"></span><br/>
-       <div wicket:id="links">
-               <span wicket:id="comment_for_link1"></span><br/>
-               <a wicket:id="invitation_link1"><span 
wicket:id="clickMe"></span></a><br/>
-               <br/><br/>
-               <span wicket:id="comment_for_link2"></span><br/>
-               <span wicket:id="invitation_link2"></span>
-       </div>
+
+  <table role="presentation" border="0" cellpadding="0" cellspacing="0" 
class="body" style="border-collapse: separate; mso-table-lspace: 0pt; 
mso-table-rspace: 0pt; background-color: #f6f6f6; width: 100%;" width="100%" 
bgcolor="#f6f6f6">

Review Comment:
   Hello @geoffreylgv,
   
   I've checked the code, according to my tests only internal `<div>` was 
rendered (`<html>` and `<head>` were dropped)
   
   I've fixed it (please see the branch `OPENMEETINGS-2774-emails`)
   
   `TestEmailTemplate` can be used to test the actual HTML
   
   Could you please take a look at this branch?
   
   As you can see I've moved your `<style>` tag into `AbstractTemplatePage` 
(which is parent for all emails) so the styles can be re-used :))
   
   Please let me know WDYT :)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@openmeetings.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to