1015741 - Remove trailing whitespace in mail/ for CSS files.

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Josiah Bruner [:jsbruner]]

Attachment: Patch.

This patch removes all trailing whitespace in CSS files located in the mail/ directory.

Comment 1

[Josiah Bruner [:jsbruner]]

Comment on attachment 8428367 [details] [diff] [review]
Patch.

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

Richard, this patch strips all trailing white space, but I created it via a command, so don't feel like you have to look at it too closely.

Here's the command: find . -name "*.css" | xargs gsed -i 's/[ \t]*$//'

Comment 2

[Josiah Bruner [:jsbruner]]

Oh, I probably just broke your other patches. Sorry about that.

Comment 3

[Josiah Bruner [:jsbruner]]

So better idea, I'll create a new patch after you land your other two patches. That way you don't have to change anything.

Comment 4

[Richard Marti (:Paenglab)]

I had a midair collision proposing the same. ;)

Comment 5

[Josiah Bruner [:jsbruner]]

Attachment: Patch.

There, this applies over yours.

Comment 6

[Josiah Bruner [:jsbruner]]

I'll make you a deal though. You review this soon, and I'll land all these patches when I get home. :)

Comment 7

[Richard Marti (:Paenglab)]

Comment on attachment 8428368 [details] [diff] [review]
Patch.

Should almost half the size of omni.ja. ;)

Comment 8

[Josiah Bruner [:jsbruner]]

Haha, yeah. :)

https://hg.mozilla.org/comm-central/rev/0e9cac0a39c7