Swing JComboBox Hintergrundfarbe

[Enigma228]

also ich habe eine JComboBox und dort möchte ich die Hintergrundfarbe und die Hintergrundfarbe der Auswahl auf eine bestimmte bringen:
dazu habe ich einen renderer geschrieben:

package doc;

import java.awt.Color;
import java.awt.Component;
import java.awt.Desktop;

import javax.swing.DefaultListCellRenderer;
import javax.swing.JLabel;
import javax.swing.JList;

public class GenderComboBoxListCellRenderer extends DefaultListCellRenderer {

	private JLabel label = null;
	 
	@Override
	public Component getListCellRendererComponent(JList list, Object value,
			int index, boolean isSelected, boolean cellHasFocus) {
		label = new JLabel("  "+value.toString());
		if(isSelected){
			setBackground(list.getSelectionBackground());
            setForeground(list.getSelectionForeground());

		}else{
			list.setBackground(new Color(255,255,215));
            setForeground(list.getForeground());

		}
		return this.label;
		
	}
	
}

ich habe mich fast komplett an das Sun Tutorial gehalten..
ABER:
die Liste der ausgewählten Elemente wird zwar gelb, aber die Selectionsfarbe fehlt und der Hintergrund der nichtaufgeklappten ComboBox ist immer noch Standardfarbe..

Was mache ich verkehrt?

 

[Verjigorm]

label.setBackground... ebenso
label.setForeground...
anstatt nur setForeground etc.

sollte es tun.

 

[Michael...]

der Hintergrund der nichtaufgeklappten ComboBox ist immer noch Standardfarbe..

Das ist bei einer ComboBox nicht so einfach, da diese eine zusammengesetzte Komponente ist.

Das JLabel im Renderer ist nicht notwendig, da DefaultListCellRender von JLabel erbt.
Wenn man es doch verwendet, sollte man ein nur einmal initialisieren und nicht jedesmal wenn die Rendererkomponente abgefragt wird. Du erzeugst da jedesmal ein neues Objekt obwohl das garnicht notwendig ist (Zeile 18). Gleiches gilt für die für die Farbe in Zeile 24. Wenn Du eine eigenens Label als Renderer verwendest, dann musst Du immer label.setForeground(...) und label.setBackground(...) verwenden. In deinem Code weist Du die Farben dem Renderer bzw. der Liste zu, aber niemals Deinem Label.
Also entweder:

...
label.setText(value.toString());
label.setBackground(...) ;
return label;

oder

...
setText(value.toString());
setBackground(...);
return this;

 

[Enigma228]

Habs korrigiert..
jetzt ist aber noch der Hintergrund der eigentlichen JComboBox grau..
wie bekomme ich die auch in Gelb..
Die Auswahlliste und die Markierung funktionieren einwandfrei..

package doc;

import java.awt.Color;
import java.awt.Component;
import javax.swing.DefaultListCellRenderer;
import javax.swing.JList;

public class GenderComboBoxListCellRenderer extends DefaultListCellRenderer {

	 
	@Override
	public Component getListCellRendererComponent(JList list, Object value,
			int index, boolean isSelected, boolean cellHasFocus) {
		this.setText(" "+value.toString());
		if(isSelected){
			setBackground(list.getSelectionBackground());
            setForeground(list.getSelectionForeground());
		}else{
			setBackground(new Color(255,255,215));
            setForeground(list.getForeground());
		}
		return this;
		
	}
	
}

 

[Michael...]

jetzt ist aber noch der Hintergrund der eigentlichen JComboBox grau..
wie bekomme ich die auch in Gelb..

Du meinst nur den Hintergrund des Textfelds ohne den Button?? Das ist bei einer ComboBox etwas komplizierter, da man nicht ohne weiteres an die richtige Komponente ran kommt. Hab mich noch nicht so intensiv mit der ComboBox auseinander gesetzt.

Ansonsten würde ich Zeile 14 und Zeile 19 noch optimieren

 

[Enigma228]

Wie kann man Z. 14 und 19 optimieren?

 

[Enigma228]

Da hier wohl keine Antwort mehr kommt, werde ich das Thema schliessen..

Nochmal danke für Hilfen...

Thomas