implement go-parent command

jh-changes
Shin'ya Ueoka 7 years ago
parent 0be9776cb3
commit e9863299ab
  1. 2
      README.md
  2. 1
      src/background/keys.js
  3. 2
      src/content/index.js
  4. 21
      src/content/navigates.js
  5. 1
      src/operations/index.js
  6. 12
      test/content/navigates.test.js

@ -39,7 +39,7 @@ Firefox by WebExtensions API.
- [ ] navigations
- [ ] yank/paste page
- [x] pagenation
- [ ] open parent page
- [x] open parent page
- [ ] hints
- [x] open a link
- [ ] open a link in new tab

@ -32,6 +32,7 @@ const defaultKeymap = {
'L': { type: operations.NAVIGATE_HISTORY_NEXT },
'[[': { type: operations.NAVIGATE_LINK_PREV },
']]': { type: operations.NAVIGATE_LINK_NEXT },
'gu': { type: operations.NAVIGATE_PARENT },
};
const asKeymapChars = (keys) => {

@ -43,6 +43,8 @@ const execOperation = (operation) => {
return navigates.linkPrev(window);
case operations.NAVIGATE_LINK_NEXT:
return navigates.linkNext(window);
case operations.NAVIGATE_PARENT:
return navigates.parent(window);
}
};

@ -44,4 +44,23 @@ const linkNext = (win) => {
}
};
export { historyPrev, historyNext, linkPrev, linkNext };
const parent = (win) => {
let loc = win.location;
if (loc.hash !== '') {
loc.hash = '';
return;
} else if (loc.search !== '') {
loc.search = '';
return;
}
const basenamePattern = /\/[^/]+$/;
const lastDirPattern = /\/[^/]+\/$/;
if (basenamePattern.test(loc.pathname)) {
loc.pathname = loc.pathname.replace(basenamePattern, '/');
} else if (lastDirPattern.test(loc.pathname)) {
loc.pathname = loc.pathname.replace(lastDirPattern, '/');
}
};
export { historyPrev, historyNext, linkPrev, linkNext, parent };

@ -15,6 +15,7 @@ export default {
NAVIGATE_HISTORY_NEXT: 'navigate.history.next',
NAVIGATE_LINK_PREV: 'navigate.link.prev',
NAVIGATE_LINK_NEXT: 'navigate.link.next',
NAVIGATE_PARENT: 'navigate.parent',
// Background
TABS_CLOSE: 'tabs.close',

@ -2,9 +2,6 @@ import { expect } from "chai";
import * as navigates from '../../src/content/navigates';
describe('navigates module', () => {
beforeEach(() => {
});
describe('#linkPrev', () => {
it('clicks prev link by text content', (done) => {
document.body.innerHTML = '<a href="#dummy">xprevx</a> <a href="#prev">go to prev</a>';
@ -45,6 +42,15 @@ describe('navigates module', () => {
}, 0);
});
});
describe('#parent', () => {
// NOTE: not able to test location
it('removes hash', () => {
window.location.hash = "#section-1";
navigates.parent(window);
expect(document.location.hash).to.be.empty;
});
});
});