Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Why are colours and patterns set on 0-width borders? #390

Closed
Synchro opened this issue Apr 26, 2016 · 6 comments
Closed

Why are colours and patterns set on 0-width borders? #390

Synchro opened this issue Apr 26, 2016 · 6 comments
Labels

Comments

@Synchro
Copy link
Contributor

Synchro commented Apr 26, 2016

I'm not sure if this is a bug or not. The CSS includes lots of elements like this:

border: 0px solid #3adb76;

If a border has 0 width, whether it's solid or what colour it is doesn't matter, yet this pattern appears in many places. If this has some specific purpose then please ignore this ticket, but if it's not intentional, I'll make a PR to clean them up.

@rafibomb
Copy link
Member

Can you point to the files or lines that have these? Sounds incorrect but we can make sure.

@Synchro
Copy link
Contributor Author

Synchro commented Apr 26, 2016

Actually I think they are all in _buttons.scss, it just expands to more when generating the CSS, but these are in the source (there are others in there too):

1 2 3 4 5

@Synchro
Copy link
Contributor Author

Synchro commented Apr 26, 2016

The only reason I can see for having them there is if later overrides change only the width, they will still inherit the line style and colour. Not sure if that's actually the case though.

@tdhartwick
Copy link
Contributor

@Synchro Yeah, this looks a little odd. Looks like I changed it a few months ago as we were developing this. See: f7762f4

It looks like there was an odd gap that appeared in some email clients. I don't remember if this was a hack or a quick fix...

Any chance you can test it out to see if anything happens once it's removed?

@codedbypaul
Copy link

codedbypaul commented May 13, 2016

I've found that changing:

table.button table td a

to:

border: none

or

border: 0px solid transparent

causes the button to lose all padding in Outlook 2013 (possibly other Outlooks but only using the free trial of Litmus atm).

If I keep it I end up with a border around the link tag as well.

@rafibomb
Copy link
Member

I think this was part of the hybrid buttons patterns. We're also looking to get Bulletproof buttons into Inky!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants