diff --git a/.travis.yml b/.travis.yml index e8e09d4..b9b8fa2 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,5 +1,4 @@ -## Project specific config ### -language: generic +language: objective-c env: global: @@ -10,36 +9,12 @@ env: - ATOM_CHANNEL=stable - ATOM_CHANNEL=beta -os: - - linux - -dist: trusty - -### Generic setup follows ### -script: - - curl -s -O https://raw.githubusercontent.com/atom/ci/master/build-package.sh - - chmod u+x build-package.sh - - ./build-package.sh - notifications: email: on_success: never on_failure: change -branches: - only: - - master +script: 'curl -s https://raw.githubusercontent.com/atom/ci/master/build-package.sh | sh' git: depth: 10 - -sudo: false - -addons: - apt: - packages: - - build-essential - - git - - libgnome-keyring-dev - - libsecret-1-dev - - fakeroot diff --git a/CHANGELOG.md b/CHANGELOG.md index 30a0532..bc87c40 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,10 +1,5 @@ ## (unpublished) -* Fix `:enew` not working ([#215](https://github.com/lloeki/ex-mode/pull/215)) - -## 0.18.0 - -* Add Gdefault support ([#191](https://github.com/lloeki/ex-mode/pull/191)) * Add :sort command ([#190](https://github.com/lloeki/ex-mode/pull/190)) ## 0.17.0 diff --git a/README.md b/README.md old mode 100755 new mode 100644 index ea4a068..83f5566 --- a/README.md +++ b/README.md @@ -30,34 +30,7 @@ atom.packages.onDidActivatePackage (pack) -> Ex.registerAlias 'Wq', 'wq' ``` -## Existing commands - -This is the baseline list of commands supported in `ex-mode`. - -| Command | Operation | -| --------------------------------------- | ---------------------------------- | -| `q/quit/tabc/tabclose` | Close active tab | -| `qall/quitall` | Close all tabs | -| `tabe/tabedit/tabnew` | Open new tab | -| `e/edit/tabe/tabedit/tabnew ` | Edit given file | -| `tabn/tabnext` | Go to next tab | -| `tabp/tabprevious` | Go to previous tab | -| `tabo/tabonly` | Close other tabs | -| `w/write` | Save active tab | -| `w/write/saveas ` | Save as | -| `wall/wa` | Save all tabs | -| `sp/split` | Split window | -| `sp/split ` | Open file in split window | -| `s/substitute` | Substitute regular expression in active line | -| `vsp/vsplit` | Vertical split window | -| `vsp/vsplit ` | Open file in vertical split window | -| `delete` | Cut active line | -| `yank` | Copy active line | -| `set ` | Set options | -| `sort` | Sort all lines in file | -| `sort ` | Sort lines in line range | - -See `lib/ex.coffee` for the implementations of these commands. Contributions are very welcome! +See `lib/ex.coffee` for some examples commands. Contributions are very welcome! ## Status @@ -66,3 +39,5 @@ Groundwork is done. More ex commands are easy to add and will be coming as time ## License MIT + + diff --git a/keymaps/ex-mode.cson b/keymaps/ex-mode.cson index a071efd..d857ccb 100644 --- a/keymaps/ex-mode.cson +++ b/keymaps/ex-mode.cson @@ -5,12 +5,12 @@ # Below is a basic keybinding which registers on all platforms by applying to # the root workspace element. + # For more detailed documentation see # https://atom.io/docs/latest/advanced/keymaps 'atom-text-editor.vim-mode-plus:not(.insert-mode)': ':': 'ex-mode:open' 'atom-text-editor.ex-mode-editor': 'ctrl-c': 'ex-mode:close' - 'ctrl-[': 'ex-mode:close' 'atom-text-editor.vim-mode:not(.insert-mode)': ':': 'ex-mode:open' diff --git a/lib/ex-mode.coffee b/lib/ex-mode.coffee index d8fc766..5274e0a 100644 --- a/lib/ex-mode.coffee +++ b/lib/ex-mode.coffee @@ -50,13 +50,3 @@ module.exports = ExMode = description: 'when splitting, split from right' type: 'boolean' default: 'false' - gdefault: - title: 'Gdefault' - description: 'When on, the ":substitute" flag \'g\' is default on' - type: 'boolean' - default: 'false' - onlyCloseBuffers: - title: 'Only close buffers' - description: 'When on, quitall only closes all buffers, not entire Atom instance' - type: 'boolean' - default: 'false' diff --git a/lib/ex.coffee b/lib/ex.coffee index b262023..a1e5ad3 100644 --- a/lib/ex.coffee +++ b/lib/ex.coffee @@ -3,7 +3,6 @@ CommandError = require './command-error' fs = require 'fs-plus' VimOption = require './vim-option' _ = require 'underscore-plus' -atom defer = () -> deferred = {} @@ -19,7 +18,7 @@ trySave = (func) -> try response = func() - + if response instanceof Promise response.then -> deferred.resolve() @@ -133,11 +132,7 @@ class Ex atom.workspace.getActivePane().destroyActiveItem() quitall: -> - if !atom.config.get('ex-mode.onlyCloseBuffers') - atom.close() - else - atom.workspace.getTextEditors().forEach (editor) -> - editor.destroy() + atom.close() q: => @quit() @@ -211,7 +206,9 @@ class Ex e: (args) => @edit(args) enew: -> - atom.workspace.open() + buffer = atom.workspace.getActiveTextEditor().buffer + buffer.setPath(undefined) + buffer.load() write: ({ range, args, editor, saveas }) -> saveas ?= false @@ -229,30 +226,29 @@ class Ex deferred = defer() editor = atom.workspace.getActiveTextEditor() - - # Case 1; path is provided + saved = false if filePath.length isnt 0 - fullPath = getFullPath filePath + fullPath = getFullPath(filePath) + if editor.getPath()? and (not fullPath? or editor.getPath() == fullPath) + if saveas + throw new CommandError("Argument required") + else + # Use editor.save when no path is given or the path to the file is given + trySave(-> editor.save()).then(deferred.resolve) + saved = true + else if not fullPath? + fullPath = atom.showSaveDialogSync() - # Only write when it does not exist or we have a force flag set. - if force or not fs.existsSync(fullPath) - editor.saveAs(fullPath) - return deferred.promise + if not saved and fullPath? + if not force and fs.existsSync(fullPath) + throw new CommandError("File exists (add ! to override)") + if saveas or editor.getFileName() == null + editor = atom.workspace.getActiveTextEditor() + trySave(-> editor.saveAs(fullPath, editor)).then(deferred.resolve) + else + trySave(-> saveAs(fullPath, editor)).then(deferred.resolve) - throw new CommandError("File exists (add ! to override)") - - # Case 2; no path provided, call editor save. - editor = atom.workspace.getActiveTextEditor() - - # Does the current buffer exist? - if editor.getPath()? and fs.existsSync(editor.getPath()) - trySave(-> editor.save()).then(deferred.promise) - else - # Cant see what the better API is but Pane.saveActiveItemAs() is the only call - # I could find that states it will ask the user. - trySave(-> atom.workspace.getActivePane().saveActiveItemAs()).then(deferred.promise) - - return deferred.promise + deferred.promise wall: -> atom.workspace.saveAll() @@ -365,9 +361,6 @@ class Ex try flagsObj = {} flags.split('').forEach((flag) -> flagsObj[flag] = true) - # gdefault option - if atom.config.get('ex-mode.gdefault') - flagsObj.g = !flagsObj.g patternRE = getSearchTerm(pattern, flagsObj) catch e if e.message.indexOf('Invalid flags supplied to RegExp constructor') is 0 diff --git a/lib/vim-option.coffee b/lib/vim-option.coffee index 223395f..ddf4688 100644 --- a/lib/vim-option.coffee +++ b/lib/vim-option.coffee @@ -56,10 +56,4 @@ class VimOption noscs: => @nosmartcase() - gdefault: => - atom.config.set("ex-mode.gdefault", true) - - nogdefault: => - atom.config.set("ex-mode.gdefault", false) - module.exports = VimOption diff --git a/package.json b/package.json index cda6de9..1b23c20 100644 --- a/package.json +++ b/package.json @@ -1,7 +1,7 @@ { "name": "ex-mode", "main": "./lib/ex-mode", - "version": "0.18.0", + "version": "0.17.0", "description": "Ex for Atom's vim-mode", "activationCommands": { "atom-workspace": "ex-mode:open" diff --git a/spec/ex-commands-spec.coffee b/spec/ex-commands-spec.coffee index 29f211e..2eb252a 100644 --- a/spec/ex-commands-spec.coffee +++ b/spec/ex-commands-spec.coffee @@ -715,17 +715,6 @@ describe "the commands", -> submitNormalModeInputText(':%substitute/abc/ghi/ig') expect(editor.getText()).toEqual('ghiaghi\ndefdDEF\nghiaghi') - it "set gdefault option", -> - openEx() - atom.config.set('ex-mode.gdefault', true) - submitNormalModeInputText(':substitute/a/x') - expect(editor.getText()).toEqual('xbcxABC\ndefdDEF\nabcaABC') - - atom.commands.dispatch(editorElement, 'ex-mode:open') - atom.config.set('ex-mode.gdefault', true) - submitNormalModeInputText(':substitute/a/x/g') - expect(editor.getText()).toEqual('xbcaABC\ndefdDEF\nabcaABC') - describe ":yank", -> beforeEach -> editor.setText('abc\ndef\nghi\njkl') @@ -955,14 +944,6 @@ describe "the commands", -> submitNormalModeInputText(':set nosmartcase') expect(atom.config.get('vim-mode.useSmartcaseForSearch')).toBe(false) - it "sets (no)gdefault", -> - openEx() - submitNormalModeInputText(':set gdefault') - expect(atom.config.get('ex-mode.gdefault')).toBe(true) - atom.commands.dispatch(editorElement, 'ex-mode:open') - submitNormalModeInputText(':set nogdefault') - expect(atom.config.get('ex-mode.gdefault')).toBe(false) - describe "aliases", -> it "calls the aliased function without arguments", -> ExClass.registerAlias('W', 'w')