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

Examples/charts #467

Closed
wants to merge 7 commits into from
Closed

Examples/charts #467

wants to merge 7 commits into from

Conversation

TatriX
Copy link
Member

@TatriX TatriX commented May 25, 2020

Here's a chart example.
It's a bit messy right now, as a lot of things are hardcoded and not properly structured.
But it should be good enough to serve as a starting point.

2020-05-25-145127_3077x1267_scrot
As you can see in the dev tools, there is a xmlns attribute on every svg node. I believe all of those can be removed: see "note" on the mdn

@MartinKavik MartinKavik marked this pull request as draft May 25, 2020 14:51
@TatriX TatriX force-pushed the examples/charts branch 2 times, most recently from 870e1d2 to 213de47 Compare July 7, 2020 11:14
@TatriX TatriX marked this pull request as ready for review August 4, 2020 11:42
Copy link
Member

@MartinKavik MartinKavik left a comment

Choose a reason for hiding this comment

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

I've added some code suggestions. And please update CHANGELOG and examples/README.
Also rebasing on master will be needed.

Nice example, thanks!

examples/charts/src/lib.rs Outdated Show resolved Hide resolved
examples/charts/src/line.rs Outdated Show resolved Hide resolved
examples/charts/src/line.rs Outdated Show resolved Hide resolved
examples/charts/src/line.rs Outdated Show resolved Hide resolved
Comment on lines +116 to +117
let onenter = onenter.clone();
let onout = onout.clone();
Copy link
Member

Choose a reason for hiding this comment

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

on_enter, on_out

Copy link
Member

@MartinKavik MartinKavik left a comment

Choose a reason for hiding this comment

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

Please fix on_out + on_enter issue that breaks example compilation (see related review comment).
And CI passes because the example is missing in Cargo.toml members - please add it. Thanks!

Update - it's already in members, sorry. It's weird.

@flosse flosse added the example label Apr 6, 2022
@flosse flosse mentioned this pull request Apr 6, 2022
@flosse
Copy link
Member

flosse commented Apr 6, 2022

closing in favor of #664

@flosse flosse closed this Apr 6, 2022
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