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

G mobile #910

Merged
merged 3 commits into from
Mar 16, 2022
Merged

G mobile #910

merged 3 commits into from
Mar 16, 2022

Conversation

xiaoiver
Copy link
Contributor

🤔 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
🇨🇳 Chinese

☑️ 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

zengyue and others added 3 commits March 11, 2022 09:31
* feat: support  in Image #814 (#897)

* feat: support  in Image #814

* fix: use attributes in Arrow

* docs: add yoga plugin docs

Co-authored-by: yuqi.pyq <yuqi.pyq@antgroup.com>

* feat: add event

* fix: trigger click event instead of tap on mobile #900 (#902)

* fix: trigger click event instead of tap on mobile #900

* feat: support dy attribute in <text>

* fix: use latest WGSL decorator syntax in g-plugin-gpgpu #904

* feat: add mutation-observer to G API

* fix: add reflect-metadata to deps

* chore: remove debugger

Co-authored-by: yuqi.pyq <yuqi.pyq@antgroup.com>

* feat: add PointMaterial in g-plugin-3d (#907)

* feat: add PointMaterial in g-plugin-3d

* docs: add mutation-observer docs

Co-authored-by: yuqi.pyq <yuqi.pyq@antgroup.com>

* fix: add user-defined dpr in g-mobile

* chore: add lint rules

Co-authored-by: xiaoiver <yuqi.pyq@antfin.com>
Co-authored-by: yuqi.pyq <yuqi.pyq@antgroup.com>
Co-authored-by: xuying.xu <xuying.xu@alibaba-inc.com>
@xiaoiver xiaoiver merged commit cbe3c4f into next Mar 16, 2022
@xiaoiver xiaoiver deleted the g-mobile branch March 16, 2022 07:00
@lgtm-com
Copy link
Contributor

lgtm-com bot commented Mar 16, 2022

This pull request fixes 7 alerts when merging 8205c97 into 5e3292d - view on LGTM.com

fixed alerts:

  • 7 for Unused variable, import, function or class

@github-actions
Copy link
Contributor

github-actions bot commented Mar 16, 2022

🎊 PR Preview 8205c97 has been successfully built and deployed to https://antvis-g-preview-pr-910.surge.sh?type=diff&date=2022-03-16

🕐 Build time: 106.093s

🤖 By Surge Ui Insight

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1991211812

  • 140 of 372 (37.63%) changed or added relevant lines in 28 files are covered.
  • 299 unchanged lines in 12 files lost coverage.
  • Overall coverage increased (+0.3%) to 49.462%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/g-mobile/src/dom.ts 5 6 83.33%
packages/g/src/Canvas.ts 0 1 0.0%
packages/g/src/canvas-module.ts 1 2 50.0%
packages/g/src/display-objects/Image.ts 0 1 0.0%
packages/g/src/dom/CSSStyleDeclaration.ts 0 1 0.0%
packages/g/src/services/EventService.ts 0 1 0.0%
packages/g/src/services/text/index.ts 1 2 50.0%
packages/g/src/dom/Document.ts 1 3 33.33%
packages/g/src/property-handlers/textAlign.ts 0 10 0.0%
packages/g/src/dom/MutationRecord.ts 2 19 10.53%
Files with Coverage Reduction New Missed Lines %
packages/g/src/display-objects/Text.ts 1 42.86%
packages/g/src/dom/MutationRecord.ts 1 14.29%
packages/g/src/canvas-module.ts 3 54.55%
packages/g/src/dom/EventTarget.ts 7 16.22%
packages/g/src/display-objects/CustomElement.ts 9 11.11%
packages/g/src/property-handlers/dimension.ts 15 71.19%
packages/g/src/dom/Element.ts 18 80.63%
packages/g/src/dom/Document.ts 23 17.07%
packages/g/src/Canvas.ts 31 11.36%
packages/g/src/services/SceneGraphService.ts 32 63.74%
Totals Coverage Status
Change from base Build 1985677925: 0.3%
Covered Lines: 3567
Relevant Lines: 6377

💛 - Coveralls

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.

4 participants