Removal of IE6/7 specific code from the code base

291 views
Skip to first unread message

Goktug Gokdogan

unread,
Jul 8, 2014, 4:22:49 PM7/8/14
to google-web-toolkit-contributors
At the end of last year, Daniel started to remove IE6/7 specific code from the SDK and make simplifications based on that; especially for the deferred bindings. We had some progress on it (see some examples [1]) though there are still work to do and looking at our plans it doesn't look like we will able to invest much time into it any time soon.

So if you love cleaning up code and making things simpler and also want to be a good GWT citizen, you are highly encouraged to help out :) Please don't forget to attach IE6 topic to your patches as it helps to track them.

To find such code, searching the code base with keywords (e.g. IE6. IE7, IE, ActiveX etc) is your best bet. Please always double check that the code or the workaround is only needed for IE6/7. If you encounter cases where the workaround is only required by IE8, please also add a comment for that so we can easily find it when the time comes.

I don't know how many people would be interested in this work but just in case to avoid duplicate work, please update this thread after you invest some time into the patch.

Cheers,

 - Goktug

Michael Vogt

unread,
Jul 11, 2014, 10:34:41 PM7/11/14
to google-web-tool...@googlegroups.com
I will be offline [1] from tomorrow on, probably until 7/22, but I
will have a look at what I can do to remove IE6/7 specific code when
this is helpful. Can push to Gerrit on 22nd.


Greetings,
Michael



[1] except a really slow tourist sim internet connection

Jens

unread,
Jul 12, 2014, 9:15:30 AM7/12/14
to google-web-tool...@googlegroups.com
Michael, you should track https://gwt-review.googlesource.com/#/c/5304/ so you don't do the same work again.

-- J.

Goktug Gokdogan

unread,
Oct 2, 2014, 12:56:50 PM10/2/14
to google-web-toolkit-contributors
There are still 80+ references to IE6 in the code base. Still looking for more patches...

On Sat, Jul 12, 2014 at 6:15 AM, Jens <jens.ne...@gmail.com> wrote:
Michael, you should track https://gwt-review.googlesource.com/#/c/5304/ so you don't do the same work again.

-- J.

--
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-co...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/6eb89ab9-5109-4658-9dae-95e14072dd3c%40googlegroups.com.

For more options, visit https://groups.google.com/d/optout.

Michael Vogt

unread,
Oct 3, 2014, 2:32:35 PM10/3/14
to google-web-tool...@googlegroups.com
Sorry for being away so long. I will have some time next week to work on this.

Colin Alworth

unread,
Oct 7, 2014, 6:31:23 PM10/7/14
to google-web-tool...@googlegroups.com
I've just submitted a shallow removal of IE6 specific code in https://gwt-review.googlesource.com/#/c/9513/ - this patch starts with all .gwt.xml files that mention the user.agent value ie6 and removes those property checks, along with any classes *only* referenced from those rebind rules. Far from a complete job of what remains, but takes another swipe at removing old dead junk.
Reply all
Reply to author
Forward
0 new messages