Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhance: 単一のHTMLElementを受け付けるように #35

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Oct 26, 2024

現状、単一のHTMLElementをそのまま引数に入れてScrollHintを初期化することはできず、一度配列に入れる必要がありましたが、このPRではそれを解消します。
また、TypeScriptの型定義では NodeListOf<HTMLElement> のみを受け付けるようになっていましたが、HTMLElementの配列(HTMLElement[])でも実動上問題なさそうでしたので、型定義も併せて更新しました。

これにより、以下のように記述することができるようになります(Vue):

<template>
    <div>
        <div ref="scroller"><!-- 横長な要素 --></div>
    </div>
</template>

<script setup lang="ts">
import { onMounted, useTemplateRef } from 'vue';
import ScrollHint from 'scroll-hint';

const scroller = useTemplateRef('scroller');

onMounted(() => {
    if (scroller.value == null) return;
    new ScrollHint(scroller.value, { // ←単に要素を指定するだけでも動くように
        i18n: {
            scrollable: 'スクロールできます',
        },
    });
})
</script>

別途ビルドしたもの(/js配下)もこのPRに含めるべきかどうかがわかりませんでしたので一旦含めておりません。

@uidev1116
Copy link
Contributor

@kakkokari-gtyih
ありがとうございます。問題なさそうですのでマージさせていただきます!

@uidev1116 uidev1116 merged commit 03f2b06 into appleple:master Nov 21, 2024
@kakkokari-gtyih kakkokari-gtyih deleted the fix-type branch November 21, 2024 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants