From e29d431dfdc97988102a88aac2c5029da82ce730 Mon Sep 17 00:00:00 2001 From: pengzhanbo Date: Wed, 11 Oct 2023 04:32:15 +0800 Subject: [PATCH] style: update eslint config and fix code lint --- packages/plugin-auto-frontmatter/src/shared/index.ts | 4 +--- packages/plugin-caniuse/src/client/resolveCanIUse.ts | 8 +++----- packages/plugin-notes-data/src/shared/index.ts | 6 +++--- packages/plugin-page-collection/src/shared/index.ts | 4 +--- packages/theme/src/client/components/LocalNav.vue | 4 +--- packages/theme/src/client/components/Nav/NavBar.vue | 4 +--- .../theme/src/client/components/Nav/NavBarHamburger.vue | 4 +--- packages/theme/src/client/composables/flyout.ts | 3 ++- packages/theme/src/shared/options/navbar.ts | 4 ++-- packages/theme/src/shared/page.ts | 4 ++-- 10 files changed, 17 insertions(+), 28 deletions(-) diff --git a/packages/plugin-auto-frontmatter/src/shared/index.ts b/packages/plugin-auto-frontmatter/src/shared/index.ts index 7c4597c1..cc58c19f 100644 --- a/packages/plugin-auto-frontmatter/src/shared/index.ts +++ b/packages/plugin-auto-frontmatter/src/shared/index.ts @@ -13,9 +13,7 @@ export type FrontmatterFn = ( data: K ) => T | PromiseLike -export type FrontmatterObject = { - [P: string]: FrontmatterFn -} +export type FrontmatterObject = Record>; export type FrontmatterArray = { include: string | string[] diff --git a/packages/plugin-caniuse/src/client/resolveCanIUse.ts b/packages/plugin-caniuse/src/client/resolveCanIUse.ts index e9805fe6..2f738c9c 100644 --- a/packages/plugin-caniuse/src/client/resolveCanIUse.ts +++ b/packages/plugin-caniuse/src/client/resolveCanIUse.ts @@ -1,7 +1,6 @@ export const resolveCanIUse = (): void => { - const canIUseEls = document.getElementsByClassName('ciu_embed') - for (let t = 0; t < canIUseEls.length; t++) { - const el = canIUseEls[t] + const canIUseEls = Array.from(document.getElementsByClassName('ciu_embed')) + for (const el of canIUseEls) { const feature = el.getAttribute('data-feature') const periods = el.getAttribute('data-periods') const accessible = el.getAttribute('data-accessible-colours') || 'false' @@ -18,8 +17,7 @@ export const resolveCanIUse = (): void => { const data = message.data if (typeof data === 'string' && data.indexOf('ciu_embed') > -1) { const [, feature, height] = data.split(':') - for (let i = 0; i < canIUseEls.length; i++) { - const el = canIUseEls[i] + for (const el of canIUseEls) { if (el.getAttribute('data-feature') === feature) { const h = parseInt(height) + 30 ;(el.childNodes[0] as any).height = h + 'px' diff --git a/packages/plugin-notes-data/src/shared/index.ts b/packages/plugin-notes-data/src/shared/index.ts index 1a52db4d..8596556a 100644 --- a/packages/plugin-notes-data/src/shared/index.ts +++ b/packages/plugin-notes-data/src/shared/index.ts @@ -1,4 +1,4 @@ -export type NotesDataOptions = { +export interface NotesDataOptions { dir: string link: string include?: string | string[] @@ -6,7 +6,7 @@ export type NotesDataOptions = { notes: NotesItem[] } -export type NotesItem = { +export interface NotesItem { dir: string link: string text: string @@ -15,7 +15,7 @@ export type NotesItem = { export type NotesSidebar = (NotesSidebarItem | string)[] -export type NotesSidebarItem = { +export interface NotesSidebarItem { text?: string link?: string dir?: string diff --git a/packages/plugin-page-collection/src/shared/index.ts b/packages/plugin-page-collection/src/shared/index.ts index 61e3b01d..b1aa5940 100644 --- a/packages/plugin-page-collection/src/shared/index.ts +++ b/packages/plugin-page-collection/src/shared/index.ts @@ -1,3 +1 @@ -export interface PageCollectionOptions { - [x: string]: unknown -} +export type PageCollectionOptions = Record; diff --git a/packages/theme/src/client/components/LocalNav.vue b/packages/theme/src/client/components/LocalNav.vue index b3fff7cc..6fe607d9 100644 --- a/packages/theme/src/client/components/LocalNav.vue +++ b/packages/theme/src/client/components/LocalNav.vue @@ -6,9 +6,7 @@ defineProps<{ open: boolean }>() -defineEmits<{ - (e: 'open-menu'): void -}>() +defineEmits<(e: 'open-menu') => void>() const { hasSidebar } = useSidebar() diff --git a/packages/theme/src/client/components/Nav/NavBar.vue b/packages/theme/src/client/components/Nav/NavBar.vue index 268d4e60..a4c3492c 100644 --- a/packages/theme/src/client/components/Nav/NavBar.vue +++ b/packages/theme/src/client/components/Nav/NavBar.vue @@ -14,9 +14,7 @@ import NavBarTranslations from './NavBarTranslations.vue' defineProps<{ isScreenOpen: boolean }>() -defineEmits<{ - (e: 'toggle-screen'): void -}>() +defineEmits<(e: 'toggle-screen') => void>() const { y } = useWindowScroll() const { hasSidebar } = useSidebar() diff --git a/packages/theme/src/client/components/Nav/NavBarHamburger.vue b/packages/theme/src/client/components/Nav/NavBarHamburger.vue index f4129e2a..1073b423 100644 --- a/packages/theme/src/client/components/Nav/NavBarHamburger.vue +++ b/packages/theme/src/client/components/Nav/NavBarHamburger.vue @@ -3,9 +3,7 @@ defineProps<{ active: boolean }>() -defineEmits<{ - (e: 'click'): void -}>() +defineEmits<(e: 'click') => void>()