:write: resolve filename when only path given (#738)

Author: dset0x

common/content/buffer.js

@@ -1293,6 +1293,15 @@ const Buffer = Module("buffer", {
                 if (filename) {
                     let file = io.File(filename);
 
+                    if (file.exists() && file.isDirectory()) {
+                        if (doc.doctype !== null) {
+                            filename = OS.Path.join(filename, document.title + ".html");
+                        } else {
+                            filename = OS.Path.join(filename, filename = doc.location.pathname.split("/").pop());
+                        }
+                        file = io.File(filename)
+                    }
+
                     liberator.assert(!file.exists() || args.bang, "File exists (add ! to override)");
 
                     chosenData = { file: file, uri: window.makeURI(doc.location.href, doc.characterSet) };

common/locale/en-US/browsing.xml

@@ -470,11 +470,12 @@ have the following possibilities:
 
 <item>
     <tags>:w :write :sav :saveas</tags>
-    <spec>:sav<oa>eas</oa><oa>!</oa> <oa>file</oa></spec>
+    <spec>:sav<oa>eas</oa><oa>!</oa> <oa>file|directory</oa></spec>
     <description>
         <p>
-            Save current web page to disk. If <oa>file</oa> is omitted, save to the page's
-            default filename. Existing documents will only be overwritten if <oa>!</oa> is given.
+            Save current web page to disk. If <oa>file</oa> is omitted, or a <oa>directory</oa>
+            is given save to the page's default filename. Existing documents will only be overwritten
+            if <oa>!</oa> is given.
         </p>
     </description>
 </item>

vimperator/NEWS

@@ -3,6 +3,7 @@
     * new: support <label> for <input type=file>
     * new: fire 'change' event on file upload
     * fix search highlighting (hlsearch) in firefox 51
+    * allow passing a <directory> to `:w` instead of a complete <file>
     * fix ssl icon in statusbar
     * fix "preferences" button in Firefox add-ons page
     * disable urlseparator by default as its unexpected behaviour irritates users