Skip to content

Align td and th with css instead of align property#147

Merged
robin850 merged 1 commit into
vmg:masterfrom
julian7:patch-1
Jul 24, 2013
Merged

Align td and th with css instead of align property#147
robin850 merged 1 commit into
vmg:masterfrom
julian7:patch-1

Conversation

@julian7

@julian7 julian7 commented Jul 26, 2012

Copy link
Copy Markdown
Contributor

When you align tables with css, gets overridden, and content creator has no control on alignment.

However, inline styles takes precedence over any CSS the page has.

When you align tables with css, <td align="..."> gets overridden.
Using inline styles keeps alignment no matter what CSS contains.
@Tuckie

Tuckie commented Aug 3, 2012

Copy link
Copy Markdown

+1

@vmg

vmg commented Aug 3, 2012

Copy link
Copy Markdown
Owner

Hey, nice change, but this needs to go in Sundown upstream so I can backport it. Would you mind opening the PR there? 💜

@vmg vmg closed this Aug 3, 2012
@julian7

julian7 commented Aug 3, 2012

Copy link
Copy Markdown
Contributor Author

I have figured it out already, but I had no time to do that. I will do it though.

@julian7

julian7 commented Jun 5, 2013

Copy link
Copy Markdown
Contributor Author

Sundown has been discontinued, my other issue is open for a while, and I can't reopen this issue. 😞

@ghost ghost assigned mattr- Jun 5, 2013
@mattr- mattr- reopened this Jun 5, 2013
@robin850

Copy link
Copy Markdown
Collaborator

@mattr- : Any news on this? 😄 If you want me to merge it, I'm ok. Let me know ; I could also add a test for this.

@mattr-

mattr- commented Jul 24, 2013

Copy link
Copy Markdown
Collaborator

Please go ahead and merge it. Thanks! ❤️

robin850 added a commit that referenced this pull request Jul 24, 2013
Align td and th with css instead of align property
@robin850
robin850 merged commit cd696c7 into vmg:master Jul 24, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants