[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [e4-dev] e4 Contacs Demo CSS Styling broken
|
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 <angelo.zerr@xxxxxxxxx>
|
To:
| E4 Project developer mailing list <e4-dev@xxxxxxxxxxx>
|
Date:
| 02/19/2010 10:56 AM
|
Subject:
| Re: [e4-dev] e4 Contacs Demo CSS Styling
broken
|
Sent by:
| e4-dev-bounces@xxxxxxxxxxx |
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 <emoffatt@xxxxxxxxxx>
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.
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 <remysuen@xxxxxxxxxx>
Hi Kai,
On Fri, Feb 19, 2010 at 3:57 AM, Toedter, Kai <kai.toedter@xxxxxxxxxxx>
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
e4-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/e4-dev
_______________________________________________
e4-dev mailing list
e4-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/e4-dev
_______________________________________________
e4-dev mailing list
e4-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/e4-dev
_______________________________________________
e4-dev mailing list
e4-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/e4-dev