Skip to content

Commit

Permalink
Fix hb import for media provider
Browse files Browse the repository at this point in the history
  • Loading branch information
web-flow committed Nov 9, 2023
1 parent c25e4c9 commit c3e55d4
Show file tree
Hide file tree
Showing 11 changed files with 25 additions and 35 deletions.
1 change: 1 addition & 0 deletions .env
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
PW_EXPERIMENTAL_SERVICE_WORKER_NETWORK_EVENTS=1
2 changes: 0 additions & 2 deletions .github/workflows/playwright.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ jobs:
playwright:
timeout-minutes: 60
runs-on: ubuntu-latest
env:
PW_EXPERIMENTAL_SERVICE_WORKER_NETWORK_EVENTS: 1
steps:
- name: Remove all fonts
run: rm -rf /usr/share/fonts
Expand Down
3 changes: 0 additions & 3 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,5 @@
"source.fixAll.eslint": true,
},
"eslint.format.enable": true,
"playwright.env": {
"PW_EXPERIMENTAL_SERVICE_WORKER_NETWORK_EVENTS": 1
},
"javascript.preferences.importModuleSpecifierEnding": "js",
}
5 changes: 0 additions & 5 deletions ext/js/display/display-audio.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,6 @@ import {PopupMenu} from '../dom/popup-menu.js';
import {AudioSystem} from '../media/audio-system.js';
import {yomichan} from '../yomichan.js';

/* global
* AudioSystem
* PopupMenu
*/

export class DisplayAudio {
constructor(display) {
this._display = display;
Expand Down
11 changes: 0 additions & 11 deletions ext/js/display/popup-main.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,17 +27,6 @@ import {DisplayProfileSelection} from './display-profile-selection.js';
import {DisplayResizer} from './display-resizer.js';
import {Display} from './display.js';

/* global
* Display
* DisplayAnki
* DisplayAudio
* DisplayProfileSelection
* DisplayResizer
* DocumentFocusController
* HotkeyHandler
* JapaneseUtil
*/

(async () => {
try {
const documentFocusController = new DocumentFocusController();
Expand Down
4 changes: 0 additions & 4 deletions ext/js/pages/settings/dictionary-controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,6 @@ import {EventListenerCollection, log} from '../../core.js';
import {DictionaryWorker} from '../../language/dictionary-worker.js';
import {yomichan} from '../../yomichan.js';

/* global
* DictionaryWorker
*/

class DictionaryEntry {
constructor(dictionaryController, fragment, index, dictionaryInfo) {
this._dictionaryController = dictionaryController;
Expand Down
6 changes: 0 additions & 6 deletions ext/js/pages/settings/popup-preview-frame-main.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,6 @@ import {HotkeyHandler} from '../../input/hotkey-handler.js';
import {yomichan} from '../../yomichan.js';
import {PopupPreviewFrame} from './popup-preview-frame.js';

/* global
* HotkeyHandler
* PopupFactory
* PopupPreviewFrame
*/

(async () => {
try {
await yomichan.prepare();
Expand Down
4 changes: 1 addition & 3 deletions ext/js/templates/sandbox/template-renderer-media-provider.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,7 @@
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*/

/* global
* Handlebars
*/
import {Handlebars} from '../../../lib/handlebars.js';

export class TemplateRendererMediaProvider {
constructor() {
Expand Down
19 changes: 19 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -60,5 +60,8 @@
"stylelint-config-recommended": "^13.0.0",
"vitest": "^0.34.6",
"wanakana": "^5.2.0"
},
"dependencies": {
"dotenv": "^16.3.1"
}
}
2 changes: 1 addition & 1 deletion playwright.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ import {defineConfig, devices} from '@playwright/test';
* Read environment variables from file.
* https://github.com/motdotla/dotenv
*/
// require('dotenv').config();
import 'dotenv/config';

/**
* @see https://playwright.dev/docs/test-configuration
Expand Down

0 comments on commit c3e55d4

Please sign in to comment.