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
upgrade of view engine #90
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add tests to make sure this change works as expected?
/rebase |
There was a problem hiding this 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.
value={this.state.color} | ||
style={{ width: '100%' }} | ||
onChange={this.onColorChanged} | ||
/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there's a label missing for this input. That makes it hard for people using screen readers or voice control to use the input.
style={{ width: '100%' }} | ||
value={this.state.repeat} | ||
onChange={this.onRepeatChanged} | ||
> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there's a label missing for this input. That makes it hard for people using screen readers or voice control to use the input.
style={{ width: '100%' }} | ||
value={this.state.position} | ||
onChange={this.onPositionChanged} | ||
> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there's a label missing for this input. That makes it hard for people using screen readers or voice control to use the input.
style={{ width: '100%' }} | ||
value={this.state.size} | ||
onChange={this.onSizeChanged} | ||
> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there's a label missing for this input. That makes it hard for people using screen readers or voice control to use the input.
This pull request fixes 3 alerts when merging 943415a into 7a5a56b - view on LGTM.com fixed alerts:
|
This pull request fixes 3 alerts when merging 58a1a5c into 7a5a56b - view on LGTM.com fixed alerts:
|
This pull request fixes 3 alerts when merging 485f52e into 7a5a56b - view on LGTM.com fixed alerts:
|
This pull request fixes 3 alerts when merging 770f694 into 7a5a56b - view on LGTM.com fixed alerts:
|
🎉 This PR is included in the following release 🎉
Thanks for being a part of the AntV community! 💪💯 |
Description
Motivation and Context
Types of changes
Self Check before Merge