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

WIP: PreMailer.Net 2 #14

Closed
wants to merge 13 commits into from
Closed

WIP: PreMailer.Net 2 #14

wants to merge 13 commits into from

Conversation

martinnormark
Copy link
Contributor

Opening this pull request early, to track features and progress for PreMailer.Net 2.

Complete re-write to support external styles, honor CSS specificity, and add more pre-mailing features as supported by the "original" ruby gem.

New features

Additional pre-mailing features #5

  • Relative paths are converted to absolute paths Checks links in href, src and CSS url('')
  • A plain text version is created

Bugs

Robert McLaws and others added 13 commits October 7, 2012 03:33
- Moved to managing dependencies with NuGet instead of library
references.
- Changed namespacing so the compiler can differentiate between the root
namespace and the PreMailer class.
- Made the PreMailer class and it's only method static so it no longer
needs an instance to be called.
- Wrapped the style replacement loop in a Try/Catch so that Fizzler
choking on an unsupported CSS attribute does not prevent the merge from
completing.
Bumped the version to 2.0
Merge pull request, closes #11
Update copyright year, in GlobalAssemblyInfo.cs
Remove bloat
Enable package restore.
Remove unnecessary dependencies.
Manage dependencies.
Set version to 2.0.0-alpha.
Add release notes.
Redesign CssParser to support CSS Specificity.
Define and parse selector type.
@blowsie
Copy link

blowsie commented May 30, 2014

What happened to premailer 2.0?

@martinnormark
Copy link
Contributor Author

@blowsie I killed it since some of it got implemented by @robcthegeek and the direction of his implementation were more solid than what was in 2.0.

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.

None yet

2 participants