1413302 - imip.css rules are too broad and are interfering with basic HTML emails

Status: RESOLVED FIXED

(Calendar :: E-mail based Scheduling (iTIP/iMIP), defect)

Description

[Joas Schilling]

Attachment: diff.png

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0
Build ID: 20171030214405

Steps to reproduce:

We at Nextcloud are currently turning our calendar invitation emails into HTML emails.
The HTML code of an Email can be taken from https://gist.github.com/nickvergessen/5ef6c41f8a21751dd253c1f1d846f458
However t



Actual results:

The layout of the emails is messed up by the Lightning plugin, because it's rules are too general (E.g. ".content").


Expected results:

The content of the email should be displayed well, and the CSS should only affect the Lightning content which is displayed below the Email

Comment 1

[Joas Schilling]

I patched the chrome/skin/lightning-common/imip.css to just be prefixed with #invitation-table and it still displayes fine:
https://gist.github.com/nickvergessen/3872e0614140b93c62fb44698c05af8d

Comment 2

[[:MakeMyDay]]

Sure, we can wrap this better to limit the css scope. However, I would prefer if you could turn invitation-table to a class instead and use .invitation-table then respectively. To do so, you can find the places to look at for invitation-table here [1]. 

Can you prepare a patch that applies to comm-central? Using the file from the packacked versions wouldn't be sufficient for all platforms. You can find here [1] some help how to get to a patch. If you're done, please upload your patch and ask me for review (by setting the r? flag and filling in my nick in the box next to it in the upload window).
 
[1] https://dxr.mozilla.org/comm-central/search?q=invitation-table+path%3Acalendar&redirect=false
[2] https://developer.mozilla.org/en-US/docs/Mercurial/Using_Mercurial

Comment 3

[Stoyan Dimitrov [:stoyan]]

Attachment: Patch provided to the gist translated over the c-c tree

I'm providing the patch with credentials to the original author.

Comment 4

[[:MakeMyDay]]

Comment on attachment 8926365 [details] [diff] [review]
Patch provided to the gist translated over the c-c tree

Review of attachment 8926365 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for the patch, looks good. Can you mae the "emails r?=" in the commit message a "email; r=" and upload an updated version for checkin? You can carry over the r+ for the patch an set the ckeckin-needed keyword.

Comment 5

[Joas Schilling]

Attachment: Patch provided to the gist translated over the c-c tree

I hope this is what you wanted?
I don't have a lot of time atm to dive into mercurial and the way you handle issues, sorry.

Comment 6

[[:MakeMyDay]]

Comment on attachment 8927727 [details] [diff] [review]
Patch provided to the gist translated over the c-c tree

Yes, thanks.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/76c38b6ed571
imip.css rules are too broad and are interfering with basic HTML emails. r=MakeMyDay

Comment 8

[Jorg K (CEST = GMT+2)]

Hmm, no 6.1 target here :-(