Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 311516 - [Theme] OS-Specific of themes
Summary: [Theme] OS-Specific of themes
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 1.0   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 1.0 M6   Edit
Assignee: Thomas Schindl CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-04 07:42 EDT by Thomas Schindl CLA
Modified: 2010-05-20 14:29 EDT (History)
1 user (show)

See Also:


Attachments
Patch (5.71 KB, patch)
2010-05-04 13:45 EDT, Thomas Schindl CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Schindl CLA 2010-05-04 07:42:06 EDT
I'm not sure whether the current implementation of os and ws detection is a good one. We need to make it possible that we only have 1-Theme-Id and so I think the OS/WS contraint is not make sense 100% there. If we have them I think OS should allow at least to be:
* empty => applies to all OSes
* commaseparated => applies on to the given ones
Comment 1 Thomas Schindl CLA 2010-05-04 07:50:35 EDT
checked in the proposed changes to CVS
Comment 2 Thomas Schindl CLA 2010-05-04 13:44:08 EDT
Reopening to discuss variables in the Stylepath.
Comment 3 Thomas Schindl CLA 2010-05-04 13:45:42 EDT
Created attachment 166990 [details]
Patch

Bogdan - what do you think does it make sense if we add variable substitution in the stylesheet path? This would allow us to define one theme with different basecss of different os, ...
Comment 4 Thomas Schindl CLA 2010-05-04 13:46:43 EDT
the urls would look like this then: "$os-$ws$-dark.css"
Comment 5 Thomas Schindl CLA 2010-05-20 11:26:31 EDT
I checked in the change which allows people to use $ws$ and $os$ in the style path
Comment 6 Bogdan Gheorghe CLA 2010-05-20 14:29:42 EDT
Sorry I didn't get back to you earlier, I was out of the office for the last couple of weeks. 

I suppose a user might not care about os/ws, so having a way to opt out of it is OK with me. I think your variable substitution idea is good.