Goktug Gokdogan has posted comments on this change.
Change subject: Add hasClassName method in com.google.gwt.dom.client.Element
......................................................................
Patch Set 3:
Hi Colin. I understand your concerns.
However, I like to remind that we are only promising API compatibility in
maintenance releases like (2.5.1 to 2.5.2). Waiting major releases for
these kind of patches would mean that delays up to 6 months which is not
very practical and acceptable pace - especially for Google's use. Being
said that, we are also trying to avoid changes that would cause too much
refactoring to do an upgrade. In this case, the fix is pretty
straightforward for most end user usages. If this one is a real big trouble
for you, then we can wait for another 4-5 months but I would not prefer
that =)
For reducing the pain in the future; I don't expect this class to grow much
but I wonder if it is possible for you to compose instead of extend this
kind of classes? If you can't, feel free to propose other changes that you
expect to see in this base class instead of adding it your own. For
example, I'm also planning to add toggleClassName.
--
To view, visit https://gwt-review.googlesource.com/3070
To unsubscribe, visit https://gwt-review.googlesource.com/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia09567b8c58cac02f8126c33ef169b26def3d19c
Gerrit-PatchSet: 3
Gerrit-Project: gwt
Gerrit-Branch: master
Gerrit-Owner: Andrey Korzhevskiy <a.korzhevs...@gmail.com>
Gerrit-Reviewer: Andrey Korzhevskiy <a.korzhevs...@gmail.com>
Gerrit-Reviewer: Colin Alworth <niloc...@gmail.com>
Gerrit-Reviewer: Daniel Kurka <danku...@google.com>
Gerrit-Reviewer: Goktug Gokdogan <gok...@google.com>
Gerrit-HasComments: No
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to the Google Groups "GWT Contributors" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to google-web-toolkit-contributors+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.