🚀 go-pugleaf

On Sat, 20 Mar 2004 13:55:28 -0800, John McGrath  [TeamB] wrote:

> On 3/20/2004 at 8:28:45 AM, nospam wrote:
> 
>> [quoted text muted]
> 
> 
>> [quoted text muted]
> A cell renderer is an object that provides a Component that will be used
> to do the painting.  Normally, the getListCellRendererComponent() method
> will configure the properties of the Component so it will render the cell
> appropriately.
> 
> As is often the case, this cell renderer returns the same component each
> time getListCellRendererComponent() is called.  Since the above code
> configures the "Selected" property only if cellHasFocus is true, it will
> be left in whatever state it was in previous time it was used.
> 
> So you need to change the code to set the "Selected" property even when
> "cellHasFocus" is false.
> 
>     if (cellHasFocus) {
>         if (isSelected) {
>             setSelected(false);
>         }
>         else {
>             setSelected(true);
>         }
>         }
>     else {
>         setSelected(true);
>     }
>     }
> Of course, these 11 lines of code could be replace with the following
> single line:
> 
>     setSelected( cellHasFocus & !isSelected );
> 
> This is much simpler and more efficient and it also makes it far more
> clear that the "Selected" property will always be set.
> 
>> [quoted text muted]
> 
> BTW, many people frown on anonymous messages.  I can understand why you
> might want to omit your e-mail address, but I can think of no reason to
> omit your name.


Thank you for your response.  I tried what you suggested,
and it still does not work.  Here is my entire renderer
class.  I've commented the code you provided below, and
what it does.

Thanks
-Bernie



public class CheckListRenderer  extends JCheckBox implements ListCellRenderer {

int numSelected = 0;

  public CheckListRenderer() {
          setOpaque(true);
      }


  public Component getListCellRendererComponent(
         JList list,
         Object value,
         int index,
         boolean isSelected,
         boolean cellHasFocus)
     {
         setText(value.toString());
         setBackground(isSelected ? Color.blue : Color.lightGray);
         setForeground(isSelected ? Color.white : Color.black);


//THIS ISN'T CHECKING THE CHECKBOXES

         setSelected( cellHasFocus & !isSelected );


//THIS CHECKS ALL CHECKBOXES INITIALLY, BUT DOESN'T UNCHECK.

//         if (cellHasFocus) {
//                 if (isSelected) {
//                     setSelected(false);
//                 }
//                 else {
//                     setSelected(true);
//                 }
//                 }
//             else {
//                 setSelected(true);
//             }



                System.out.println(value.toString() + " is selected");
//                System.out.println("index = " + String.valueOf(index) );




//        else   // Keep CheckBox the same....
//        {
//          if (isSelected())  {
//            setSelected(false);
//            }
//          else {
//              setSelected(false);
//           }
//        }




//        if (isSelected) {
//             setSelected(false);
//         }
//        else {
//            setSelected(true);
//         }



        return this;
     }





}