From a57b7ea02d9c40580c3238f240bd9fae7d50d6f7 Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Sat, 9 Apr 2016 13:16:03 +0100
Subject: Simplify javascript: URLs in vomnibar.

Show "javascript:" URLs as "javascript:...".

Fixes #961.
---
 background_scripts/completion.coffee | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

(limited to 'background_scripts')

diff --git a/background_scripts/completion.coffee b/background_scripts/completion.coffee
index 47cc2a23..9980fb10 100644
--- a/background_scripts/completion.coffee
+++ b/background_scripts/completion.coffee
@@ -66,11 +66,15 @@ class Suggestion
            <span class="vimiumReset vomnibarTitle">#{@highlightQueryTerms Utils.escapeHtml @title}</span>
          </div>
          <div class="vimiumReset vomnibarBottomHalf">
-          <span class="vimiumReset vomnibarSource vomnibarNoInsertText">#{insertTextIndicator}</span><span class="vimiumReset vomnibarUrl">#{@highlightUrlTerms Utils.escapeHtml @shortenUrl()}</span>
+          <span class="vimiumReset vomnibarSource vomnibarNoInsertText">#{insertTextIndicator}</span><span class="vimiumReset vomnibarUrl">#{@highlightUrlTerms Utils.escapeHtml @simplifyJavascriptUrls @shortenUrl()}</span>
           #{relevancyHtml}
         </div>
         """
 
+  # Simplify "javascript:" URLs; show them as "javascript:..."; see #961.
+  simplifyJavascriptUrls: (url) ->
+    if Utils.hasJavascriptPrefix(url) then "javascript:..." else url
+
   # Use neat trick to snatch a domain (http://stackoverflow.com/a/8498668).
   getUrlRoot: (url) ->
     a = document.createElement 'a'
-- 
cgit v1.2.3