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

instance fields on CanvasRenderingContext2D [fillStyle, globalAlpha, strokeStyle] warn as 'final' in Dart Editor #3248

Closed
kevmoo opened this issue May 25, 2012 · 7 comments
Assignees

Comments

@kevmoo
Copy link
Member

kevmoo commented May 25, 2012

What steps will reproduce the problem?

  1. Create a reference to CanvasRenderingContext2D in variable ctx
  2. Assign a value to fillStyle, globalAlpha, strokeStyle
  3. Notice warnings that all of these fields are 'final'

What is the expected output? What do you see instead?

 - I expect Dart Editor to be happy I'm playing with canvas.

What version of the product are you using? On what operating system?

 - Dart Continuous 8008


Attachment:
[Screen Shot 2012-05-25 at 11.50.37 AM.png](https://storage.googleapis.com/google-code-attachments/dart/issue-3248/comment-0/Screen Shot 2012-05-25 at 11.50.37 AM.png) (34.15 KB)

@stevemessick
Copy link
Contributor

Assignments to Element.src are also getting warnings.


Set owner to @scheglov.
Added Area-Editor, Triaged labels.

@stevemessick
Copy link
Contributor

Issue #3249 has been merged into this issue.

@DartBot
Copy link

DartBot commented May 25, 2012

This comment was originally written by @bp74


and many others too:

AudioElement: currentTime/loop/volume
CanvasElement: width/height
AudioBufferSourceNode: buffer/loop
AudioGainNode: gain.value
...

I think it is a more general problem with the html library.

@scheglov
Copy link
Contributor

Issue #3252 has been merged into this issue.

@scheglov
Copy link
Contributor

@scheglov
Copy link
Contributor

@kevmoo
Copy link
Member Author

kevmoo commented May 29, 2012

Confirmed fixed in 8075. Cheers!

This issue was closed.
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

No branches or pull requests

4 participants