Merge pull request #36 from Y2Z/no-images-no-icons

Ignore icons if told to avoid images
2019-08-24 14:49:26 -04:00 · 2019-08-24 14:49:26 -04:00 · 080ed50264
commit 080ed50264
parent 008521f72f b596b5bb7d
1 changed files with 14 additions and 9 deletions
--- a/src/html.rs
+++ b/src/html.rs
@ -137,6 +137,10 @@ pub fn walk_and_embed_assets(
                NodeMatch::Icon => {
                    for attr in attrs_mut.iter_mut() {
                        if &attr.name.local == "href" {
+                            if opt_no_images {
+                                attr.value.clear();
+                                attr.value.push_slice(TRANSPARENT_PIXEL);
+                            } else {
                                let href_full_url = resolve_url(&url, &attr.value.to_string());
                                let favicon_datauri = retrieve_asset(
                                    &href_full_url.unwrap(),
@ -149,6 +153,7 @@ pub fn walk_and_embed_assets(
                            }
                        }
                    }
+                }
                NodeMatch::Image => {
                    for attr in attrs_mut.iter_mut() {
                        if &attr.name.local == "src" {