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.

Sign up for GitHub

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

Jump to bottom

feat(g-mobile): [BREAKING_CHANGE] add g-mobile package to support mobile devices && mini-programs #766

Merged
merged 617 commits into from
May 17, 2021
Merged

feat(g-mobile): [BREAKING_CHANGE] add g-mobile package to support mobile devices && mini-programs #766

merged 617 commits into from
May 17, 2021

Conversation

AlbertAZ1992
Copy link
Member

🤔 This is a ...

  • New feature
  • Bug fix
  • Site / Document optimization
  • TypeScript definition update
  • Refactoring
  • Performance improvement
  • Code style optimization
  • Test Case
  • Branch merge
  • Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English add g-mobile package to implement events simulation for mobile devices、canvas proxy for mini-program
🇨🇳 Chinese 增加 g-mobile 包，适配移动端事件，兼容小程序 canvas

☑️ Self Check before Merge

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

dengfuping and others added 30 commits February 24, 2020 22:44
@dengfuping
Merge pull request #420 from antvis/fix-issue-417
42b6233 
fix(g-base): bbox calculation should be correct for container,  close #417 and #407
@dengfuping
Publish
f546713 
 - @antv/g-base@0.3.22
 - @antv/g-canvas@0.3.23
 - @antv/g-svg@0.3.24
@dengfuping
fix(g-canvas): matrix of element should affect its clip, close #382
6893fec
@dengfuping
Merge pull request #421 from antvis/fix-issue-382
0905250 
fix(g-canvas): matrix of element should affect its clip
@dengfuping
Publish
f8e51fb 
 - @antv/g-canvas@0.3.24
@dengfuping
fix(g-canvas): should render when localRefresh change from false to t…
6193a76 
…rue, close #422
@dengfuping
Merge pull request #423 from antvis/fix-issue-422
581a001 
fix(g-canvas): should render when localRefresh change from false to true
@dengfuping
Publish
14fa0c8 
 - @antv/g-canvas@0.3.25
@dengfuping
fix(g-base): delay and repeat should work in toAttrs for animate method
1de1afa
@dengfuping
Merge pull request #424 from antvis/zhuyue-dev
156eef7 
fix(g-base): delay and repeat should work in toAttrs for animate method
@dengfuping
Publish
d0f7a58 
 - @antv/g-base@0.3.23
 - @antv/g-canvas@0.3.26
 - @antv/g-svg@0.3.25
@dxq613
fix(bbox): path bbox lineWidth
a07dd67
@dxq613
feat(version): update version
4ae1bfb
@dengfuping
Merge pull request #425 from antvis/path_bbox
f5ec42b 
Path bbox
@dengfuping
fix(g-base): delay should not work in toAttrs, ref PR: #424
9914df4
@dengfuping
Merge pull request #427 from antvis/fix-pr-424
a47eff5 
fix(g-base): delay should not work in toAttrs
@dengfuping
fix(g-svg): event should work when matrix is applied, close #428
475afbb
@dengfuping
Merge pull request #430 from antvis/fix-issue-428
ad642e4 
fix(g-svg): event should work when matrix is applied
@dengfuping
Publish
dfdbb32 
 - @antv/g-base@0.3.26
 - @antv/g-canvas@0.3.27
 - @antv/g-svg@0.3.26
@dxq613
feat(bbox): shadow refresh
4e20c4a
@dxq613
feat(shadow): shadowColor decide shadow bbox
747df0b
@dxq613
chore(vesion): update version
dd05ddf
@dengfuping
Merge pull request #434 from antvis/shadow_bbox
ae7b64f 
feat(bbox): shadow refresh
@dengfuping
Publish
076bc14 
 - @antv/g-base@0.3.28
 - @antv/g-canvas@0.3.28
 - @antv/g-svg@0.3.27
@dengfuping
fix(g-base): should consider precision of angle calculation for Path, c…
6fda5bf 
…lose #436
@dengfuping
Merge pull request #437 from antvis/fix-issue-436
ea6cff1 
fix(g-base): should consider precision of angle calculation for Path
@dengfuping
Publish
3ba3be0 
 - @antv/g-base@0.3.29
 - @antv/g-canvas@0.3.29
 - @antv/g-svg@0.3.28
