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

Fixed widget iterator going out of range with ui.Tree #87

Merged
merged 1 commit into from May 6, 2013

Conversation

Nican
Copy link
Contributor

@Nican Nican commented May 6, 2013

Hey!

I would like to thank you for porting over the awesomeness of GWT to Dart, I was really can not get used to how limited WebUI is.

One question that I have is, did you really have to port the exact event system from GWT? One thing that I hated about GWT is the fact you had 3 different classes for every event; Would it not just be easier to use Dart's Stream feature?

akserg added a commit that referenced this pull request May 6, 2013
Fixed widget iterator going out of range with ui.Tree
@akserg akserg merged commit a387e0f into akserg:master May 6, 2013
@akserg
Copy link
Owner

akserg commented May 6, 2013

Hi Nican.
Thank you very much.
About you question regarding event mechanism - we planning make changes in
DWT event system. Please, have a look our new community Dart Web
Toolkithttps://plus.google.com/u/0/communities/100328499012805124876on
G+.

On 6 May 2013 03:36, Nican notifications@github.com wrote:

Hey!

I would like to thank you for porting over the awesomeness of GWT to Dart,
I was really can not get used to how limited WebUI is.

One question that I have is, did you really have to port the exact event
system from GWT? One thing that I hated about GWT is the fact you had 3
different classes for every event; Would it not just be easier to use

Dart's Stream feature?

You can merge this Pull Request by running

git pull https://github.com/Nican/dart_web_toolkit master

Or view, comment on, or merge it at:

#87
Commit Summary

  • Fixed widget iterator going out of range with ui.Tree

File Changes

Patch Links:

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.

None yet

2 participants