Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [eclipse-pmc] Approval for Bug 516114 to add support for styling of the tabbed properties view

Dani, the patch will only influence the dark theme. The CSS engine is implemented in a way that it only applies a CSS handler  if a CSS property is set. The patch from Matthias define styling only for the dark theme.

Verification has been done on Windows and Linux. The CSS engine does rely on setters of the individual SWT widgets, so any giving limitation for SWT widgets will also apply for the CSS engine.

Matthias, do you have access to a Mac to validate the changes on a Mac?

> To me the change looks too big for RC1 and I'd vote to move it to 4.7.1.

I think this moves it to 4.7.1 unless Dani changes his mind.

Best regards, Lars


Am 10.05.2017 5:21 nachm. schrieb "Daniel Megert" <daniel_megert@xxxxxxxxxx>:
Did you verify the change on all platforms and all themes?

To me the change looks too big for RC1 and I'd vote to move it to 4.7.1.

Dani



From:        "Becker, Matthias" <ma.becker@xxxxxxx>
To:        "eclipse-pmc@xxxxxxxxxxx" <eclipse-pmc@xxxxxxxxxxx>
Date:        10.05.2017 08:58
Subject:        [eclipse-pmc] Approval for Bug 516114 to add support for styling of the tabbed properties view
Sent by:        eclipse-pmc-bounces@eclipse.org




Dear PMC,
 
I would like to ask for PMC permissions for “Bug 516114- [CSS] [Dark] Add support for styling of TabbedProperties in Properties View (and apply to dark layout)”.
 
This fix is need to provide the CSS styling implementation for the tabbed properties view and to provide the css for the dark theme. The tabbed properties view is used in the “Resource” perspective and also in SAPs “ABAP Development Tools” (https://tools.hana.ondemand.com/#abap). As the “ABAP Development Tools” use the tabbed properties view extensively (I think other products do as well) our Team currently cannot provide support for the dark theme.I really would like to have this in Oxygen so that Eclipse can deliver a relative decent dark theme with Oxygen. In addition, it’s my plan to provide the first support of the dark theme in the “ABAP Development Tools” in our Oxygen-based version.
 
The implementation took the implementation of “Bug 465148- [CSS] [Dark] Add support for styling Forms (and apply to dark layout)” as a blueprint. The change also contains Unit tests. The changes are limited to the org.eclipse.ui.views.properties.tabbed bundle. Hence the risk should be low. If we do not deliver it we have an inconsistency in our UI.
 
This is my first code contribution to the eclipse platform ui project. Up to now I only have contributed a HiDPI Icons to various eclipse projects. This change still needs the final code review by the Platform UI team. Lars Vogel is willing to do a detailed code review after PMC approval.
 
Best regards,
Matthias Becker
(SAP SE)_______________________________________________
eclipse-pmc mailing list
eclipse-pmc@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/eclipse-pmc



_______________________________________________
eclipse-pmc mailing list
eclipse-pmc@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/eclipse-pmc

Back to the top