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: 新增 Popup 和 LayerPopup 组件 #88

Merged
merged 62 commits into from
Nov 8, 2022
Merged

feat: 新增 Popup 和 LayerPopup 组件 #88

merged 62 commits into from
Nov 8, 2022

Conversation

heiyexing
Copy link
Contributor

PR includes

  • 新增 Popup 和 LayerPopup 组件
  • 将原有的 Control Hook 升级为 L7 Component 通用 Hook

heiyexing and others added 30 commits October 12, 2022 10:47
* fix: 修复 Popup 更新 DOM 问题

* fix: 修复 CustomControl style 部分属性丢失问题

* feat: 初步完成 ZoomControl 改造

Co-authored-by: yanxiong <oujinhui.ojh@antgroup.com>
Co-authored-by: yanxiong <oujinhui.ojh@antgroup.com>
* feat: 控件 ZoomControl 新增对 ReactNode 格式的支持

* fix: 还原 package.json

* style: 1. 去除 ZoomControl 已完成的 TODO

Co-authored-by: yanxiong <oujinhui.ojh@antgroup.com>
* fix: 修复 Popup 更新 DOM 问题

* fix: 修复 CustomControl style 部分属性丢失问题

* feat: 初步完成 ZoomControl 改造

* feat: 初步完成scale改造

* feat: 初步完成Fullscreen控件

* fix: 添加eslint

* fix: 修改controlOptions依赖style项

* feat: 初步完成定位组件

* fix: 删除无用代码

* fix: 修改评论问题

* fix: 撤回代码

* feat: 初步完成导出图片和光标经纬度组件

* feat: 初步完成MapTheme和LayerControl两部分组件

* fix: 修改问题

* fix: 修改type类型

* feat: 添加控件文档

* fix: 文档事件名统一

Co-authored-by: yanxiong <oujinhui.ojh@antgroup.com>
Co-authored-by: syb01094648 <syb01094648@antgroup.com>
* feat: 控件 ZoomControl 新增对 ReactNode 格式的支持

* fix: 还原 package.json

* style: 1. 去除 ZoomControl 已完成的 TODO

* refactor: 1.迁移 CustomControl 目录结构  2.补充 Control 文档

Co-authored-by: yanxiong <oujinhui.ojh@antgroup.com>
@github-actions
Copy link

github-actions bot commented Nov 7, 2022

🎊 PR Preview 3819091 has been successfully built and deployed to https://antvis-LarkMap-preview-pr-88.surge.sh

🕐 Build time: 201.498s

🤖 By surge-preview

@lgtm-com
Copy link

lgtm-com bot commented Nov 7, 2022

This pull request introduces 1 alert when merging c682d0e into 4a94338 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 7, 2022

This pull request introduces 1 alert when merging e976388 into 4a94338 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

src/index.ts Outdated Show resolved Hide resolved
src/index.ts Outdated Show resolved Hide resolved
src/types/popup.ts Outdated Show resolved Hide resolved
src/components/LayerPopup/demos/default.tsx Outdated Show resolved Hide resolved
src/components/LayerPopup/index.tsx Outdated Show resolved Hide resolved
@lgtm-com
Copy link

lgtm-com bot commented Nov 8, 2022

This pull request introduces 2 alerts when merging 0e46bfb into 4a94338 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 8, 2022

This pull request introduces 2 alerts when merging c9f7f76 into 4a94338 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 8, 2022

This pull request introduces 2 alerts when merging 295e77f into 4a94338 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

src/types/index.ts Outdated Show resolved Hide resolved
@lgtm-com
Copy link

lgtm-com bot commented Nov 8, 2022

This pull request introduces 2 alerts when merging de58b9d into 4a94338 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 8, 2022

This pull request introduces 1 alert when merging 3819091 into 4a94338 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lvisei lvisei merged commit 42382aa into main Nov 8, 2022
@lvisei lvisei deleted the feat/syb-popup branch November 8, 2022 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants