jekyll-target-blank 1.0 → 1.1.0.beta

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 7d8da96186acb1475407698753b3b1fcd72be1d6
-  data.tar.gz: 98cf97e09c6fbfeb0c3400c7d5603bb433ebed44
+SHA256:
+  metadata.gz: ef384a621db70253811f8aa3996b93419b9f61538ea9475bc743bad36c9473cd
+  data.tar.gz: e839aa7f4014df3c6ae6b86afb876478ef5867d6618996b38b913a9af6e716c4
 SHA512:
-  metadata.gz: 8d1792a7fd4dff808bfb0baeb8b09dbae3a871cb63a21433e82203c0e6cbd1c42d30dee9b3ce3862355c1958047e809caabb50c8cacd02c0a5efcd0fb98f8986
-  data.tar.gz: 1b53f91c41e53238f8101847240c8e0a08843713318859e8be43c0af3fc995988802e5efbb1547dd9e70204d3b248d255cc2960789649baeb3cb5504a853eee2
+  metadata.gz: 4b9a0c884e99230342b2967339fd31e628856c2eb161b5883a1fc8916dc06162afc0bdd203b0191a1b13a7842c1f0725caf598d52f51145daa09a77cf31c8a55
+  data.tar.gz: 00dab4bc2b1baad40c54b91628593fd488c530a5ee7db72b8fca247af9a84d7cde2a71d267837c62f8cb6ef76905cf436be06809757a09de598bd7034d500117

data/.gitignore

