Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 410192

Summary: There should be a way to override global css without modifying ide.css
Product: [ECD] Orion Reporter: Malgorzata Janczarska <malgorzata.tomczyk>
Component: ClientAssignee: Malgorzata Janczarska <malgorzata.tomczyk>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: ken_walker, Szymon.Brandys
Version: 3.0Flags: Szymon.Brandys: review+
Target Milestone: 3.0 RC2   
Hardware: PC   
OS: All   
Whiteboard:

Description Malgorzata Janczarska CLA 2013-06-07 10:11:39 EDT
Same as bug 410177, but css files.
Comment 1 Malgorzata Janczarska CLA 2013-06-10 10:19:48 EDT
This is a commit that I would like to merge:
https://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?h=Bug410192&id=16466c19db6b48cb77325442e07fd987e21d19a0
I added css/customide.css file that can be overwritten by other contributor to apply some custom css.

Ken, it's already RC2. Do I need your +1 to commit this?
Comment 2 Malgorzata Janczarska CLA 2013-06-10 11:39:20 EDT
Another way to solve this problem is to use the theme.css we already have instead of adding another css. This would however require similar changes, because right now styles from theme.css are overwritten by ide.css and it should be the other may around.
Comment 3 Malgorzata Janczarska CLA 2013-06-11 09:43:42 EDT
(In reply to comment #2)
> Another way to solve this problem is to use the theme.css we already have
> instead of adding another css. This would however require similar changes,
> because right now styles from theme.css are overwritten by ide.css and it should
> be the other may around.
Those are the changes that need to be made to make theme.css have higher priority then ide.css. For original Orion this changes nothing, because styles in theme.css do not overlap with styles in ide.css.
Comment 4 Malgorzata Janczarska CLA 2013-06-11 10:28:22 EDT
Ken said I don't need his +1 if it's not a feature, so Szymon's review is enough to merge it.
Comment 5 Malgorzata Janczarska CLA 2013-06-11 11:10:31 EDT
merged.