From c6eb5553d0477f96b5edd48012b4c4ab342f026c Mon Sep 17 00:00:00 2001 From: Shin'ya Ueoka Date: Sun, 22 Oct 2017 17:45:16 +0900 Subject: add setting actions in content --- src/content/components/common/index.js | 4 ++-- src/content/components/common/keymapper.js | 15 ++++++++------- 2 files changed, 10 insertions(+), 9 deletions(-) (limited to 'src/content/components') diff --git a/src/content/components/common/index.js b/src/content/components/common/index.js index db0ac43..5f5531b 100644 --- a/src/content/components/common/index.js +++ b/src/content/components/common/index.js @@ -1,7 +1,7 @@ import InputComponent from './input'; import KeymapperComponent from './keymapper'; import FollowComponent from './follow'; -import * as inputActions from 'content/actions/input'; +import * as settingActions from 'content/actions/setting'; import messages from 'shared/messages'; export default class Common { @@ -40,7 +40,7 @@ export default class Common { browser.runtime.sendMessage({ type: messages.SETTINGS_QUERY, }).then((settings) => { - this.store.dispatch(inputActions.setKeymaps(settings.keymaps)); + this.store.dispatch(settingActions.set(settings)); }); } } diff --git a/src/content/components/common/keymapper.js b/src/content/components/common/keymapper.js index 5070cd8..44d8212 100644 --- a/src/content/components/common/keymapper.js +++ b/src/content/components/common/keymapper.js @@ -11,19 +11,20 @@ export default class KeymapperComponent { } key(key) { - let enabled = this.store.getState().addon.enabled; - this.store.dispatch(inputActions.keyPress(key)); - let input = this.store.getState().input; - let matched = Object.keys(input.keymaps).filter((keyStr) => { + let state = this.store.getState(); + let input = state.input; + let keymaps = state.setting.keymaps; + + let matched = Object.keys(keymaps).filter((keyStr) => { return keyStr.startsWith(input.keys); }); - if (!enabled) { + if (!state.addon.enabled) { // available keymaps are only ADDON_ENABLE and ADDON_TOGGLE_ENABLED if // the addon disabled matched = matched.filter((keys) => { - let type = input.keymaps[keys].type; + let type = keymaps[keys].type; return type === operations.ADDON_ENABLE || type === operations.ADDON_TOGGLE_ENABLED; }); @@ -35,7 +36,7 @@ export default class KeymapperComponent { matched.length === 1 && input.keys !== matched[0]) { return true; } - let operation = input.keymaps[matched]; + let operation = keymaps[matched]; this.store.dispatch(operationActions.exec(operation)); this.store.dispatch(inputActions.clearKeys()); return true; -- cgit v1.2.3 From b9f2668ceab3d786ce3e587803ff30ec5c781ee3 Mon Sep 17 00:00:00 2001 From: Shin'ya Ueoka Date: Sun, 22 Oct 2017 22:17:00 +0900 Subject: add blacklist setting --- src/content/components/top-content/index.js | 26 +++++++++++++++++++++++++- src/shared/utils/re.js | 6 ++++++ src/shared/validators/setting.js | 2 +- test/shared/util/re.test.js | 20 ++++++++++++++++++++ 4 files changed, 52 insertions(+), 2 deletions(-) create mode 100644 src/shared/utils/re.js create mode 100644 test/shared/util/re.test.js (limited to 'src/content/components') diff --git a/src/content/components/top-content/index.js b/src/content/components/top-content/index.js index c318901..c4a8461 100644 --- a/src/content/components/top-content/index.js +++ b/src/content/components/top-content/index.js @@ -1,7 +1,9 @@ import CommonComponent from '../common'; import FollowController from './follow-controller'; import * as consoleFrames from '../../console-frames'; +import * as addonActions from '../../actions/addon'; import messages from 'shared/messages'; +import * as re from 'shared/utils/re'; export default class TopContent { @@ -11,17 +13,39 @@ export default class TopContent { new CommonComponent(win, store), new FollowController(win, store), ]; + this.store = store; + this.prevBlacklist = undefined; // TODO make component - consoleFrames.initialize(window.document); + consoleFrames.initialize(this.win.document); messages.onMessage(this.onMessage.bind(this)); } update() { + let blacklist = this.store.getState().setting.blacklist; + if (JSON.stringify(this.prevBlacklist) !== JSON.stringify(blacklist)) { + this.disableIfBlack(blacklist); + this.prevBlacklist = blacklist; + } + this.children.forEach(c => c.update()); } + disableIfBlack(blacklist) { + let loc = this.win.location; + let partial = loc.host + loc.pathname; + let matched = blacklist + .map((item) => { + let pattern = item.includes('/') ? item : item + '/*'; + return re.fromWildcard(pattern); + }) + .some(regex => regex.test(partial)); + if (matched) { + this.store.dispatch(addonActions.disable()); + } + } + onMessage(message) { switch (message.type) { case messages.CONSOLE_HIDE_COMMAND: diff --git a/src/shared/utils/re.js b/src/shared/utils/re.js new file mode 100644 index 0000000..7db9091 --- /dev/null +++ b/src/shared/utils/re.js @@ -0,0 +1,6 @@ +const fromWildcard = (pattern) => { + let regexStr = '^' + pattern.replace(/\*/g, '.*') + '$'; + return new RegExp(regexStr); +}; + +export { fromWildcard }; diff --git a/src/shared/validators/setting.js b/src/shared/validators/setting.js index 5039ec2..949ab29 100644 --- a/src/shared/validators/setting.js +++ b/src/shared/validators/setting.js @@ -1,6 +1,6 @@ import operations from 'shared/operations'; -const VALID_TOP_KEYS = ['keymaps', 'search']; +const VALID_TOP_KEYS = ['keymaps', 'search', 'blacklist']; const VALID_OPERATION_VALUES = Object.keys(operations).map((key) => { return operations[key]; }); diff --git a/test/shared/util/re.test.js b/test/shared/util/re.test.js new file mode 100644 index 0000000..9ed6521 --- /dev/null +++ b/test/shared/util/re.test.js @@ -0,0 +1,20 @@ +import { expect } from 'chai'; +import * as re from 'shared/utils/re'; + +describe("re util", () => { + it('matches by pattern', () => { + let regex = re.fromWildcard('*.example.com/*'); + expect('foo.example.com/bar').to.match(regex); + expect('foo.example.com').not.to.match(regex); + expect('example.com/bar').not.to.match(regex); + + regex = re.fromWildcard('example.com/*') + expect('example.com/foo').to.match(regex); + expect('example.com/').to.match(regex); + + regex = re.fromWildcard('example.com/*bar') + expect('example.com/foobar').to.match(regex); + expect('example.com/bar').to.match(regex); + expect('example.com/foobarfoo').not.to.match(regex); + }) +}); -- cgit v1.2.3