@@ -1,4 +1,5 @@
 .idea
 /*.gem
 Gemfile.lock
-dev_notes.md
+dev_notes.md
+.vscode

data/README.md

@@ -1,6 +1,8 @@
 # Jekyll Target Blank
 
-Automatically adds a `target="_blank"` attribute to all __external__ links in Jekyll Content.
+![Jekyll Target Blank Logo](assets/logo.png "Jekyll Target Blank")
+
+Automatically adds a `target="_blank" rel="noopener noreferrer"` attribute to all __external__ links in Jekyll's content plus several other automation features for the external links. [Read more here](https://keith-mifsud.me/projects/jekyll-target-blank)
 
 [![Gem Version](https://badge.fury.io/rb/jekyll-target-blank.svg)](https://badge.fury.io/rb/jekyll-target-blank)
 [![Build Status](https://travis-ci.org/keithmifsud/jekyll-target-blank.svg?branch=master)](https://travis-ci.org/keithmifsud/jekyll-target-blank)
@@ -13,7 +15,7 @@ Add the following to your site's `Gemfile`
 gem 'jekyll-target-blank'
 ```
 
-And add the following to your site's `_config.yml`
+and add the following to your site's `_config.yml`
 
 ```yml
 plugins:
@@ -29,15 +31,15 @@ gems:
 
 ## Usage
 
-All anchor tags and markdown links pointing to an external host, other than the one listed as `url` in jekyll's `_config.yml` will automatically open in a new tab once the site has been generated.
+By default. all anchor tags and markdown links pointing to an external host, other than the one listed as the `url` in Jekyll's `_config.yml` will automatically be opened in a new browser tab once the site is generated.
 
-This includes pages, posts and collections. __Plain text links are not included__.
+All the links in pages, posts and custom collections are included except for __plain text links.
 
 ### Examples
 
 #### HTML
 
-The following html anchor tag:
+The following `HTML` anchor tag:
 
 ```html
 <a href="https://google.com">Google</a>
@@ -46,7 +48,7 @@ The following html anchor tag:
 will be replaced with:
 
 ```html
-<a href="https://google.com" target="_blank">Google</a>
+<a href="https://google.com" target="_blank" rel="noopener noreferrer">Google</a>
 ```
 
 ..unless your website's URL is google.com 😉
@@ -60,19 +62,109 @@ will be replaced with:
 will be generated as:
 
 ```html
-<a href="https://google.com" target="_blank">Google</a>
+<a href="https://google.com" target="_blank" rel="noopener noreferrer">Google</a>
+```
+
+### Configuration
+
+No custom configuration is needed for using this plugin, however, you can override some default behaviours and also make use of some extra features as explained in this section. 
+
+#### Override the default behaviour
+
+You can override the default behaviour and only force external links to open in new browser if they have a CSS class name included with the same value as the one listed in the Jekyll `_config.yml` file.
+
+To override this automation, add an entry in your site's `config.yml` file, specifying which CSS class name a link must have for it to be forced to open in a new browser:
+
+```yaml
+target-blank:
+    css_class: ext-link
+```
+
+With the above setting, only links containing the `class="ext-link"` attribute will be forced to open in a new browser.
+
+#### Automatically add additional CSS Classes
+
+You can also automatically add additional CSS classes to qualifying external links. This feature is useful when you want to add CSS styling to external links such as automatically displaying an icon to show the reader that the link will open in a new browser.
+
+You can add one or more __space__ separated CSS classes in `_config.yml` like so:
+
+ ```yaml
+ target-blank:
+     add_css_classes: css-class-one css-class-two
+ ```
+ 
+ The above example will add `class="css-class-one css-class-two"` to the generated anchor tag. These CSS class names will be added in addition to any other existing CSS class names of a link.
+ 
+ #### Override the default rel attributes
+ 
+ For security reasons, `rel="noopener noreferrer"` are added by default to all the processed external links. You can override adding any of the `noopener` and `noreferrer` values with the following entries in your site's `_config.yml` file. 
+  
+ __To exclude the `noopener` value:__
+ 
+ ```yaml
+target-blank:
+    noopener: false
+```
+
+__To exclude the `noreferrer` value:__
+ 
+ ```yaml
+target-blank:
+    noreferrer: false
+```
+
+__To exclude both `noopner` and `noreferrer` values:__
+ 
+ ```yaml
+target-blank:
+    noopener: false
+    noreferrer: false
+```
+
+#### Adding additional rel attribute values
+
+You can add additional `rel=""` attribute values by simply specifying them in your site's `_config.yml` file.
+
+```yaml
+target-blank:
+    rel: nofollow
+```
+
+or even more than one extra:
+
+```yaml
+target-blank:
+    rel: nofollow
+```
+
+__Note:__
+
+
+The `rel` setting overrides other default `rel` attribute values. Therefore, (for example), if you exclude the `noopener` value and then add it to the `rel` property, it will still be added. The following `config`:
+
+```yaml
+target-blank:
+    noopener: false
+    rel: noopener
+```
+
+will output:
+
+```html
+<a href"https://some-external-website.what" target="_blank" rel="noreferrer noopener">Some link</a>
 ```
 
+
 ## Support
 
 Simply [create an issue](https://github.com/keithmifsud/jekyll-target-blank/issues/new) and I will respond as soon as possible.
 
- 
+
 ## Contributing
 
 1. [Fork it](https://github.com/keithmifsud/jekyll-target-blank/fork)
 2. Create your feature branch (`git checkout -b my-new-feature)
-3. Commit your changes (`git commit -am 'Add some feature'`)
+3. Commit your changes (`git commit -m 'Add some feature'`)
 4. Push to the branch (git push origin my-new-feature)
 4. Create a new Pull Request
 
@@ -85,6 +177,11 @@ rake spec
 rspec
 ```
 
+## Credits
+
+The logo illustration was <a href="http://www.freepik.com">Designed by Freepik</a>. Thank you ❤️
+
+
 ## Legal
 
 This software is distributed under the [MIT](LICENSE.md) license.

data/lib/jekyll-target-blank.rb

@@ -6,19 +6,31 @@ require "uri"
 
 module Jekyll
   class TargetBlank
-    BODY_START_TAG = "<body"
+    BODY_START_TAG         = "<body"
     OPENING_BODY_TAG_REGEX = %r!<body(.*)>\s*!
 
     class << self
       # Public: Processes the content and updated the external links
-      # by adding the target="_blank" attribute.
+      # by adding target="_blank" and rel="noopener noreferrer" attributes.
       #
       # content - the document or page to be processes.
       def process(content)
-        @site_url = content.site.config["url"]
+        @site_url                              = content.site.config["url"]
+        @config                                = content.site.config
+        @target_blank_config                   = class_config
+        @requires_specified_css_class          = false
+        @required_css_class_name               = nil
+        @should_add_css_classes                = false
+        @css_classes_to_add                    = nil
+        @should_add_noopener                   = true
+        @should_add_noreferrrer                = true
+        @should_add_extra_rel_attribute_values = false
+        @extra_rel_attribute_values            = nil
 
         return unless content.output.include?("<a")
 
+        initialise
+
         content.output = if content.output.include? BODY_START_TAG
                            process_html(content)
                          else
@@ -26,21 +38,28 @@ module Jekyll
                          end
       end
 
-      # Public: Determines if the content should be processed.
+      # Public: Determines if the document should be processed.
       #
-      # doc - the document being processes.
-      def processable?(doc)
+      # doc - the document being processed.
+      def document_processable?(doc)
         (doc.is_a?(Jekyll::Page) || doc.write?) &&
           doc.output_ext == ".html" || (doc.permalink&.end_with?("/"))
       end
 
       private
 
+      def initialise
+        requires_css_class_name
+        configure_adding_additional_css_classes
+        add_default_rel_attributes?
+        add_extra_rel_attributes?
+      end
+
       # Private: Processes html content which has a body opening tag.
       #
       # content - html to be processes.
       def process_html(content)
-        head, opener, tail = content.output.partition(OPENING_BODY_TAG_REGEX)
+        head, opener, tail  = content.output.partition(OPENING_BODY_TAG_REGEX)
         body_content, *rest = tail.partition("</body>")
 
         processed_markup = process_anchor_tags(body_content)
@@ -49,22 +68,124 @@ module Jekyll
       end
 
       # Private: Processes the anchor tags and adds the target
-      # attribute if the link is external.
+      # attribute if the link is external and depending on the config settings.
       #
       # html = the html which includes the anchor tags.
       def process_anchor_tags(html)
         content = Nokogiri::HTML::DocumentFragment.parse(html)
         anchors = content.css("a[href]")
         anchors.each do |item|
-          if not_mailto_link?(item["href"]) && external?(item["href"])
-            item["target"] = "_blank"
+          if processable_link?(item)
+            add_target_blank_attribute(item)
+            add_rel_attributes(item)
+            add_css_classes_if_required(item)
           end
+          next
         end
         content.to_html
       end
 
+      # Private: Determines of the link should be processed.
+      #
+      # link = Nokogiri node.
+      def processable_link?(link)
+        if not_mailto_link?(link["href"]) && external?(link["href"])
+          if @requires_specified_css_class
+            return false unless includes_specified_css_class?(link)
+          end
+          true
+        end
+      end
+
+      # Private: Handles adding the target attribute of the config
+      # requires a specifies class.
+      def requires_css_class_name
+        if css_class_name_specified_in_config?
+          @requires_specified_css_class = true
+          @required_css_class_name      = specified_class_name_from_config
+        end
+      end
+
+      # Private: Configures any additional CSS classes
+      # if needed.
+      def configure_adding_additional_css_classes
+        if should_add_css_classes?
+          @should_add_css_classes = true
+          @css_classes_to_add     = css_classes_to_add_from_config.to_s
+        end
+      end
+
+      # Private: Handles the default rel attribute values
+      def add_default_rel_attributes?
+        if should_not_include_noopener?
+          @should_add_noopener = false
+        end
+
+        if should_not_include_noreferrer?
+          @should_add_noreferrrer = false
+        end
+      end
+
+      # Private: Sets any extra rel attribute values
+      # if required.
+      def add_extra_rel_attributes?
+        if should_add_extra_rel_attribute_values?
+          @should_add_extra_rel_attribute_values = true
+          @extra_rel_attribute_values            = extra_rel_attribute_values_to_add
+        end
+      end
+
+      # Private: adds the cs classes if set in config.
+      #
+      # link = Nokogiri node.
+      def add_css_classes_if_required(link)
+        if @should_add_css_classes
+          existing_classes = get_existing_css_classes(link)
+          existing_classes = " " + existing_classes unless existing_classes.to_s.empty?
+          link["class"]    = @css_classes_to_add + existing_classes
+        end
+      end
+
+      # Private: Adds a target="_blank" to the link.
+      #
+      # link = Nokogiri node.
+      def add_target_blank_attribute(link)
+        link["target"] = "_blank"
+      end
+
+      # Private: Adds the rel attribute and values to the link.
+      #
+      # link = Nokogiri node.
+      def add_rel_attributes(link)
+        rel = ""
+        if @should_add_noopener
+          rel = "noopener"
+        end
+
+        if @should_add_noreferrrer
+          unless rel.empty?
+            rel += " "
+          end
+          rel += "noreferrer"
+        end
+
+        if @should_add_extra_rel_attribute_values
+          unless rel.empty?
+            rel += " "
+          end
+          rel += @extra_rel_attribute_values
+        end
+
+        unless rel.empty?
+          link["rel"] = rel
+        end
+      end
+
+      # Private: Checks if the link is a mailto url.
+      #
+      # link - a url.
       def not_mailto_link?(link)
-        return true unless link.to_s.start_with?("mailto:")
+        true unless link.to_s.start_with?("mailto:")
       end
 
       # Private: Checks if the links points to a host
@@ -76,11 +197,141 @@ module Jekyll
           URI.parse(link).host != URI.parse(@site_url).host
         end
       end
+
+      # Private: Checks if a css class name is specified in config
+      def css_class_name_specified_in_config?
+        target_blank_config = @target_blank_config
+        case target_blank_config
+        when nil, NilClass
+          false
+        else
+          target_blank_config.fetch("css_class", false)
+        end
+      end
+
+      # Private: Checks if the link contains the same css class name
+      # as specified in config.
+      #
+      # link - the url under test.
+      def includes_specified_css_class?(link)
+        link_classes = get_existing_css_classes(link)
+        if link_classes
+          link_classes = link_classes.split(" ")
+          contained    = false
+          link_classes.each do |name|
+            contained = true unless name != @required_css_class_name
+          end
+          return contained
+        end
+        false
+      end
+
+      # Private: Gets the the css classes of the link.
+      #
+      # link - an anchor tag.
+      def get_existing_css_classes(link)
+        link["class"].to_s
+      end
+
+      # Private: Checks if the link contains the class attribute.
+      #
+      # link - an anchor tag.
+      def link_has_class_attribute?(link)
+        link.include?("class=")
+      end
+
+      # Private: Fetches the specified css class name
+      # from config.
+      def specified_class_name_from_config
+        target_blank_config = @target_blank_config
+        target_blank_config.fetch("css_class")
+      end
+
+      # Private: Checks if it should add additional CSS classes.
+      def should_add_css_classes?
+        config = @target_blank_config
+        case config
+        when nil, NilClass
+          false
+        else
+          config.fetch("add_css_classes", false)
+        end
+      end
+
+      # Private: Checks if any addional rel attribute values
+      # should be added.
+      def should_add_extra_rel_attribute_values?
+        config = @target_blank_config
+        case config
+        when nil, NilClass
+          false
+        else
+          config.fetch("rel", false)
+        end
+      end
+
+      # Private: Gets any additional rel attribute values
+      # values to add from config.
+      def extra_rel_attribute_values_to_add
+        config = @target_blank_config
+        config.fetch("rel")
+      end
+
+      # Private: Gets the CSS classes to be added to the link from
+      # config.
+      def css_classes_to_add_from_config
+        config = @target_blank_config
+        config.fetch("add_css_classes")
+      end
+
+      # Private: Determines if the noopener rel attribute value should be added
+      # based on the specified config values.
+      #
+      # Returns true if noopener is false in config.
+      def should_not_include_noopener?
+        config = @target_blank_config
+        case config
+        when nil, NilClass
+          false
+        else
+          noopener = config.fetch("noopener", true)
+          if noopener == false
+            return true
+          else
+            return false
+          end
+        end
+      end
+
+      # Private: Determines if the noreferrer rel attribute value should be added
+      # based on the specified config values.
+      #
+      # Returns true if noreferrer is false in config.
+      def should_not_include_noreferrer?
+        config = @target_blank_config
+        case config
+        when nil, NilClass
+          false
+        else
+          noreferrer = config.fetch("noreferrer", true)
+          if noreferrer == false
+            return true
+          else
+            return false
+          end
+        end
+      end
+
+      # Private: Gets the relative config values
+      # if they exist.
+      def class_config
+        @target_blank_config = @config.fetch("target-blank", nil)
+      end
     end
   end
 end
 
 # Hooks into Jekyll's post_render event.
 Jekyll::Hooks.register %i[pages documents], :post_render do |doc|
-  Jekyll::TargetBlank.process(doc) if Jekyll::TargetBlank.processable?(doc)
+  Jekyll::TargetBlank.process(doc) if Jekyll::TargetBlank.document_processable?(doc)
 end

data/lib/jekyll-target-blank/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module JekyllTargetBlank
-  VERSION = "1.0"
+  VERSION = "1.1.0.beta"
 end

data/spec/fixtures/_config.yml

@@ -2,3 +2,5 @@ url: https://keith-mifsud.me
 collections:
   docs:
     output: true
+target-blank:
+  add_css_classes: some-class other-some-class another-some-class

data/spec/fixtures/_posts/2018-07-02-post-with-external-html-link-and-random-css-classes.md

@@ -0,0 +1,6 @@
+---
+layout: default
+title: Post with external html link and random css classes
+---
+
+<a href="https://google.com" class="random-class another-random-class">Link</a>.

data/spec/fixtures/_posts/2018-07-02-post-with-html-link-containing-the-specified-css-class.md

@@ -0,0 +1,6 @@
+---
+layout: default
+title: Post with html link containing the specified css class
+---
+
+<a href="https://google.com" class="ext-link">Link with the css class specified in config</a>.

data/spec/fixtures/_posts/2018-07-05-post-with-external-link-containing-the-specified-css-class-and-other-css-classes.md

@@ -0,0 +1,6 @@
+---
+layout: default
+title: Post with external link containing the specified css class and other css classes
+---
+
+This is <a href="https://not-keith-mifsud.me" class="random-class ext-link another-random-class">a link containing the specified css class and two other random css classes</a>.

data/spec/jekyll-target_spec.rb

@@ -4,13 +4,21 @@ RSpec.describe(Jekyll::TargetBlank) do
   Jekyll.logger.log_level = :error
 
   let(:config_overrides) { {} }
+  let(:config_overrides) do
+    {
+      "url"         => "https://keith-mifsud.me",
+      "collections" => { "docs" => { "output" => "true" } },
+    }
+  end
   let(:configs) do
-    Jekyll.configuration(config_overrides.merge({
-      "skip_config_files" => false,
-      "collections"       => { "docs" => { "output" => true } },
-      "source"            => fixtures_dir,
-      "destination"       => fixtures_dir("_site"),
-    }))
+    Jekyll.configuration(config_overrides.merge(
+      {
+        "skip_config_files" => false,
+        "collections"       => { "docs" => { "output" => true } },
+        "source"            => fixtures_dir,
+        "destination"       => fixtures_dir("_site"),
+      }
+    ))
   end
   let(:target_blank) { described_class }
   let(:site) { Jekyll::Site.new(configs) }
@@ -32,7 +40,7 @@ RSpec.describe(Jekyll::TargetBlank) do
   let(:document_with_a_processable_link) { find_by_title(site.collections["docs"].docs, "Document with a processable link") }
 
   let(:text_file) { find_by_title(site.collections["docs"].docs, "Text file") }
-
+  0
   let(:post_with_code_block) { find_by_title(posts, "Post with code block") }
   let(:document_with_liquid_tag) { find_by_title(site.collections["docs"].docs, "Document with liquid tag") }
 
@@ -40,6 +48,12 @@ RSpec.describe(Jekyll::TargetBlank) do
 
   let(:post_with_mailto_link) { find_by_title(posts, "Post with mailto link") }
 
+  let(:post_with_external_html_link_and_random_css_classes) { find_by_title(posts, "Post with external html link and random css classes") }
+
+  let(:post_with_html_link_containing_the_specified_css_class) { find_by_title(posts, "Post with html link containing the specified css class") }
+
+  let(:post_with_external_link_containing_the_specified_css_class_and_other_css_classes) { find_by_title(posts, "Post with external link containing the specified css class and other css classes") }
+
   # define common wrappers.
   def para(content)
     "<p>#{content}</p>"
@@ -52,82 +66,360 @@ RSpec.describe(Jekyll::TargetBlank) do
     site.render
   end
 
-  it "should add target attribute to external markdown link" do
-    expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank">Google</a>.'))
-  end
+  context "Without entries in config file" do
+    let(:config_overrides) do
+      { "target-blank" => { "add_css_classes" => false } }
+    end
 
-  it "should add target attribute to multiple external markdown links" do
-    expect(post_with_multiple_external_markdown_links.output).to include('<p>This post contains three links. The first link is to <a href="https://google.com" target="_blank">Google</a>, the second link is, well, to <a href="https://keithmifsud.github.io" target="_blank">my website</a> and since <a href="https://github.com" target="_blank">GitHub</a> is so awesome, why not link to them too?</p>')
-  end
+    it "should add target attribute to external markdown link" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noopener noreferrer">Google</a>.'))
+    end
 
-  it "should not add target attribute to relative markdown link" do
-    expect(post_with_relative_markdown_link.output).to include(para('Link to <a href="/contact">contact page</a>.'))
+    it "should add target attribute to multiple external markdown links" do
+      expect(post_with_multiple_external_markdown_links.output).to include('<p>This post contains three links. The first link is to <a href="https://google.com" target="_blank" rel="noopener noreferrer">Google</a>, the second link is, well, to <a href="https://keithmifsud.github.io" target="_blank" rel="noopener noreferrer">my website</a> and since <a href="https://github.com" target="_blank" rel="noopener noreferrer">GitHub</a> is so awesome, why not link to them too?</p>')
+    end
 
-    expect(post_with_relative_markdown_link.output).to_not include(para('Link to <a href="/contact" target="_blank">contact page</a>'))
-  end
+    it "should not add target attribute to relative markdown link" do
+      expect(post_with_relative_markdown_link.output).to include(para('Link to <a href="/contact">contact page</a>.'))
 
-  it "should not add target attribute to absolute internal link" do
-    expect(post_with_absolute_internal_markdown_link.output).to include('<p>This is an absolute internal <a href="https://keith-mifsud.me/contact">link</a>.</p>
-')
-  end
+      expect(post_with_relative_markdown_link.output).to_not include(para('Link to <a href="/contact" target="_blank" rel="noopener noreferrer">contact page</a>'))
+    end
+
+    it "should not add target attribute to absolute internal link" do
+      expect(post_with_absolute_internal_markdown_link.output).to include('<p>This is an absolute internal <a href="https://keith-mifsud.me/contact">link</a>.</p>')
+    end
+
+    it "should correctly handle existing html anchor tag" do
+      expect(post_with_html_anchor_tag.output).to include('<p>This is an <a href="https://google.com" target="_blank" rel="noopener noreferrer">anchor tag</a>.</p>')
+    end
+
+    it "should not interfere with plain text link" do
+      expect(post_with_plain_text_link.output).to include("<p>This is a plain text link to https://google.com.</p>")
+    end
+
+    it "should process external links in collections" do
+      expect(document_with_a_processable_link.output).to include('<p>This is a valid <a href="https://google.com" target="_blank" rel="noopener noreferrer">link</a>.</p>')
+    end
+
+    it "should process external links in pages" do
+      expect(site.pages.first.output).to include('<p>This is a valid <a href="https://google.com" target="_blank" rel="noopener noreferrer">link</a>.</p>')
+    end
+
+    it "should not process links in non html files" do
+      expect(text_file.output).to eq("Valid [link](https://google.com).")
+    end
+
+    it "should not process link in code block but process link outside of block" do
+      expect(post_with_code_block.output).to include('<span class="s1">\'https://google.com\'</span>')
 
-  it "should correctly handle existing html anchor tag" do
-    expect(post_with_html_anchor_tag.output).to include('<p>This is an <a href="https://google.com" target="_blank">anchor tag</a>.</p>
-')
+      expect(post_with_code_block.output).not_to include('<span class="s1"><a href="https://google.com" target="_blank" rel="noopener noreferrer">https://google.com</a></span>')
+
+      expect(post_with_code_block.output).to include('<p>Valid <a href="https://google.com" target="_blank" rel="noopener noreferrer">link</a></p>')
+    end
+
+    it "should not break layouts" do
+      expect(site.pages.first.output).to include('<html lang="en-US">')
+      expect(site.pages.first.output).to include('<body class="wrap">')
+    end
+
+    it "should not interfere with liquid tags" do
+      expect(document_with_liquid_tag.output).to include('<p>This <a href="/docs/document-with-liquid-tag.html">_docs/document-with-liquid-tag.md</a> is a document with a liquid tag.</p>')
+    end
+
+    it "should not interfere with includes" do
+      expect(document_with_include.output).to include("<p>This is a document with an include: This is an include.</p>")
+    end
+
+    it "should not break layout content" do
+      expect(site.pages.first.output).to include("<div>Layout content started.</div>")
+
+      expect(site.pages.first.output).to include("<div>Layout content ended.</div>")
+    end
+
+    it "should not duplicate post content" do
+      expect(post_with_external_markdown_link.output).to eq(post_with_layout_result)
+    end
+
+    it "should ignore mailto links" do
+      expect(post_with_mailto_link.output).to include(para('This is a <a href="mailto:mifsud.k@gmail.com?Subject=Just%20an%20email">mailto link</a>.'))
+    end
   end
 
-  it "should not interfere with plain text link" do
-    expect(post_with_plain_text_link.output).to include("<p>This is a plain text link to https://google.com.</p>
-")
+  context "With a specified css class name" do
+    let(:target_blank_css_class) { "ext-link" }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "css_class"       => target_blank_css_class,
+          "add_css_classes" => false,
+        },
+      }
+    end
+
+    it "should not add target attribute to external markdown link that does not have the specified css class" do
+      expect(post_with_external_markdown_link.output).to_not include(para('Link to <a href="https://google.com" target="_blank">Google</a>.'))
+    end
+
+    it "should not add target attribute to external markdown link that does not have the specified css class even if it does have other css classes" do
+      expect(post_with_external_html_link_and_random_css_classes.output).to include(para('<a href="https://google.com" class="random-class another-random-class">Link</a>.'))
+
+      expect(post_with_external_html_link_and_random_css_classes.output).to_not include('target="_blank" rel="noopener noreferrer"')
+    end
+
+    it "should add target attribute to an external link containing the specified css class" do
+      expect(post_with_html_link_containing_the_specified_css_class.output).to include(para('<a href="https://google.com" class="ext-link" target="_blank" rel="noopener noreferrer">Link with the css class specified in config</a>.'))
+    end
+
+    it "should add target attribute to an external link containing the specified css class even when other css classes are specified" do
+      expect(post_with_external_link_containing_the_specified_css_class_and_other_css_classes.output).to include(para('This is <a href="https://not-keith-mifsud.me" class="random-class ext-link another-random-class" target="_blank" rel="noopener noreferrer">a link containing the specified css class and two other random css classes</a>.'))
+    end
   end
 
-  it "should process external links in collections" do
-    expect(document_with_a_processable_link.output).to include('<p>This is a valid <a href="https://google.com" target="_blank">link</a>.</p>
-')
+  context "Adds a CSS classes to the links" do
+    let(:target_blank_add_css_class) { "some-class" }
+    let(:config_overrides) do
+      { "target-blank" => { "add_css_classes" => target_blank_add_css_class } }
+    end
+
+    it "should add the CSS class specified in config" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noopener noreferrer" class="some-class">Google</a>.'))
+    end
+
+    it "should add the CSS class specified in config even when the link already has a CSS class specified" do
+      expect(post_with_html_link_containing_the_specified_css_class.output).to include(para('<a href="https://google.com" class="some-class ext-link" target="_blank" rel="noopener noreferrer">Link with the css class specified in config</a>.'))
+    end
+
+    it "should add the CSS class specified in config even when the link has more than CSS classes already included" do
+      expect(post_with_external_link_containing_the_specified_css_class_and_other_css_classes.output).to include(para('This is <a href="https://not-keith-mifsud.me" class="some-class random-class ext-link another-random-class" target="_blank" rel="noopener noreferrer">a link containing the specified css class and two other random css classes</a>.'))
+    end
   end
 
-  it "should process external links in pages" do
-    expect(site.pages.first.output).to include('<p>This is a valid <a href="https://google.com" target="_blank">link</a>.</p>')
+  context "When more than one CSS classes are specified in config" do
+    it "should add the CSS classes specified in config" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noopener noreferrer" class="some-class other-some-class another-some-class">Google</a>.'))
+    end
+
+    it "should add the CSS classes specified in config even when the link already has a CSS class included" do
+      expect(post_with_html_link_containing_the_specified_css_class.output).to include(para('<a href="https://google.com" class="some-class other-some-class another-some-class ext-link" target="_blank" rel="noopener noreferrer">Link with the css class specified in config</a>.'))
+    end
+
+    it "should add the CSS classes specified in config even when the link already has more than one CSS classes included" do
+      expect(post_with_external_link_containing_the_specified_css_class_and_other_css_classes.output).to include(para('This is <a href="https://not-keith-mifsud.me" class="some-class other-some-class another-some-class random-class ext-link another-random-class" target="_blank" rel="noopener noreferrer">a link containing the specified css class and two other random css classes</a>.'))
+    end
   end
 
-  it "should not process links in non html files" do
-    expect(text_file.output).to eq("Valid [link](https://google.com).")
+  context "When noopener is set to false in config" do
+    let(:noopener) { false }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "noopener"        => noopener,
+        },
+      }
+    end
+
+    it "should not add noopener value to the rel attribute" do
+      expect(post_with_external_markdown_link.output).to_not include(para('Link to <a href="https://google.com" target="_blank" rel="noopener noreferrer">Google</a>.'))
+    end
+
+    it "should still add noreferrer value to the rel attribute" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noreferrer">Google</a>.'))
+    end
   end
 
-  it "should not process link in code block but process link outside of block" do
-    expect(post_with_code_block.output).to include("<span class=\"s1\">'https://google.com'</span>")
+  context "When noreferrer is set to false in config" do
+    let(:noreferrer) { false }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "noreferrer"      => noreferrer,
+        },
+      }
+    end
+
+    it "should not add noreferrer value to the rel attribute" do
+      expect(post_with_external_markdown_link.output).to_not include(para('Link to <a href="https://google.com" target="_blank" rel="noopener noreferrer">Google</a>.'))
+    end
+
+    it "should still add noopener value to the rel attribute" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noopener">Google</a>.'))
+    end
+  end
 
-    expect(post_with_code_block.output).not_to include("<span class=\"s1\"><a href=\"https://google.com\" target=\"_blank\">https://google.com</a></span>")
+  context "When both noopener and noreferrer values are set to false in config" do
+    let(:noopener) { false }
+    let(:noreferrer) { false }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "noopener"        => noopener,
+          "noreferrer"      => noreferrer,
+        },
+      }
+    end
+
+    it "should not include the rel attribute values" do
+      expect(post_with_external_markdown_link.output).to_not include(para('Link to <a href="https://google.com" target="_blank" rel="noopener noreferrer">Google</a>.'))
+    end
+
+    it "should not include the rel attribute noopener value" do
+      expect(post_with_external_markdown_link.output).to_not include(para('Link to <a href="https://google.com" target="_blank" rel="noreferrer">Google</a>.'))
+    end
+
+    it "should not include the rel attribute noreferrer value" do
+      expect(post_with_external_markdown_link.output).to_not include(para('Link to <a href="https://google.com" target="_blank" rel="noopener">Google</a>.'))
+    end
+
+    it "should not include any rel attributes" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank">Google</a>.'))
+    end
+  end
 
-    expect(post_with_code_block.output).to include('<p>Valid <a href="https://google.com" target="_blank">link</a></p>
-')
+  context "When one additional rel attribute is added in config" do
+    let(:rel_attribute) { "nofollow" }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "rel"             => rel_attribute,
+        },
+      }
+    end
+
+    it "should add the extra rel attribute together with the default ones" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noopener noreferrer nofollow">Google</a>.'))
+    end
   end
 
-  it "should not break layouts" do
-    expect(site.pages.first.output).to include('<html lang="en-US">')
-    expect(site.pages.first.output).to include('<body class="wrap">')
+  context "When more than one additional rel attributes are added in config" do
+    let(:rel_attribute) { "nofollow tag" }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "rel"             => rel_attribute,
+        },
+      }
+    end
+
+    it "should add the extra rel attributes together with the default ones" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noopener noreferrer nofollow tag">Google</a>.'))
+    end
   end
 
-  it "should not interfere with liquid tags" do
-    expect(document_with_liquid_tag.output).to include('<p>This <a href="/docs/document-with-liquid-tag.html">_docs/document-with-liquid-tag.md</a> is a document with a liquid tag.</p>')
+  context "When one extra rel attribute value are set in config and noopener is set to false" do
+    let(:rel_attribute) { "nofollow" }
+    let(:noopener) { false }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "noopener"        => noopener,
+          "rel"             => rel_attribute,
+        },
+      }
+    end
+
+    it "should the extra rel attribute value and not add the default noopener value" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noreferrer nofollow">Google</a>.'))
+    end
   end
 
-  it "should not interfere with includes" do
-    expect(document_with_include.output).to include("<p>This is a document with an include: This is an include.</p>")
+  context "When more than one extra rel attribute values are set in config and noopener is set to false" do
+    let(:rel_attribute) { "nofollow tag" }
+    let(:noopener) { false }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "noopener"        => noopener,
+          "rel"             => rel_attribute,
+        },
+      }
+    end
+
+    it "should the extra rel attribute values and not add the default noopener value" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noreferrer nofollow tag">Google</a>.'))
+    end
   end
 
-  it "should not break layout content" do
-    expect(site.pages.first.output).to include("<div>Layout content started.</div>")
+  context "When one extra rel attributes is set in config and both noopener and noreferer are set to false" do
+    let(:rel_attribute) { "nofollow" }
+    let(:noopener) { false }
+    let(:noreferrer) { false }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "noopener"        => noopener,
+          "noreferrer"      => noreferrer,
+          "rel"             => rel_attribute,
+        },
+      }
+    end
+
+    it "should add the extra rel attribute value and no default ones" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="nofollow">Google</a>.'))
+    end
+  end
 
-    expect(site.pages.first.output).to include("<div>Layout content ended.</div>")
+  context "When more than one extra rel attribute values are set in config and both noopener and noreferer are set to false" do
+    let(:rel_attribute) { "nofollow tag" }
+    let(:noopener) { false }
+    let(:noreferrer) { false }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "noopener"        => noopener,
+          "noreferrer"      => noreferrer,
+          "rel"             => rel_attribute,
+        },
+      }
+    end
+
+    it "should add the extra rel attribute values and no default ones" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="nofollow tag">Google</a>.'))
+    end
   end
 
-  it "should not duplicate post content" do
-    expect(post_with_external_markdown_link.output).to eq(post_with_layout_result)
+  context "When noopener is set to false in config but added to the rel config property" do
+    let(:rel_attribute) { "noopener" }
+    let(:noopener) { false }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "noopener"        => noopener,
+          "rel"             => rel_attribute,
+        },
+      }
+    end
+
+    it "should still include the noopener rel attribute value" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noreferrer noopener">Google</a>.'))
+    end
   end
 
-  it "should ignore mailto links" do
-    expect(post_with_mailto_link.output).to include(para('This is a <a href="mailto:mifsud.k@gmail.com?Subject=Just%20an%20email">mailto link</a>.'))
+  context "When noopener is set to false in config but added t0 the rel config property alongside one more extra rel attribute value." do
+    let(:rel_attribute) { "noopener nofollow" }
+    let(:noopener) { false }
+    let(:config_overrides) do
+      {
+        "target-blank" => {
+          "add_css_classes" => false,
+          "noopener"        => noopener,
+          "rel"             => rel_attribute,
+        },
+      }
+    end
+
+    it "should still include the noopener rel attribute value along the extra one" do
+      expect(post_with_external_markdown_link.output).to include(para('Link to <a href="https://google.com" target="_blank" rel="noreferrer noopener nofollow">Google</a>.'))
+    end
   end
 
   private
@@ -144,7 +436,7 @@ RSpec.describe(Jekyll::TargetBlank) do
 </head>
 <body class="wrap">
     <div>Layout content started.</div>
-<p>Link to <a href="https://google.com" target="_blank">Google</a>.</p>
+<p>Link to <a href="https://google.com" target="_blank" rel="noopener noreferrer">Google</a>.</p>
 
     <div>Layout content ended.</div>
 </body>

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-target-blank
 version: !ruby/object:Gem::Version
-  version: '1.0'
+  version: 1.1.0.beta
 platform: ruby
 authors:
 - Keith Mifsud
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-30 00:00:00.000000000 Z
+date: 2018-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -110,6 +110,7 @@ files:
 - LICENSE.md
 - README.md
 - Rakefile
+- assets/logo.png
 - jekyll-target-blank.gemspec
 - lib/jekyll-target-blank.rb
 - lib/jekyll-target-blank/version.rb
@@ -131,6 +132,9 @@ files:
 - spec/fixtures/_posts/2018-05-23-post-with-absolute-internal-markdown-link.md
 - spec/fixtures/_posts/2018-05-24-post-with-code-block.md
 - spec/fixtures/_posts/2018-05-30-post-with-mailto-link.md
+- spec/fixtures/_posts/2018-07-02-post-with-external-html-link-and-random-css-classes.md
+- spec/fixtures/_posts/2018-07-02-post-with-html-link-containing-the-specified-css-class.md
+- spec/fixtures/_posts/2018-07-05-post-with-external-link-containing-the-specified-css-class-and-other-css-classes.md
 - spec/fixtures/index.md
 - spec/jekyll-target_spec.rb
 - spec/spec_helper.rb
@@ -149,12 +153,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14.1
+rubygems_version: 2.7.7
 signing_key: 
 specification_version: 4
 summary: Target Blank automatically changes the external links to open in a new browser.