Support rendering incomplete task list items with non-breaking spaces

Author: mtodd

lib/task_list/filter.rb

@@ -31,8 +31,9 @@ def self.filter(*args)
   #   :task_list_items - An array of TaskList::Item objects.
   class Filter < HTML::Pipeline::Filter
 
-    Incomplete  = "[ ]".freeze
-    Complete    = "[x]".freeze
+    Incomplete     = "[ ]".freeze
+    IncompleteNBSP = "[\xC2\xA0]".freeze
+    Complete       = "[x]".freeze
 
     # Pattern used to identify all task list items.
     # Useful when you need iterate over all items.
@@ -42,7 +43,8 @@ class Filter < HTML::Pipeline::Filter
       \s*                     # optional whitespace prefix
       (                       # checkbox
         #{Regexp.escape(Complete)}|
-        #{Regexp.escape(Incomplete)}
+        #{Regexp.escape(Incomplete)}|
+        #{Regexp.escape(IncompleteNBSP)}
       )
       (?=\s)                  # followed by whitespace
     /x
@@ -51,9 +53,11 @@ class Filter < HTML::Pipeline::Filter
       # select UL/OL
       ".//li[starts-with(text(),'[ ]')]/..",
       ".//li[starts-with(text(),'[x]')]/..",
+      ".//li[starts-with(text(),'[\xC2\xA0]')]/..",
       # and those wrapped in Ps
       ".//li/p[1][starts-with(text(),'[ ]')]/../..",
-      ".//li/p[1][starts-with(text(),'[x]')]/../.."
+      ".//li/p[1][starts-with(text(),'[x]')]/../..",
+      ".//li/p[1][starts-with(text(),'[\xC2\xA0]')]/../.."
     ].join(' | ').freeze
 
     # Selects all LIs from a TaskList UL/OL

test/task_list/filter_test.rb

@@ -78,6 +78,14 @@ def test_handles_encoding_correctly
     assert_equal unicode, item.text.strip
   end
 
+  # NOTE: This is an edge case experienced regularly by users using a Swiss
+  # German keyboard.
+  def test_non_breaking_space_between_brackets
+    text = "- [\xC2\xA0] ok"
+    assert item = filter(text)[:output].css('.task-list-item').pop, "item expected"
+    assert_equal 'ok', item.text.strip
+  end
+
   protected
 
   def filter(input, context = @context, result = nil)