Hi Eric, I have getted the last version of CSS engine and engine has not changed about IDynamicPseudoClasses. I was creating the patch by restart from scratch my patch. Do you want I post my patch which split SWTElement per type of SWT widget and which remove the awfull IDynamicPseudoClasses?
Regards Angelo 2010/2/19 Eric Moffatt <[email protected]> > > Angelo, we'd be better off to let Bogdan handle the CSS questions, he's > deeply familiar with the current implementation...I've specifically CC'd him > in case he doesn't follow the list. > > BTW, I've already checked in the fix for the text color... > > Onwards, > Eric > > > From: > Angelo zerr <[email protected]> > To: > E4 Project developer mailing list <[email protected]> > Date: 02/19/2010 10:56 AM > Subject: Re: [e4-dev] e4 Contacs Demo CSS Styling broken Sent by: > [email protected] > > ------------------------------ > > > > Ok Kai I will try to do that. > > Eric thank alot for your intention. I'm wiating for your answer to start > the developmenet. I think we should doen that into several steps : > > 1. split SWTElement per SWT Widget. benefit with that it's that the awfull > IDynamicPseudoClassesHandler could be removed. > > 2. After I think we should remove ICSSPropertyHandler. Apply/reset of > valiue must be done by the SWTElement. > > Regards Angelo > > 2010/2/19 Eric Moffatt <*[email protected]* <[email protected]>> > > Angelo, I've opened > * > **https://bugs.eclipse.org/bugs/show_bug.cgi?id=303328*<https://bugs.eclipse.org/bugs/show_bug.cgi?id=303328> > > to track this. I've already captured your proposed fix (thanks again) and > I'll be talking with Bogdan about it today. > > > > From: Angelo zerr <*[email protected]* <[email protected]>> To: E4 > Project developer mailing list <*[email protected]* <[email protected]>> > Date: 02/19/2010 10:34 AM Subject: Re: [e4-dev] e4 Contacs Demo CSS > Styling broken Sent by: > *[email protected]*<[email protected]> > > ------------------------------ > > > > > Hi Kai, > > I have seen problem with SWT Label and to correct the problem you must add > this code : > > else if (widget instanceof Control) { > ((Control) widget).setForeground(newColor); > } > > at end of methode applyCSSPropertyColo of the class > CSSPropertyTextSWTHandler > > Here the complete code this method. > > public void applyCSSPropertyColor(Object element, CSSValue value, > String pseudo, CSSEngine engine) throws Exception { > Widget widget = (Widget) element; > if (value.getCssValueType() == CSSValue.CSS_PRIMITIVE_VALUE) { > Color newColor = (Color) engine.convert(value, Color.class, > widget > .getDisplay()); > if (widget instanceof CTabItem) { > CTabFolder folder = ((CTabItem) widget).getParent(); > if ("selected".equals(pseudo)) { > folder.setSelectionForeground(newColor); > } else { > folder.setForeground(newColor); > } > } > else if (widget instanceof Control) { > ((Control) widget).setForeground(newColor); > } > } > } > > > I would like just say that it should be very good to refactor CSS engine to > manage to : > > * 1) Have an SWTElement per SWT Type widget > * 2) SWTElement should (perhaps) apply/reset styles in order to avoiding > cast Widget into *Handler. > * 3) use extension point or develop OSGi Extender to customize the CSS > engine. > > For the point 1 I can done that if you wish. I'm waiting for if you are OK > with that. > > Regards Angelo > > 2010/2/19 Remy Suen <*[email protected]* <[email protected]>> > Hi Kai, > > > > On Fri, Feb 19, 2010 at 3:57 AM, Toedter, Kai > <*[email protected]*<[email protected]>> > wrote: > > When I started the e4 contacts demo I noticed: > > > > - Label css styling does not work anymore > > - Tabs css styling does not work anymore > > - Toolbar is not styled correctly > > - Toolbar has a border that is not styled > > - Toolbar does not fill the space to the right > > I see this on my computer on XP as well. I believe Eric also noticed it on > his machine. > > > > I would like to help fixing the above topics. Who can I best talk with to > proceed? > > Eric and/or Bogdan would be the best people to approach. Can you file a bug > about this matter? Do you know if the setForeground(Color) methods are even > being called? Please feel free to drop by on IRC to speak with us directly. > > For the tabs problem, we removed ETabFolder but seems to have forgotten to > change the CSS files. I have just released changes to the CSS files. Please > synchronize again to get the changes.* > **https://bugs.eclipse.org/bugs/show_bug.cgi?id=301739*<https://bugs.eclipse.org/bugs/show_bug.cgi?id=301739> > > Regards, > Remy > > ---------- > Remy Suen > Eclipse Platform/UI Committer > IBM Ottawa > 613-356-5162 > > _______________________________________________ > e4-dev mailing list* > **[email protected]* <[email protected]>* > **https://dev.eclipse.org/mailman/listinfo/e4-dev*<https://dev.eclipse.org/mailman/listinfo/e4-dev> > > _______________________________________________ > e4-dev mailing list* > **[email protected]* <[email protected]>* > **https://dev.eclipse.org/mailman/listinfo/e4-dev*<https://dev.eclipse.org/mailman/listinfo/e4-dev> > > > _______________________________________________ > e4-dev mailing list* > **[email protected]* <[email protected]>* > **https://dev.eclipse.org/mailman/listinfo/e4-dev*<https://dev.eclipse.org/mailman/listinfo/e4-dev> > > _______________________________________________ > e4-dev mailing list > [email protected] > https://dev.eclipse.org/mailman/listinfo/e4-dev > > > > _______________________________________________ > e4-dev mailing list > [email protected] > https://dev.eclipse.org/mailman/listinfo/e4-dev > >
_______________________________________________ e4-dev mailing list [email protected] https://dev.eclipse.org/mailman/listinfo/e4-dev