@budlion
fix(g-canvas): g-canvas@0.3.29 is not compatible with @antv/path-util…
0e2a199 
…@2.0.4

no path-2-segments.js in @antv/path-util@2.0.4
@simaQ
feat(g-base): 补充 ShapeAttrs 类型定义
9958372
@dengfuping
feat(g-canvas): arrow support bbox calculation and hit, ref: #432
51fa512
@AlbertAZ1992 AlbertAZ1992 requested a review from baizn April 26, 2021 12:47
accesslint[bot]
accesslint bot suggested changes Apr 26, 2021
View reviewed changes
Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

packages/g-mobile/README-zh_CN.md

# g-mobile

[![](https://img.shields.io/travis/antvis/g.svg)](https://travis-ci.org/antvis/g)
Copy link

Choose a reason for hiding this comment

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

This image is missing a text alternative (alt attribute). This is a problem for people using screen readers.

packages/g-mobile/README-zh_CN.md
# g-mobile

[![](https://img.shields.io/travis/antvis/g.svg)](https://travis-ci.org/antvis/g)
![](https://img.shields.io/badge/language-javascript-red.svg)
Copy link

Choose a reason for hiding this comment

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

This image is missing a text alternative (alt attribute). This is a problem for people using screen readers.

packages/g-mobile/README-zh_CN.md

[![](https://img.shields.io/travis/antvis/g.svg)](https://travis-ci.org/antvis/g)
![](https://img.shields.io/badge/language-javascript-red.svg)
![](https://img.shields.io/badge/license-MIT-000000.svg)
Copy link

Choose a reason for hiding this comment

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

This image is missing a text alternative (alt attribute). This is a problem for people using screen readers.

packages/g-mobile/README.md

[![](https://img.shields.io/travis/antvis/g.svg)](https://travis-ci.org/antvis/g)
Copy link

Choose a reason for hiding this comment

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

This image is missing a text alternative (alt attribute). This is a problem for people using screen readers.

packages/g-mobile/README.md

[![](https://img.shields.io/travis/antvis/g.svg)](https://travis-ci.org/antvis/g)
![](https://img.shields.io/badge/language-javascript-red.svg)
Copy link

Choose a reason for hiding this comment

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

This image is missing a text alternative (alt attribute). This is a problem for people using screen readers.

packages/g-mobile/README.md

[![](https://img.shields.io/travis/antvis/g.svg)](https://travis-ci.org/antvis/g)
![](https://img.shields.io/badge/language-javascript-red.svg)
![](https://img.shields.io/badge/license-MIT-000000.svg)
Copy link

Choose a reason for hiding this comment

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

This image is missing a text alternative (alt attribute). This is a problem for people using screen readers.

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Apr 26, 2021

This pull request introduces 10 alerts when merging b54bd7b into 5330549 - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class
  • 2 for Polynomial regular expression used on uncontrolled data
  • 1 for Unreachable statement
  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Apr 27, 2021

This pull request introduces 9 alerts when merging b264721 into 5330549 - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class
  • 2 for Polynomial regular expression used on uncontrolled data
  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented May 10, 2021

This pull request introduces 9 alerts when merging 6c51b96 into e7fc299 - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class
  • 2 for Polynomial regular expression used on uncontrolled data
  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented May 11, 2021

This pull request introduces 9 alerts when merging d74e460 into e7fc299 - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class
  • 2 for Polynomial regular expression used on uncontrolled data
  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented May 17, 2021

This pull request introduces 9 alerts when merging 6e1b109 into e7fc299 - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class
  • 2 for Polynomial regular expression used on uncontrolled data
  • 1 for Useless conditional

@xiaoiver xiaoiver merged commit 7b05e80 into master May 17, 2021
@xiaoiver xiaoiver deleted the g-mobile branch May 17, 2021 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@accesslint accesslint[bot] accesslint[bot] requested changes

@baizn baizn Awaiting requested review from baizn

Assignees
No one assigned
Labels
PR: merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.