Merge pull request #1148 from duckduckgo/jag/web-editor-triggers

Editor: "editor" on its own is too broad of a trigger
duckduckgo · Oct 8, 2014 · 5cfd4ce · 5cfd4ce
2 parents 9f493e9 + bfbb04c
commit 5cfd4ce
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lib/DDG/Spice/Editor.pm b/lib/DDG/Spice/Editor.pm
@@ -17,7 +17,7 @@ attribution github => ['https://github.com/jmg','Juan Manuel García'],
             email => ['[email protected]','Juan Manuel García'];
 
 
-triggers startend => 'editor';
+triggers startend => 'web editor', 'online editor', 'syntax highlighter', 'syntax highlighting', 'code viewer';
 spice call_type => 'self';
 
 my @supported_languages = ("javascript", "python");

diff --git a/t/Editor.t b/t/Editor.t
@@ -7,27 +7,27 @@ use DDG::Test::Spice;
 
 ddg_spice_test(
     [qw( DDG::Spice::Editor )],
-    'python editor' => test_spice(
+    'python web editor' => test_spice(
         'python',
         call_type => 'self',
         caller => 'DDG::Spice::Editor',
     ),
-    'editor python' => test_spice(
+    'online editor python' => test_spice(
         'python',
         call_type => 'self',
         caller => 'DDG::Spice::Editor',
     ),
-    'javascript editor' => test_spice(
+    'javascript web editor' => test_spice(
         'javascript',
         call_type => 'self',
         caller => 'DDG::Spice::Editor',
     ),
-    'editor javascript' => test_spice(
+    'syntax highlighter javascript' => test_spice(
         'javascript',
         call_type => 'self',
         caller => 'DDG::Spice::Editor',
     ),
-    'editor JavaScript' => test_spice(
+    'online editor JavaScript' => test_spice(
         'JavaScript',
         call_type => 'self',
         caller => 'DDG::Spice::Editor',