LGTM On Mon, Dec 22, 2008 at 11:41 AM, <jlaba...@google.com> wrote:
> Reviewers: ecc, > > Description: > The dateBoxFormatError style name is applied to a DateBox with an > invalid date entry. This patch adds style definitions to the dark and > chrome style themes. Style definitions were already added for the > standard theme. > > Patch was submitted TBR by ecc. > > Please review this at http://gwt-code-reviews.appspot.com/1601 > > Affected files: > user/src/com/google/gwt/user/theme/chrome/public/gwt/chrome/chrome.css > user/src/com/google/gwt/user/theme/chrome/public/gwt/chrome/chrome_rtl.css > user/src/com/google/gwt/user/theme/dark/public/gwt/dark/dark.css > user/src/com/google/gwt/user/theme/dark/public/gwt/dark/dark_rtl.css > > user/src/com/google/gwt/user/theme/standard/public/gwt/standard/standard.css > > > Index: > user/src/com/google/gwt/user/theme/standard/public/gwt/standard/standard.css > =================================================================== > --- > user/src/com/google/gwt/user/theme/standard/public/gwt/standard/standard.css > (revision 4363) > +++ > user/src/com/google/gwt/user/theme/standard/public/gwt/standard/standard.css > (working copy) > @@ -1093,10 +1093,9 @@ > .gwt-DateBox input { > width: 8em; > } > -.dateBoxFormatError{ > - background:#ffcccc; > +.dateBoxFormatError { > + background: #ffcccc; > } > - > .dateBoxPopup { > } > > Index: user/src/com/google/gwt/user/theme/dark/public/gwt/dark/dark.css > =================================================================== > --- user/src/com/google/gwt/user/theme/dark/public/gwt/dark/dark.css > (revision 4363) > +++ user/src/com/google/gwt/user/theme/dark/public/gwt/dark/dark.css > (working copy) > @@ -999,6 +999,10 @@ > .gwt-DateBox input { > width: 8em; > } > +.dateBoxFormatError { > + background: #a8eeff; > + color: red; > +} > .dateBoxPopup { > } > > Index: user/src/com/google/gwt/user/theme/dark/public/gwt/dark/dark_rtl.css > =================================================================== > --- user/src/com/google/gwt/user/theme/dark/public/gwt/dark/dark_rtl.css > (revision 4363) > +++ user/src/com/google/gwt/user/theme/dark/public/gwt/dark/dark_rtl.css > (working copy) > @@ -1000,6 +1000,10 @@ > .gwt-DateBox input { > width: 8em; > } > +.dateBoxFormatError { > + background: #a8eeff; > + color: red; > +} > .dateBoxPopup { > } > > Index: > user/src/com/google/gwt/user/theme/chrome/public/gwt/chrome/chrome.css > =================================================================== > --- user/src/com/google/gwt/user/theme/chrome/public/gwt/chrome/chrome.css > (revision 4363) > +++ user/src/com/google/gwt/user/theme/chrome/public/gwt/chrome/chrome.css > (working copy) > @@ -1094,6 +1094,9 @@ > .gwt-DateBox input { > width: 8em; > } > +.dateBoxFormatError { > + background: #eed6d6; > +} > .dateBoxPopup { > } > > Index: > user/src/com/google/gwt/user/theme/chrome/public/gwt/chrome/chrome_rtl.css > =================================================================== > --- > user/src/com/google/gwt/user/theme/chrome/public/gwt/chrome/chrome_rtl.css > (revision 4363) > +++ > user/src/com/google/gwt/user/theme/chrome/public/gwt/chrome/chrome_rtl.css > (working copy) > @@ -1095,6 +1095,9 @@ > .gwt-DateBox input { > width: 8em; > } > +.dateBoxFormatError { > + background: #eed6d6; > +} > .dateBoxPopup { > } > > > > -- "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 -~----------~----~----~----~------~----~------~--~---