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]> Angelo, I've opened 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]> To: E4 Project developer mailing list <[email protected]> Date: 02/19/2010 10:34 AM Subject: Re: [e4-dev] e4 Contacs Demo CSS Styling broken Sent by: [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]> Hi Kai, On Fri, Feb 19, 2010 at 3:57 AM, Toedter, Kai <[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 Regards, Remy ---------- Remy Suen Eclipse Platform/UI Committer IBM Ottawa 613-356-5162 _______________________________________________ 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 _______________________________________________ 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
