Bug 516003

Summary: Define Constant for "org.eclipse.jdt.ui.Javadoc.backgroundColor"
Product: [Eclipse Project] JDT Reporter: Leo Ufimtsev <lufimtse>
Component: UIAssignee: JDT-UI-Inbox <jdt-ui-inbox>
Status: NEW --- QA Contact:
Severity: enhancement    
Priority: P3 CC: daniel_megert, gautier.desaintmartinlacaze, kalyan_prasad, Lars.Vogel, noopur_gupta, rgrunber
Version: 4.7Keywords: api
Target Milestone: ---   
Hardware: All   
OS: All   
See Also: https://bugs.eclipse.org/bugs/show_bug.cgi?id=515972
https://bugs.eclipse.org/bugs/show_bug.cgi?id=516364
Whiteboard:
Bug Depends on:    
Bug Blocks: 507072    

Description Leo Ufimtsev CLA 2017-05-01 11:53:24 EDT
Currently "org.eclipse.jdt.ui.Javadoc.backgroundColor" is used as a literal string in various parts of JDT.

I need to define a constant and refactor out all the strings.

(Btw, anyone know any good place in JDT for this? If not, I'll dig around).
Comment 1 Dani Megert CLA 2017-05-02 10:36:15 EDT
(In reply to Leo Ufimtsev from comment #0)
> Currently "org.eclipse.jdt.ui.Javadoc.backgroundColor" is used as a literal
> string in various parts of JDT.
> 
> I need to define a constant and refactor out all the strings.
> 
> (Btw, anyone know any good place in JDT for this? If not, I'll dig around).

org.eclipse.jdt.ui.text.IJavaColorConstants


I'm moving this to 4.8 since API freeze is already effective.
Comment 2 Leo Ufimtsev CLA 2017-05-02 15:39:02 EDT
(In reply to Dani Megert from comment #1)
> (In reply to Leo Ufimtsev from comment #0)
> > Currently "org.eclipse.jdt.ui.Javadoc.backgroundColor" is used as a literal
> > string in various parts of JDT.
> > 
> > I need to define a constant and refactor out all the strings.
> > 
> > (Btw, anyone know any good place in JDT for this? If not, I'll dig around).
> 
> org.eclipse.jdt.ui.text.IJavaColorConstants
> 
> 
> I'm moving this to 4.8 since API freeze is already effective.
Comment 3 Leo Ufimtsev CLA 2017-05-02 15:40:17 EDT
(In reply to Dani Megert from comment #1)
> I'm moving this to 4.8 since API freeze is already effective.

Ok, I'll implement post-oxygen release.
Comment 4 Leo Ufimtsev CLA 2018-02-06 15:02:49 EST
Moving to default inbox for now. I'll be focusing on SWT bug fixes.
Comment 5 Noopur Gupta CLA 2018-04-13 07:07:34 EDT
(In reply to Dani Megert from comment #1)
> I'm moving this to 4.8 since API freeze is already effective.
Moving to 4.9 for the same reason.
Comment 6 Kalyan Prasad Tatavarthi CLA 2018-08-27 02:38:23 EDT
(In reply to Noopur Gupta from comment #5)
> (In reply to Dani Megert from comment #1)
> > I'm moving this to 4.8 since API freeze is already effective.
> Moving to 4.9 for the same reason.

Moving to 4.10