Skip to content

Comments

Update data_texture.dart#180

Open
ivosek wants to merge 1 commit intothreeDart:masterfrom
ivosek:patch-1
Open

Update data_texture.dart#180
ivosek wants to merge 1 commit intothreeDart:masterfrom
ivosek:patch-1

Conversation

@ivosek
Copy link

@ivosek ivosek commented Feb 4, 2015

Using DataTexture fails in web_gl_renderer.dart:6837, because image parameter is instance of LinkedHashMap and there is no "width" nor "height" getter. Creating and passing instance of DataImage class instead of {"data":data,"width":width,"height":height} solves the problem.
Other problem is that all optional parameters are automatically passed as null and WebGL prints out warning about illegal texture parameters value.

Using DataTexture fails in web_gl_renderer.dart:6837, because image parameter is instance of LinkedHashMap and there is no "width" nor "height" getter. Creating and passing  instance of DataImage class instead of {"data":data,"width":width,"height":height} solves the problem.
Other problem is that all optional parameters are automatically passed as null and WebGL prints out warning about illegal texture parameters value.
@johsin18
Copy link
Contributor

Can you reformat this please according to tool/dartfmt.sh, and rebase on the latest master? Looks good to me otherwise.

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.

2 participants