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

feat(taro-hooks): add Keyboard hook #54

Merged
merged 10 commits into from
Sep 3, 2023
Merged

Conversation

StringKe
Copy link
Contributor

@StringKe StringKe commented Aug 15, 2023

Completed 1709 next StringKe 13 Powered by Pull Request Badge

🤔 这个变动的性质是?/ What is the nature of this change?

  • 新特性提交 / New feature
  • bug 修复 / Fix bug
  • 样式优化 / Style optimization
  • 代码风格优化 / Code style optimization
  • 性能优化 / Performance optimization
  • 构建优化 / Build optimization
  • 网站、文档、Demo 改进 / Website, documentation, demo improvements
  • 重构代码或样式 / Refactor code or style
  • 测试相关 / Test related
  • 其他 / Other

🔗 相关 Issue / Related Issue

https://docs.taro.zone/docs/apis/device/keyboard/onKeyboardHeightChange

💡 需求背景和解决方案 / Background or solution

📝 更新日志 / Changelog

Language Changelog
🇺🇸 English
🇨🇳 Chinese

@vercel
Copy link

vercel bot commented Aug 15, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @innocces on Vercel.

@innocces first needs to authorize it.

@innocces
Copy link
Owner

innocces commented Aug 15, 2023

您好,需要补充对应的示例.

示例文件夹为: examples 下的两个项目. 分别为react和vue的

创建好示例之后再文档内添加类似以下的内容即可自动展示:

## 代码演示
<code src="useAPICheck/index" group="wechat" />

其中 src 是你在具体分组下的文件地址。 group 为分组文件夹名称

@innocces
Copy link
Owner

vue 的示例也补一下

@innocces
Copy link
Owner

如果看到的话记得合一下我提到你的pr的一个pr

[feature] - useKeyBoard
@StringKe
Copy link
Contributor Author

如果看到的话记得合一下我提到你的pr的一个pr

原来是提到我那里去了,我以为是你直接在当前这个 pr 上修改。。

@innocces innocces marked this pull request as ready for review August 29, 2023 03:09
@innocces
Copy link
Owner

innocces commented Sep 1, 2023

我周末合一下吧。周内没什么时间

@innocces innocces self-requested a review September 1, 2023 01:52
@innocces innocces merged commit 94ecfde into innocces:next Sep 3, 2023
2 of 7 checks passed
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