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

Bug 371937

Summary: Changing font colour to White means never seeing your line numbers
Product: [ECD] Orion Reporter: Ken Walker <ken_walker>
Component: ClientAssignee: Silenio Quarti <Silenio_Quarti>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: antonm, eclipse.felipe, Mike_Wilson, Silenio_Quarti
Version: 0.4Flags: eclipse.felipe: review+
ken_walker: review+
Target Milestone: 0.3 RC3   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:

Description Ken Walker CLA 2012-02-17 15:10:59 EST
The font colour effects the line number display.  The background does not change the background of that area so if you change your font to white, you don't have line numbers any more.
Comment 1 Silenio Quarti CLA 2012-02-21 11:40:35 EST
Here is a fix (branch=bug371937):

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?h=bug371937&id=ca592f60f56585e4e72de02c613dd3c724145eac

This fix makes the background property affect the rulers as well. I believe this is the safest to do for 0.4.  After 0.4, you should add more entries in the settings page to control those colours independently.  

Other colour settings that should be in the page:

- the highlight current line colour
- ruler color/background
- tags/html markup in js doc comments
Comment 2 Felipe Heidrich CLA 2012-02-21 11:51:52 EST
safe fix.
Comment 3 Anton McConville CLA 2012-02-21 12:04:06 EST
Made a new bug to track this request.

https://bugs.eclipse.org/bugs/show_bug.cgi?id=372139

(In reply to comment #1)
> Here is a fix (branch=bug371937):
> 
> http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?h=bug371937&id=ca592f60f56585e4e72de02c613dd3c724145eac
> 
> This fix makes the background property affect the rulers as well. I believe
> this is the safest to do for 0.4.  After 0.4, you should add more entries in
> the settings page to control those colours independently.  
> 
> Other colour settings that should be in the page:
> 
> - the highlight current line colour
> - ruler color/background
> - tags/html markup in js doc comments
Comment 4 Ken Walker CLA 2012-02-21 12:37:51 EST
As project lead I +1 this for RC3