From 6594841b1d1f082107bdb1489f02d3c99cff8ffa Mon Sep 17 00:00:00 2001
From: Shin'ya Ueoka <ueokande@i-beam.org>
Date: Wed, 4 Oct 2017 21:30:57 +0900
Subject: use createStore short-hand method
---
src/content/index.js | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
(limited to 'src/content')
diff --git a/src/content/index.js b/src/content/index.js
index 655bea4..ea08982 100644
--- a/src/content/index.js
+++ b/src/content/index.js
@@ -3,16 +3,16 @@ import * as consoleFrames from './console-frames';
import * as scrolls from '../content/scrolls';
import * as navigates from '../content/navigates';
import * as followActions from '../actions/follow';
-import * as store from '../store';
+import { createStore } from '../store';
import ContentInputComponent from '../components/content-input';
import FollowComponent from '../components/follow';
import followReducer from '../reducers/follow';
import operations from '../operations';
import messages from './messages';
-const followStore = store.createStore(followReducer);
-const followComponent = new FollowComponent(window.document.body, followStore);
-followStore.subscribe(() => {
+const store = createStore(followReducer);
+const followComponent = new FollowComponent(window.document.body, store);
+store.subscribe(() => {
try {
followComponent.update();
} catch (e) {
@@ -39,7 +39,7 @@ const execOperation = (operation) => {
case operations.SCROLL_END:
return scrolls.scrollRight(window);
case operations.FOLLOW_START:
- return followStore.dispatch(followActions.enable(false));
+ return store.dispatch(followActions.enable(false));
case operations.NAVIGATE_HISTORY_PREV:
return navigates.historyPrev(window);
case operations.NAVIGATE_HISTORY_NEXT:
--
cgit v1.2.3
From 79a4a805f6be14572b4486ddb79b0ebb98e37690 Mon Sep 17 00:00:00 2001
From: Shin'ya Ueoka <ueokande@i-beam.org>
Date: Wed, 4 Oct 2017 22:01:16 +0900
Subject: single reducer
---
src/components/completion.js | 2 +-
src/components/follow.js | 2 +-
src/content/index.js | 4 ++--
src/pages/console.js | 7 +++----
src/reducers/completion.js | 2 +-
src/reducers/index.js | 6 ++++++
6 files changed, 14 insertions(+), 9 deletions(-)
(limited to 'src/content')
diff --git a/src/components/completion.js b/src/components/completion.js
index 489061c..d1fdd06 100644
--- a/src/components/completion.js
+++ b/src/components/completion.js
@@ -6,7 +6,7 @@ export default class Completion {
}
update() {
- let state = this.store.getState();
+ let state = this.store.getState().completion;
if (JSON.stringify(this.prevState) === JSON.stringify(state)) {
return;
}
diff --git a/src/components/follow.js b/src/components/follow.js
index 4fe4c58..d2d3902 100644
--- a/src/components/follow.js
+++ b/src/components/follow.js
@@ -44,7 +44,7 @@ export default class FollowComponent {
update() {
let prevState = this.state;
- this.state = this.store.getState();
+ this.state = this.store.getState().follow;
if (!prevState.enabled && this.state.enabled) {
this.create();
} else if (prevState.enabled && !this.state.enabled) {
diff --git a/src/content/index.js b/src/content/index.js
index ea08982..38ad837 100644
--- a/src/content/index.js
+++ b/src/content/index.js
@@ -6,11 +6,11 @@ import * as followActions from '../actions/follow';
import { createStore } from '../store';
import ContentInputComponent from '../components/content-input';
import FollowComponent from '../components/follow';
-import followReducer from '../reducers/follow';
+import reducers from '../reducers';
import operations from '../operations';
import messages from './messages';
-const store = createStore(followReducer);
+const store = createStore(reducers);
const followComponent = new FollowComponent(window.document.body, store);
store.subscribe(() => {
try {
diff --git a/src/pages/console.js b/src/pages/console.js
index 4ce9e47..0e152d3 100644
--- a/src/pages/console.js
+++ b/src/pages/console.js
@@ -2,11 +2,11 @@ import './console.scss';
import messages from '../content/messages';
import CompletionComponent from '../components/completion';
import ConsoleComponent from '../components/console';
-import completionReducer from '../reducers/completion';
+import reducers from '../reducers';
import { createStore } from '../store';
import * as completionActions from '../actions/completion';
-const store = createStore(completionReducer);
+const store = createStore(reducers);
let completionComponent = null;
let consoleComponent = null;
let prevState = {};
@@ -15,14 +15,13 @@ window.addEventListener('load', () => {
let wrapper = document.querySelector('#vimvixen-console-completion');
completionComponent = new CompletionComponent(wrapper, store);
- // TODO use root root store instead of store
consoleComponent = new ConsoleComponent(document.body, store);
});
store.subscribe(() => {
completionComponent.update();
- let state = store.getState();
+ let state = store.getState().completion;
if (state.groupSelection >= 0) {
let item = state.groups[state.groupSelection].items[state.itemSelection];
diff --git a/src/reducers/completion.js b/src/reducers/completion.js
index a8a6444..878aeac 100644
--- a/src/reducers/completion.js
+++ b/src/reducers/completion.js
@@ -63,6 +63,6 @@ export default function reducer(state = defaultState, action = {}) {
});
}
default:
- return defaultState;
+ return state;
}
}
diff --git a/src/reducers/index.js b/src/reducers/index.js
index 9beb81c..db17edf 100644
--- a/src/reducers/index.js
+++ b/src/reducers/index.js
@@ -1,11 +1,15 @@
import inputReducer from '../reducers/input';
import consoleReducer from '../reducers/console';
import settingReducer from '../reducers/setting';
+import followReducer from '../reducers/follow';
+import completionReducer from '../reducers/completion';
const defaultState = {
input: inputReducer(undefined, {}),
console: consoleReducer(undefined, {}),
setting: settingReducer(undefined, {}),
+ follow: followReducer(undefined, {}),
+ completion: completionReducer(undefined, {}),
};
export default function reducer(state = defaultState, action = {}) {
@@ -13,5 +17,7 @@ export default function reducer(state = defaultState, action = {}) {
input: inputReducer(state.input, action),
console: consoleReducer(state.console, action),
setting: settingReducer(state.setting, action),
+ follow: followReducer(state.follow, action),
+ completion: completionReducer(state.completion, action),
});
}
--
cgit v1.2.3
From 5fb3de3263357b3b42a0d1a67a0b9153e4c34260 Mon Sep 17 00:00:00 2001
From: Shin'ya Ueoka <ueokande@i-beam.org>
Date: Wed, 4 Oct 2017 22:05:29 +0900
Subject: move operations
---
src/actions/operation.js | 2 +-
src/content/index.js | 2 +-
src/operations/index.js | 38 --------------------------------------
src/shared/operations.js | 38 ++++++++++++++++++++++++++++++++++++++
src/shared/validators/setting.js | 2 +-
5 files changed, 41 insertions(+), 41 deletions(-)
delete mode 100644 src/operations/index.js
create mode 100644 src/shared/operations.js
(limited to 'src/content')
diff --git a/src/actions/operation.js b/src/actions/operation.js
index 5b7f127..3ed7f05 100644
--- a/src/actions/operation.js
+++ b/src/actions/operation.js
@@ -1,4 +1,4 @@
-import operations from '../operations';
+import operations from '../shared/operations';
import messages from '../content/messages';
import * as consoleActions from './console';
import * as tabs from '../background/tabs';
diff --git a/src/content/index.js b/src/content/index.js
index 38ad837..a9a50be 100644
--- a/src/content/index.js
+++ b/src/content/index.js
@@ -7,7 +7,7 @@ import { createStore } from '../store';
import ContentInputComponent from '../components/content-input';
import FollowComponent from '../components/follow';
import reducers from '../reducers';
-import operations from '../operations';
+import operations from '../shared/operations';
import messages from './messages';
const store = createStore(reducers);
diff --git a/src/operations/index.js b/src/operations/index.js
deleted file mode 100644
index b68f59d..0000000
--- a/src/operations/index.js
+++ /dev/null
@@ -1,38 +0,0 @@
-export default {
- // Command
- COMMAND_SHOW: 'command.show',
- COMMAND_SHOW_OPEN: 'command.show.open',
- COMMAND_SHOW_TABOPEN: 'command.show.tabopen',
- COMMAND_SHOW_BUFFER: 'command.show.buffer',
-
- // Scrolls
- SCROLL_LINES: 'scroll.lines',
- SCROLL_PAGES: 'scroll.pages',
- SCROLL_TOP: 'scroll.top',
- SCROLL_BOTTOM: 'scroll.bottom',
- SCROLL_HOME: 'scroll.home',
- SCROLL_END: 'scroll.end',
-
- // Follows
- FOLLOW_START: 'follow.start',
-
- // Navigations
- NAVIGATE_HISTORY_PREV: 'navigate.history.prev',
- NAVIGATE_HISTORY_NEXT: 'navigate.history.next',
- NAVIGATE_LINK_PREV: 'navigate.link.prev',
- NAVIGATE_LINK_NEXT: 'navigate.link.next',
- NAVIGATE_PARENT: 'navigate.parent',
- NAVIGATE_ROOT: 'navigate.root',
-
- // Tabs
- TAB_CLOSE: 'tabs.close',
- TAB_REOPEN: 'tabs.reopen',
- TAB_PREV: 'tabs.prev',
- TAB_NEXT: 'tabs.next',
- TAB_RELOAD: 'tabs.reload',
-
- // Zooms
- ZOOM_IN: 'zoom.in',
- ZOOM_OUT: 'zoom.out',
- ZOOM_NEUTRAL: 'zoom.neutral',
-};
diff --git a/src/shared/operations.js b/src/shared/operations.js
new file mode 100644
index 0000000..b68f59d
--- /dev/null
+++ b/src/shared/operations.js
@@ -0,0 +1,38 @@
+export default {
+ // Command
+ COMMAND_SHOW: 'command.show',
+ COMMAND_SHOW_OPEN: 'command.show.open',
+ COMMAND_SHOW_TABOPEN: 'command.show.tabopen',
+ COMMAND_SHOW_BUFFER: 'command.show.buffer',
+
+ // Scrolls
+ SCROLL_LINES: 'scroll.lines',
+ SCROLL_PAGES: 'scroll.pages',
+ SCROLL_TOP: 'scroll.top',
+ SCROLL_BOTTOM: 'scroll.bottom',
+ SCROLL_HOME: 'scroll.home',
+ SCROLL_END: 'scroll.end',
+
+ // Follows
+ FOLLOW_START: 'follow.start',
+
+ // Navigations
+ NAVIGATE_HISTORY_PREV: 'navigate.history.prev',
+ NAVIGATE_HISTORY_NEXT: 'navigate.history.next',
+ NAVIGATE_LINK_PREV: 'navigate.link.prev',
+ NAVIGATE_LINK_NEXT: 'navigate.link.next',
+ NAVIGATE_PARENT: 'navigate.parent',
+ NAVIGATE_ROOT: 'navigate.root',
+
+ // Tabs
+ TAB_CLOSE: 'tabs.close',
+ TAB_REOPEN: 'tabs.reopen',
+ TAB_PREV: 'tabs.prev',
+ TAB_NEXT: 'tabs.next',
+ TAB_RELOAD: 'tabs.reload',
+
+ // Zooms
+ ZOOM_IN: 'zoom.in',
+ ZOOM_OUT: 'zoom.out',
+ ZOOM_NEUTRAL: 'zoom.neutral',
+};
diff --git a/src/shared/validators/setting.js b/src/shared/validators/setting.js
index caba5cc..4dc35ff 100644
--- a/src/shared/validators/setting.js
+++ b/src/shared/validators/setting.js
@@ -1,4 +1,4 @@
-import operations from '../../operations';
+import operations from '../operations';
const VALID_TOP_KEYS = ['keymaps', 'search'];
const VALID_OPERATION_VALUES = Object.keys(operations).map((key) => {
--
cgit v1.2.3
From 32168a94e07478325a53779513533b76a6ef2c18 Mon Sep 17 00:00:00 2001
From: Shin'ya Ueoka <ueokande@i-beam.org>
Date: Thu, 5 Oct 2017 20:14:58 +0900
Subject: fix imports in src
---
src/actions/command.js | 4 ++--
src/actions/completion.js | 2 +-
src/actions/console.js | 2 +-
src/actions/follow.js | 2 +-
src/actions/input.js | 2 +-
src/actions/operation.js | 8 ++++----
src/actions/setting.js | 6 +++---
src/background/index.js | 14 +++++++-------
src/components/background-input.js | 4 ++--
src/components/background.js | 12 ++++++------
src/components/console.js | 4 ++--
src/components/content-input.js | 2 +-
src/components/follow.js | 8 ++++----
src/components/setting.js | 4 ++--
src/content/index.js | 16 ++++++++--------
src/pages/console.js | 12 ++++++------
src/pages/settings.js | 6 +++---
src/reducers/completion.js | 2 +-
src/reducers/console.js | 2 +-
src/reducers/follow.js | 2 +-
src/reducers/index.js | 10 +++++-----
src/reducers/input.js | 2 +-
src/reducers/setting.js | 2 +-
src/shared/validators/setting.js | 2 +-
webpack.config.js | 2 +-
25 files changed, 66 insertions(+), 66 deletions(-)
(limited to 'src/content')
diff --git a/src/actions/command.js b/src/actions/command.js
index f578afd..a40cc97 100644
--- a/src/actions/command.js
+++ b/src/actions/command.js
@@ -1,5 +1,5 @@
-import * as tabs from '../background/tabs';
-import * as histories from '../background/histories';
+import * as tabs from 'background/tabs';
+import * as histories from 'background/histories';
import * as consoleActions from './console';
const normalizeUrl = (string, searchConfig) => {
diff --git a/src/actions/completion.js b/src/actions/completion.js
index 1ffb025..733f516 100644
--- a/src/actions/completion.js
+++ b/src/actions/completion.js
@@ -1,4 +1,4 @@
-import actions from '../actions';
+import actions from 'actions';
const setItems = (groups) => {
return {
diff --git a/src/actions/console.js b/src/actions/console.js
index e0ec631..ba1fbeb 100644
--- a/src/actions/console.js
+++ b/src/actions/console.js
@@ -1,4 +1,4 @@
-import actions from '../actions';
+import actions from 'actions';
const showCommand = (text) => {
return {
diff --git a/src/actions/follow.js b/src/actions/follow.js
index 7ab689e..708cd95 100644
--- a/src/actions/follow.js
+++ b/src/actions/follow.js
@@ -1,4 +1,4 @@
-import actions from '../actions';
+import actions from 'actions';
const enable = (newTab) => {
return {
diff --git a/src/actions/input.js b/src/actions/input.js
index 67788dd..61acb76 100644
--- a/src/actions/input.js
+++ b/src/actions/input.js
@@ -1,4 +1,4 @@
-import actions from '../actions';
+import actions from 'actions';
const asKeymapChars = (key, ctrl) => {
if (ctrl) {
diff --git a/src/actions/operation.js b/src/actions/operation.js
index 3ed7f05..97bcf45 100644
--- a/src/actions/operation.js
+++ b/src/actions/operation.js
@@ -1,8 +1,8 @@
-import operations from '../shared/operations';
-import messages from '../content/messages';
+import operations from 'shared/operations';
+import messages from 'content/messages';
import * as consoleActions from './console';
-import * as tabs from '../background/tabs';
-import * as zooms from '../background/zooms';
+import * as tabs from 'background/tabs';
+import * as zooms from 'background/zooms';
const exec = (operation, tab) => {
switch (operation.type) {
diff --git a/src/actions/setting.js b/src/actions/setting.js
index 7898f06..2a47608 100644
--- a/src/actions/setting.js
+++ b/src/actions/setting.js
@@ -1,6 +1,6 @@
-import actions from '../actions';
-import messages from '../content/messages';
-import DefaultSettings from '../shared/default-settings';
+import actions from 'actions';
+import messages from 'content/messages';
+import DefaultSettings from 'shared/default-settings';
const load = () => {
return browser.storage.local.get('settings').then((value) => {
diff --git a/src/background/index.js b/src/background/index.js
index 9a1adc6..8dc55cb 100644
--- a/src/background/index.js
+++ b/src/background/index.js
@@ -1,10 +1,10 @@
-import * as consoleActions from '../actions/console';
-import * as settingsActions from '../actions/setting';
-import BackgroundComponent from '../components/background';
-import BackgroundInputComponent from '../components/background-input';
-import reducers from '../reducers';
-import messages from '../content/messages';
-import { createStore } from '../store';
+import * as consoleActions from 'actions/console';
+import * as settingsActions from 'actions/setting';
+import BackgroundComponent from 'components/background';
+import BackgroundInputComponent from 'components/background-input';
+import reducers from 'reducers';
+import messages from 'content/messages';
+import { createStore } from 'store';
const store = createStore(reducers, (e, sender) => {
console.error('Vim-Vixen:', e);
diff --git a/src/components/background-input.js b/src/components/background-input.js
index 4735d5a..bd6ecf9 100644
--- a/src/components/background-input.js
+++ b/src/components/background-input.js
@@ -1,5 +1,5 @@
-import * as inputActions from '../actions/input';
-import * as operationActions from '../actions/operation';
+import * as inputActions from 'actions/input';
+import * as operationActions from 'actions/operation';
export default class BackgroundInputComponent {
constructor(store) {
diff --git a/src/components/background.js b/src/components/background.js
index 0585a04..08d5115 100644
--- a/src/components/background.js
+++ b/src/components/background.js
@@ -1,9 +1,9 @@
-import messages from '../content/messages';
-import * as commandActions from '../actions/command';
-import * as consoleActions from '../actions/console';
-import * as inputActions from '../actions/input';
-import * as settingsActions from '../actions/setting';
-import * as tabActions from '../actions/tab';
+import messages from 'content/messages';
+import * as commandActions from 'actions/command';
+import * as consoleActions from 'actions/console';
+import * as inputActions from 'actions/input';
+import * as settingsActions from 'actions/setting';
+import * as tabActions from 'actions/tab';
export default class BackgroundComponent {
constructor(store) {
diff --git a/src/components/console.js b/src/components/console.js
index 9580dcf..25b135c 100644
--- a/src/components/console.js
+++ b/src/components/console.js
@@ -1,5 +1,5 @@
-import messages from '../content/messages';
-import * as completionActions from '../actions/completion';
+import messages from 'content/messages';
+import * as completionActions from 'actions/completion';
export default class ConsoleComponent {
constructor(wrapper, store) {
diff --git a/src/components/content-input.js b/src/components/content-input.js
index 10c785b..38d57fd 100644
--- a/src/components/content-input.js
+++ b/src/components/content-input.js
@@ -1,4 +1,4 @@
-import messages from '../content/messages';
+import messages from 'content/messages';
export default class ContentInputComponent {
constructor(target) {
diff --git a/src/components/follow.js b/src/components/follow.js
index d2d3902..9221759 100644
--- a/src/components/follow.js
+++ b/src/components/follow.js
@@ -1,7 +1,7 @@
-import * as followActions from '../actions/follow';
-import messages from '../content/messages';
-import Hint from '../content/hint';
-import HintKeyProducer from '../content/hint-key-producer';
+import * as followActions from 'actions/follow';
+import messages from 'content/messages';
+import Hint from 'content/hint';
+import HintKeyProducer from 'content/hint-key-producer';
const DEFAULT_HINT_CHARSET = 'abcdefghijklmnopqrstuvwxyz';
diff --git a/src/components/setting.js b/src/components/setting.js
index 1f3b3fe..c2f99b6 100644
--- a/src/components/setting.js
+++ b/src/components/setting.js
@@ -1,5 +1,5 @@
-import * as settingActions from '../actions/setting';
-import { validate } from '../shared/validators/setting';
+import * as settingActions from 'actions/setting';
+import { validate } from 'shared/validators/setting';
export default class SettingComponent {
constructor(wrapper, store) {
diff --git a/src/content/index.js b/src/content/index.js
index a9a50be..31b37cf 100644
--- a/src/content/index.js
+++ b/src/content/index.js
@@ -1,13 +1,13 @@
import './console-frame.scss';
import * as consoleFrames from './console-frames';
-import * as scrolls from '../content/scrolls';
-import * as navigates from '../content/navigates';
-import * as followActions from '../actions/follow';
-import { createStore } from '../store';
-import ContentInputComponent from '../components/content-input';
-import FollowComponent from '../components/follow';
-import reducers from '../reducers';
-import operations from '../shared/operations';
+import * as scrolls from 'content/scrolls';
+import * as navigates from 'content/navigates';
+import * as followActions from 'actions/follow';
+import { createStore } from 'store';
+import ContentInputComponent from 'components/content-input';
+import FollowComponent from 'components/follow';
+import reducers from 'reducers';
+import operations from 'shared/operations';
import messages from './messages';
const store = createStore(reducers);
diff --git a/src/pages/console.js b/src/pages/console.js
index 0e152d3..a4536ec 100644
--- a/src/pages/console.js
+++ b/src/pages/console.js
@@ -1,10 +1,10 @@
import './console.scss';
-import messages from '../content/messages';
-import CompletionComponent from '../components/completion';
-import ConsoleComponent from '../components/console';
-import reducers from '../reducers';
-import { createStore } from '../store';
-import * as completionActions from '../actions/completion';
+import messages from 'content/messages';
+import CompletionComponent from 'components/completion';
+import ConsoleComponent from 'components/console';
+import reducers from 'reducers';
+import { createStore } from 'store';
+import * as completionActions from 'actions/completion';
const store = createStore(reducers);
let completionComponent = null;
diff --git a/src/pages/settings.js b/src/pages/settings.js
index 076d23d..6e25e6f 100644
--- a/src/pages/settings.js
+++ b/src/pages/settings.js
@@ -1,7 +1,7 @@
import './settings.scss';
-import SettingComponent from '../components/setting';
-import settingReducer from '../reducers/setting';
-import { createStore } from '../store';
+import SettingComponent from 'components/setting';
+import settingReducer from 'reducers/setting';
+import { createStore } from 'store';
const store = createStore(settingReducer);
let settingComponent = null;
diff --git a/src/reducers/completion.js b/src/reducers/completion.js
index 878aeac..f85a500 100644
--- a/src/reducers/completion.js
+++ b/src/reducers/completion.js
@@ -1,4 +1,4 @@
-import actions from '../actions';
+import actions from 'actions';
const defaultState = {
groupSelection: -1,
diff --git a/src/reducers/console.js b/src/reducers/console.js
index 5c49c3b..3e63672 100644
--- a/src/reducers/console.js
+++ b/src/reducers/console.js
@@ -1,4 +1,4 @@
-import actions from '../actions';
+import actions from 'actions';
const defaultState = {
errorShown: false,
diff --git a/src/reducers/follow.js b/src/reducers/follow.js
index 136b367..a2397b4 100644
--- a/src/reducers/follow.js
+++ b/src/reducers/follow.js
@@ -1,4 +1,4 @@
-import actions from '../actions';
+import actions from 'actions';
const defaultState = {
enabled: false,
diff --git a/src/reducers/index.js b/src/reducers/index.js
index db17edf..5a4f14b 100644
--- a/src/reducers/index.js
+++ b/src/reducers/index.js
@@ -1,8 +1,8 @@
-import inputReducer from '../reducers/input';
-import consoleReducer from '../reducers/console';
-import settingReducer from '../reducers/setting';
-import followReducer from '../reducers/follow';
-import completionReducer from '../reducers/completion';
+import inputReducer from 'reducers/input';
+import consoleReducer from 'reducers/console';
+import settingReducer from 'reducers/setting';
+import followReducer from 'reducers/follow';
+import completionReducer from 'reducers/completion';
const defaultState = {
input: inputReducer(undefined, {}),
diff --git a/src/reducers/input.js b/src/reducers/input.js
index 8be701e..2e4bcd8 100644
--- a/src/reducers/input.js
+++ b/src/reducers/input.js
@@ -1,4 +1,4 @@
-import actions from '../actions';
+import actions from 'actions';
const defaultState = {
keys: '',
diff --git a/src/reducers/setting.js b/src/reducers/setting.js
index 735d4fb..7326ed7 100644
--- a/src/reducers/setting.js
+++ b/src/reducers/setting.js
@@ -1,4 +1,4 @@
-import actions from '../actions';
+import actions from 'actions';
const defaultState = {
settings: {}
diff --git a/src/shared/validators/setting.js b/src/shared/validators/setting.js
index 4dc35ff..5039ec2 100644
--- a/src/shared/validators/setting.js
+++ b/src/shared/validators/setting.js
@@ -1,4 +1,4 @@
-import operations from '../operations';
+import operations from 'shared/operations';
const VALID_TOP_KEYS = ['keymaps', 'search'];
const VALID_OPERATION_VALUES = Object.keys(operations).map((key) => {
diff --git a/webpack.config.js b/webpack.config.js
index e4d5a17..3d4ef03 100644
--- a/webpack.config.js
+++ b/webpack.config.js
@@ -40,7 +40,7 @@ module.exports = {
resolve: {
extensions: [ '.js' ],
- modules: [path.join(__dirname), 'node_modules']
+ modules: [path.join(__dirname, 'src'), 'node_modules']
},
plugins: [
--
cgit v1.2.3
From 10ad62e60698c5d53ffcf58ae6abd182f7d3fc9c Mon Sep 17 00:00:00 2001
From: Shin'ya Ueoka <ueokande@i-beam.org>
Date: Fri, 6 Oct 2017 23:03:28 +0900
Subject: console command actions without store
---
src/actions/command.js | 147 -------------------------------------------
src/components/background.js | 12 ++--
src/components/console.js | 6 +-
src/content/messages.js | 2 +-
src/pages/console.js | 1 -
src/shared/commands.js | 143 +++++++++++++++++++++++++++++++++++++++++
6 files changed, 153 insertions(+), 158 deletions(-)
delete mode 100644 src/actions/command.js
create mode 100644 src/shared/commands.js
(limited to 'src/content')
diff --git a/src/actions/command.js b/src/actions/command.js
deleted file mode 100644
index a40cc97..0000000
--- a/src/actions/command.js
+++ /dev/null
@@ -1,147 +0,0 @@
-import * as tabs from 'background/tabs';
-import * as histories from 'background/histories';
-import * as consoleActions from './console';
-
-const normalizeUrl = (string, searchConfig) => {
- try {
- return new URL(string).href;
- } catch (e) {
- if (string.includes('.') && !string.includes(' ')) {
- return 'http://' + string;
- }
- let query = encodeURI(string);
- let template = searchConfig.engines[
- searchConfig.default
- ];
- for (let key in searchConfig.engines) {
- if (string.startsWith(key + ' ')) {
- query = encodeURI(string.replace(key + ' ', ''));
- template = searchConfig.engines[key];
- }
- }
- return template.replace('{}', query);
- }
-};
-
-const openCommand = (url) => {
- return browser.tabs.query({
- active: true, currentWindow: true
- }).then((gotTabs) => {
- if (gotTabs.length > 0) {
- return browser.tabs.update(gotTabs[0].id, { url: url });
- }
- });
-};
-
-const tabopenCommand = (url) => {
- return browser.tabs.create({ url: url });
-};
-
-const bufferCommand = (keywords) => {
- return browser.tabs.query({
- active: true, currentWindow: true
- }).then((gotTabs) => {
- if (gotTabs.length > 0) {
- if (isNaN(keywords)) {
- return tabs.selectByKeyword(gotTabs[0], keywords);
- }
- let index = parseInt(keywords, 10) - 1;
- return tabs.selectAt(index);
- }
- });
-};
-
-const getOpenCompletions = (command, keywords, searchConfig) => {
- return histories.getCompletions(keywords).then((pages) => {
- let historyItems = pages.map((page) => {
- return {
- caption: page.title,
- content: command + ' ' + page.url,
- url: page.url
- };
- });
- let engineNames = Object.keys(searchConfig.engines);
- let engineItems = engineNames.filter(name => name.startsWith(keywords))
- .map(name => ({
- caption: name,
- content: command + ' ' + name
- }));
-
- let completions = [];
- if (engineItems.length > 0) {
- completions.push({
- name: 'Search Engines',
- items: engineItems
- });
- }
- if (historyItems.length > 0) {
- completions.push({
- name: 'History',
- items: historyItems
- });
- }
- return completions;
- });
-};
-
-const doCommand = (name, remaining, settings) => {
- switch (name) {
- case 'o':
- case 'open':
- // TODO use search engined and pass keywords to them
- return openCommand(normalizeUrl(remaining, settings.search));
- case 't':
- case 'tabopen':
- return tabopenCommand(normalizeUrl(remaining, settings.search));
- case 'b':
- case 'buffer':
- return bufferCommand(remaining);
- }
- throw new Error(name + ' command is not defined');
-};
-
-const getCompletions = (command, keywords, settings) => {
- switch (command) {
- case 'o':
- case 'open':
- case 't':
- case 'tabopen':
- return getOpenCompletions(command, keywords, settings.search);
- case 'b':
- case 'buffer':
- return tabs.getCompletions(keywords).then((gotTabs) => {
- let items = gotTabs.map((tab) => {
- return {
- caption: tab.title,
- content: command + ' ' + tab.title,
- url: tab.url,
- icon: tab.favIconUrl
- };
- });
- return [
- {
- name: 'Buffers',
- items: items
- }
- ];
- });
- }
- return Promise.resolve([]);
-};
-
-const exec = (line, settings) => {
- let name = line.split(' ')[0];
- let remaining = line.replace(name + ' ', '');
- return doCommand(name, remaining, settings).then(() => {
- return consoleActions.hide();
- });
-};
-
-const complete = (line, settings) => {
- let command = line.split(' ', 1)[0];
- let keywords = line.replace(command + ' ', '');
- return getCompletions(command, keywords, settings)
- .then(consoleActions.setCompletions);
-};
-
-export { exec, complete };
diff --git a/src/components/background.js b/src/components/background.js
index 08d5115..195cfd9 100644
--- a/src/components/background.js
+++ b/src/components/background.js
@@ -1,9 +1,9 @@
import messages from 'content/messages';
-import * as commandActions from 'actions/command';
import * as consoleActions from 'actions/console';
import * as inputActions from 'actions/input';
import * as settingsActions from 'actions/setting';
import * as tabActions from 'actions/tab';
+import * as commands from 'shared/commands';
export default class BackgroundComponent {
constructor(store) {
@@ -12,7 +12,7 @@ export default class BackgroundComponent {
browser.runtime.onMessage.addListener((message, sender) => {
try {
- this.onMessage(message, sender);
+ return this.onMessage(message, sender);
} catch (e) {
this.store.dispatch(consoleActions.showError(e.message), sender);
}
@@ -47,11 +47,9 @@ export default class BackgroundComponent {
return this.store.dispatch(
consoleActions.hide(), sender);
case messages.CONSOLE_ENTERED:
- return this.store.dispatch(
- commandActions.exec(message.text, this.settings), sender);
- case messages.CONSOLE_CHANGEED:
- return this.store.dispatch(
- commandActions.complete(message.text, this.settings), sender);
+ return commands.exec(message.text, this.settings);
+ case messages.CONSOLE_QUERY_COMPLETIONS:
+ return commands.complete(message.text, this.settings);
case messages.SETTINGS_RELOAD:
this.store.dispatch(settingsActions.load());
}
diff --git a/src/components/console.js b/src/components/console.js
index 25b135c..177cfe5 100644
--- a/src/components/console.js
+++ b/src/components/console.js
@@ -36,7 +36,7 @@ export default class ConsoleComponent {
return browser.runtime.sendMessage({
type: messages.CONSOLE_ENTERED,
text: e.target.value
- });
+ }).then(this.onBlur);
case KeyboardEvent.DOM_VK_TAB:
if (e.shiftKey) {
this.store.dispatch(completionActions.selectPrev());
@@ -63,8 +63,10 @@ export default class ConsoleComponent {
this.prevValue = e.target.value;
return browser.runtime.sendMessage({
- type: messages.CONSOLE_CHANGEED,
+ type: messages.CONSOLE_QUERY_COMPLETIONS,
text: e.target.value
+ }).then((completions) => {
+ this.store.dispatch(completionActions.setItems(completions));
});
}
diff --git a/src/content/messages.js b/src/content/messages.js
index df9fba2..72a566b 100644
--- a/src/content/messages.js
+++ b/src/content/messages.js
@@ -4,7 +4,7 @@ export default {
CONSOLE_BLURRED: 'console.blured',
CONSOLE_ENTERED: 'console.entered',
- CONSOLE_CHANGEED: 'console.changed',
+ CONSOLE_QUERY_COMPLETIONS: 'console.query.completions',
KEYDOWN: 'keydown',
diff --git a/src/pages/console.js b/src/pages/console.js
index a4536ec..4d78826 100644
--- a/src/pages/console.js
+++ b/src/pages/console.js
@@ -39,6 +39,5 @@ browser.runtime.onMessage.addListener((action) => {
if (action.type === messages.STATE_UPDATE) {
let state = action.state.console;
consoleComponent.update(state);
- store.dispatch(completionActions.setItems(state.completions));
}
});
diff --git a/src/shared/commands.js b/src/shared/commands.js
new file mode 100644
index 0000000..b1d8780
--- /dev/null
+++ b/src/shared/commands.js
@@ -0,0 +1,143 @@
+import * as tabs from 'background/tabs';
+import * as histories from 'background/histories';
+
+const normalizeUrl = (string, searchConfig) => {
+ try {
+ return new URL(string).href;
+ } catch (e) {
+ if (string.includes('.') && !string.includes(' ')) {
+ return 'http://' + string;
+ }
+ let query = encodeURI(string);
+ let template = searchConfig.engines[
+ searchConfig.default
+ ];
+ for (let key in searchConfig.engines) {
+ if (string.startsWith(key + ' ')) {
+ query = encodeURI(string.replace(key + ' ', ''));
+ template = searchConfig.engines[key];
+ }
+ }
+ return template.replace('{}', query);
+ }
+};
+
+const openCommand = (url) => {
+ return browser.tabs.query({
+ active: true, currentWindow: true
+ }).then((gotTabs) => {
+ if (gotTabs.length > 0) {
+ return browser.tabs.update(gotTabs[0].id, { url: url });
+ }
+ });
+};
+
+const tabopenCommand = (url) => {
+ return browser.tabs.create({ url: url });
+};
+
+const bufferCommand = (keywords) => {
+ return browser.tabs.query({
+ active: true, currentWindow: true
+ }).then((gotTabs) => {
+ if (gotTabs.length > 0) {
+ if (isNaN(keywords)) {
+ return tabs.selectByKeyword(gotTabs[0], keywords);
+ }
+ let index = parseInt(keywords, 10) - 1;
+ return tabs.selectAt(index);
+ }
+ });
+};
+
+const getOpenCompletions = (command, keywords, searchConfig) => {
+ return histories.getCompletions(keywords).then((pages) => {
+ let historyItems = pages.map((page) => {
+ return {
+ caption: page.title,
+ content: command + ' ' + page.url,
+ url: page.url
+ };
+ });
+ let engineNames = Object.keys(searchConfig.engines);
+ let engineItems = engineNames.filter(name => name.startsWith(keywords))
+ .map(name => ({
+ caption: name,
+ content: command + ' ' + name
+ }));
+
+ let completions = [];
+ if (engineItems.length > 0) {
+ completions.push({
+ name: 'Search Engines',
+ items: engineItems
+ });
+ }
+ if (historyItems.length > 0) {
+ completions.push({
+ name: 'History',
+ items: historyItems
+ });
+ }
+ return completions;
+ });
+};
+
+const doCommand = (name, remaining, settings) => {
+ switch (name) {
+ case 'o':
+ case 'open':
+ // TODO use search engined and pass keywords to them
+ return openCommand(normalizeUrl(remaining, settings.search));
+ case 't':
+ case 'tabopen':
+ return tabopenCommand(normalizeUrl(remaining, settings.search));
+ case 'b':
+ case 'buffer':
+ return bufferCommand(remaining);
+ }
+ throw new Error(name + ' command is not defined');
+};
+
+const getCompletions = (command, keywords, settings) => {
+ switch (command) {
+ case 'o':
+ case 'open':
+ case 't':
+ case 'tabopen':
+ return getOpenCompletions(command, keywords, settings.search);
+ case 'b':
+ case 'buffer':
+ return tabs.getCompletions(keywords).then((gotTabs) => {
+ let items = gotTabs.map((tab) => {
+ return {
+ caption: tab.title,
+ content: command + ' ' + tab.title,
+ url: tab.url,
+ icon: tab.favIconUrl
+ };
+ });
+ return [
+ {
+ name: 'Buffers',
+ items: items
+ }
+ ];
+ });
+ }
+ return Promise.resolve([]);
+};
+
+const exec = (line, settings) => {
+ let name = line.split(' ')[0];
+ let remaining = line.replace(name + ' ', '');
+ return doCommand(name, remaining, settings);
+};
+
+const complete = (line, settings) => {
+ let command = line.split(' ', 1)[0];
+ let keywords = line.replace(command + ' ', '');
+ return getCompletions(command, keywords, settings);
+};
+
+export { exec, complete };
--
cgit v1.2.3
From 4cb17031d11d76275de51e31218fb87359e7d826 Mon Sep 17 00:00:00 2001
From: Shin'ya Ueoka <ueokande@i-beam.org>
Date: Fri, 6 Oct 2017 23:55:52 +0900
Subject: [wip] remove STATE_UPDATE
---
src/actions/operation.js | 20 +++++++++++++-------
src/background/index.js | 9 ---------
src/components/console.js | 4 ++--
src/content/index.js | 13 ++++---------
src/content/messages.js | 4 +++-
src/pages/console.js | 13 +++++++++----
6 files changed, 31 insertions(+), 32 deletions(-)
(limited to 'src/content')
diff --git a/src/actions/operation.js b/src/actions/operation.js
index 97bcf45..295fd4f 100644
--- a/src/actions/operation.js
+++ b/src/actions/operation.js
@@ -1,9 +1,15 @@
import operations from 'shared/operations';
import messages from 'content/messages';
-import * as consoleActions from './console';
import * as tabs from 'background/tabs';
import * as zooms from 'background/zooms';
+const sendConsoleShowCommand = (tab, command) => {
+ return browser.tabs.sendMessage(tab.id, {
+ type: messages.CONSOLE_SHOW_COMMAND,
+ command,
+ });
+};
+
const exec = (operation, tab) => {
switch (operation.type) {
case operations.TAB_CLOSE:
@@ -23,21 +29,21 @@ const exec = (operation, tab) => {
case operations.ZOOM_NEUTRAL:
return zooms.neutral();
case operations.COMMAND_SHOW:
- return consoleActions.showCommand('');
+ return sendConsoleShowCommand(tab, '');
case operations.COMMAND_SHOW_OPEN:
if (operation.alter) {
// alter url
- return consoleActions.showCommand('open ' + tab.url);
+ return sendConsoleShowCommand(tab, 'open ' + tab.url);
}
- return consoleActions.showCommand('open ');
+ return sendConsoleShowCommand(tab, 'open ');
case operations.COMMAND_SHOW_TABOPEN:
if (operation.alter) {
// alter url
- return consoleActions.showCommand('tabopen ' + tab.url);
+ return sendConsoleShowCommand(tab, 'tabopen ' + tab.url);
}
- return consoleActions.showCommand('tabopen ');
+ return sendConsoleShowCommand(tab, 'tabopen ');
case operations.COMMAND_SHOW_BUFFER:
- return consoleActions.showCommand('buffer ');
+ return sendConsoleShowCommand(tab, 'buffer ');
default:
return browser.tabs.sendMessage(tab.id, {
type: messages.CONTENT_OPERATION,
diff --git a/src/background/index.js b/src/background/index.js
index 8dc55cb..05d3553 100644
--- a/src/background/index.js
+++ b/src/background/index.js
@@ -3,7 +3,6 @@ import * as settingsActions from 'actions/setting';
import BackgroundComponent from 'components/background';
import BackgroundInputComponent from 'components/background-input';
import reducers from 'reducers';
-import messages from 'content/messages';
import { createStore } from 'store';
const store = createStore(reducers, (e, sender) => {
@@ -18,13 +17,5 @@ store.subscribe((sender) => {
backgroundComponent.update(sender);
backgroundInputComponent.update(sender);
});
-store.subscribe((sender) => {
- if (sender) {
- return browser.tabs.sendMessage(sender.tab.id, {
- type: messages.STATE_UPDATE,
- state: store.getState()
- });
- }
-});
store.dispatch(settingsActions.load());
diff --git a/src/components/console.js b/src/components/console.js
index 177cfe5..3a7f88b 100644
--- a/src/components/console.js
+++ b/src/components/console.js
@@ -70,8 +70,8 @@ export default class ConsoleComponent {
});
}
- // TODO use store/reducer to update state.
- update(state) {
+ update() {
+ let state = this.store.getState().console;
if (!this.prevState.commandShown && state.commandShown) {
this.showCommand(state.commandText);
} else if (!state.commandShown) {
diff --git a/src/content/index.js b/src/content/index.js
index 31b37cf..b29118d 100644
--- a/src/content/index.js
+++ b/src/content/index.js
@@ -55,17 +55,12 @@ const execOperation = (operation) => {
}
};
-const update = (state) => {
- if (!state.console.commandShown) {
- window.focus();
- consoleFrames.blur(window.document);
- }
-};
-
browser.runtime.onMessage.addListener((action) => {
switch (action.type) {
- case messages.STATE_UPDATE:
- return update(action.state);
+ case messages.CONSOLE_HIDE:
+ window.focus();
+ consoleFrames.blur(window.document);
+ return Promise.resolve();
case messages.CONTENT_OPERATION:
execOperation(action.operation);
return Promise.resolve();
diff --git a/src/content/messages.js b/src/content/messages.js
index 72a566b..0e66fa0 100644
--- a/src/content/messages.js
+++ b/src/content/messages.js
@@ -1,10 +1,12 @@
export default {
- STATE_UPDATE: 'state.update',
CONTENT_OPERATION: 'content.operation',
CONSOLE_BLURRED: 'console.blured',
CONSOLE_ENTERED: 'console.entered',
CONSOLE_QUERY_COMPLETIONS: 'console.query.completions',
+ CONSOLE_SHOW_COMMAND: 'console.show.command',
+ CONSOLE_SHOW_ERROR: 'console.show.error',
+ CONSOLE_HIDE: 'console.hide',
KEYDOWN: 'keydown',
diff --git a/src/pages/console.js b/src/pages/console.js
index 4d78826..4d3dd3f 100644
--- a/src/pages/console.js
+++ b/src/pages/console.js
@@ -4,7 +4,7 @@ import CompletionComponent from 'components/completion';
import ConsoleComponent from 'components/console';
import reducers from 'reducers';
import { createStore } from 'store';
-import * as completionActions from 'actions/completion';
+import * as consoleActions from 'actions/console';
const store = createStore(reducers);
let completionComponent = null;
@@ -20,6 +20,7 @@ window.addEventListener('load', () => {
store.subscribe(() => {
completionComponent.update();
+ consoleComponent.update();
let state = store.getState().completion;
@@ -36,8 +37,12 @@ store.subscribe(() => {
});
browser.runtime.onMessage.addListener((action) => {
- if (action.type === messages.STATE_UPDATE) {
- let state = action.state.console;
- consoleComponent.update(state);
+ switch (action.type) {
+ case messages.CONSOLE_SHOW_COMMAND:
+ return store.dispatch(consoleActions.showCommand(action.command));
+ case messages.CONSOLE_SHOW_ERROR:
+ return store.dispatch(consoleActions.showError(action.command));
+ case messages.CONSOLE_HIDE:
+ return store.dispatch(consoleActions.hide(action.command));
}
});
--
cgit v1.2.3