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

Jason/enable interfaces #140

Merged
merged 6 commits into from
Jun 2, 2023
Merged

Jason/enable interfaces #140

merged 6 commits into from
Jun 2, 2023

Conversation

duidae
Copy link
Contributor

@duidae duidae commented Nov 1, 2022

fix #138 #139, 新增scrollableAncestor/offsetTop options,讓使用者可以調整embed code

option設定方式:在最後一個script tag加入data-scrollable-ancestor data-offset-top,

ex:

<script type="text/javascript" crossorigin="" src="/dist/main.a93c58f44676c91d15a5.bundle.js" data-scrollable-ancestor="window" data-offset-top="70"></script>

@nickhsine 我這樣改您覺得ok嗎? 還請幫忙建議, 謝謝!

cc @fzhantw

@duidae duidae requested a review from nickhsine November 1, 2022 01:57
@duidae duidae self-assigned this Nov 1, 2022
@fzhantw
Copy link

fzhantw commented Nov 1, 2022

這個做法很讚耶
有新的 CDN 位置再請提供,我需要拿去測試

Copy link
Collaborator

@nickhsine nickhsine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT 👍 💯 🥇

@duidae
Copy link
Contributor Author

duidae commented Nov 2, 2022

@duidae duidae merged commit 7422c87 into twreporter:master Jun 2, 2023
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.

Feature Request(dual-channel) 希望增加 scrollableAncestor option
3 participants