feat: CSSInliner::compact() constructor

Stranger6667 · Stranger6667 · commit a13fc374b0da · 2020-06-23T23:35:18.000+02:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,7 @@
 
 - `CSSInliner` and customization options. [#9](https://github.com/Stranger6667/css-inline/issues/9)
 - Option to remove "style" tags. [#11](https://github.com/Stranger6667/css-inline/issues/11)
+- `CSSInliner::compact()` constructor for producing smaller HTML output.
 
 ### Changed
 
diff --git a/src/lib.rs b/src/lib.rs
@@ -95,8 +95,7 @@
 //! </html>"#;
 //!
 //!fn main() -> Result<(), css_inline::InlineError> {
-//!    let options = css_inline::InlineOptions { remove_style_tags: true };
-//!    let inliner = css_inline::CSSInliner::new(options);
+//!    let inliner = css_inline::CSSInliner::compact();
 //!    let inlined = inliner.inline(HTML)?;
 //!    // Do something with inlined HTML, e.g. send an email
 //!    Ok(())
@@ -159,6 +158,16 @@ pub struct InlineOptions {
     pub remove_style_tags: bool,
 }
 
+impl InlineOptions {
+    /// Options for "compact" HTML output
+    #[inline]
+    pub fn compact() -> Self {
+        InlineOptions {
+            remove_style_tags: true,
+        }
+    }
+}
+
 impl Default for InlineOptions {
     #[inline]
     fn default() -> Self {
@@ -181,6 +190,15 @@ impl CSSInliner {
         CSSInliner { options }
     }
 
+    /// Inliner, that will produce "compact" HTML.
+    /// For example, "style" tags will be removed.
+    #[inline]
+    pub fn compact() -> Self {
+        CSSInliner {
+            options: InlineOptions::compact(),
+        }
+    }
+
     /// Inline CSS styles from <style> tags to matching elements in the HTML tree.
     #[inline]
     pub fn inline(&self, html: &str) -> Result<String, InlineError> {
diff --git a/tests/test_inlining.rs b/tests/test_inlining.rs
@@ -1,4 +1,4 @@
-use css_inline::{inline, CSSInliner, InlineOptions};
+use css_inline::{inline, CSSInliner};
 
 macro_rules! html {
     ($style: expr, $body: expr) => {
@@ -104,10 +104,7 @@ fn invalid_rule() {
 #[test]
 fn remove_style_tag() {
     let html = html!("h1 {background-color: blue;}", "<h1>Hello world!</h1>");
-    let options = InlineOptions {
-        remove_style_tags: true,
-    };
-    let inliner = CSSInliner::new(options);
+    let inliner = CSSInliner::compact();
     let result = inliner.inline(&html).unwrap();
     assert_eq!(result, "<html><head><title>Test</title></head><body><h1 style=\"background-color: blue;\">Hello world!</h1></body></html>")
 }
