Should have specified which issue tracker! As usually we put issues against
incubator bugs in the incubator issue tracker. I've ported the one you
entered and left breadcrumbs.
Cheers,
Emily
On Wed, Oct 1, 2008 at 9:25 AM, Paul Robinson <[EMAIL PROTECTED]> wrote:
>
> Done.
>
> http://code.google.com/p/google-web-toolkit/issues/detail?id=2932
>
> Paul
>
> Emily Crutcher wrote:
> > Can you file a bug report? That way I can assign it appropriately.
> >
> > On Wed, Oct 1, 2008 at 6:39 AM, Paul Robinson <[EMAIL PROTECTED]
> > <mailto:[EMAIL PROTECTED]>> wrote:
> >
> >
> > I don't know whether TimePicker is going to be reworked into gen2 as
> > well as DatePicker, but I've found a bug in the current incubator
> > version of TimePicker. It's in the ValueSpinner constructor. It adds
> a
> > text box, then sets the vertical alignment for the ValueSpinner, then
> > adds the spinner. On IE this displays normally, but on chrome and
> > firefox the text box vertical alignment is inconsistent with the
> > rest of
> > the line.
> >
> > The solution is to move the call to
> > "setVerticalAlignment(ALIGN_MIDDLE);" to be before the text box is
> > added
> > (just swap lines 157 & 158)
> >
> > I also had problems with ValueSpinner's use of
> > StyleInjector.injectStylesheet() on IE. This is deprecated anyway,
> > so I
> > presume any reworking would replace this. I removed StyleInjector
> from
> > my copy of ValueSpiiner.
> >
> > HTH
> > Paul
> >
> >
> >
> >
> >
> > --
> > "There are only 10 types of people in the world: Those who understand
> > binary, and those who don't"
> >
> > >
>
> >
>
--
"There are only 10 types of people in the world: Those who understand
binary, and those who don't"
--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---
