JProgressBar in Napkin look and feel is not working

Hi,

Please have a look at the following code

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import napkin.NapkinLookAndFeel;

public class ProgressBar
{
 private JButton start;
 private JPanel panel;
 private JProgressBar bar;
 Thread t;
 JFrame frame;
 
 public ProgressBar()
 {
 start = new JButton("Start");
 start.addActionListener(new StartAction());
 
 bar = new JProgressBar(0,100);
 bar.setVisible(false);
 
 
 panel = new JPanel();
 panel.setLayout(new FlowLayout());
 panel.add(start);
 panel.add(bar);
 
 t = new Thread(new Threads());
 
 frame = new JFrame();
 frame.add(panel);
 frame.setUndecorated(false);
 
 try
 {
 UIManager.setLookAndFeel(new NapkinLookAndFeel());
 SwingUtilities.updateComponentTreeUI(frame);
 
 }
 catch(Exception e)
 {
 e.printStackTrace();
 }
 
 frame.setVisible(true);
 frame.setSize(200,100);
 frame.validate();
 frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
 
 
 
 }
 
 private class StartAction implements ActionListener
 { 
 public void actionPerformed(ActionEvent ae)
 {
 t.start();
 }
 }
 
 private class Threads implements Runnable
 {
 public void run()
 {
 bar.setVisible(true);
 
 for(int i=0;i<101;i++)
 {
 bar.setValue(i);
 bar.repaint();
 
 
 try
 {
 Thread.sleep(1000);
 }
 catch(InterruptedException ie)
 {
 ie.printStackTrace();
 }
 }
 
 
 }
 }
 
 public static void main(String[]args)
 {
 
 ProgressBar pb = new ProgressBar();
 
 }
}

In this code, the JProgressBar is not getting updated because of the napkin look and feel. When I change the look and feel to system look and feel, it works fine. Please help.

I quote from this thread:

Rob Spoor wrote:Don't use a Thread, use a javax.swing.Timer. Read Concurrency in Swing for more information.

Rob Spoor wrote:I quote from this thread:

Rob Spoor wrote:Don't use a Thread, use a javax.swing.Timer. Read Concurrency in Swing for more information.

Thanks for the reply. My original application will not use either timers or threads. So, I completely removed the thread and tested. Still no good

. Please help

What is a progressbar without a thread or a timer?

Michael Dunn wrote:What is a progressbar without a thread or a timer?

It shows how many images has been converted as a progress bar percentage, yes, I am creating a image converting software.

Michael Dunn wrote:What is a progressbar without a thread or a timer?

errr, anyway, is that is a MUST to use either thread or Timer? But my progressbar has to show the workload done, so things like Thread.sleep(1000) like things cannot be applied.

Your progress bar needs to show the workload, true, but how do you update it? Do you perform all the processing from an actionPerformed event or any other event on the EDT? If so, then this will block the EDT and therefore will let only the last progress bar value be shown.

Rob Spoor wrote:Your progress bar needs to show the workload, true, but how do you update it? Do you perform all the processing from an actionPerformed event or any other event on the EDT? If so, then this will block the EDT and therefore will let only the last progress bar value be shown.

err, this is not much clear to me, so I am posting my code with the edits I have mentioned

import java.awt.*; 
 import java.awt.event.*; 
 import javax.swing.*; 
 import napkin.NapkinLookAndFeel; 
 
 public class ProgressBar 
 { 
 private JButton start; 
 private JPanel panel; 
 private JProgressBar bar; 
 Thread t; 
 JFrame frame; 
 
 public ProgressBar() 
 { 
 start = new JButton("Start"); 
 start.addActionListener(new StartAction()); 
 
 bar = new JProgressBar(0,100); 
 bar.setVisible(false); 
 
 
 panel = new JPanel(); 
 panel.setLayout(new FlowLayout()); 
 panel.add(start); 
 panel.add(bar); 
 
 t = new Thread(new Threads()); 
 
 frame = new JFrame(); 
 frame.add(panel); 
 frame.setUndecorated(false); 
 
 try 
 { 
 UIManager.setLookAndFeel(new NapkinLookAndFeel()); 
 SwingUtilities.updateComponentTreeUI(frame); 
 
 } 
 catch(Exception e) 
 { 
 e.printStackTrace(); 
 } 
 
 frame.setVisible(true); 
 frame.setSize(200,100); 
 frame.validate(); 
 frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
 
 
 
 
 } 
 
 private class StartAction implements ActionListener 
 { 
 public void actionPerformed(ActionEvent ae) 
 { 
 bar.setVisible(true); 
 
 for(int i=0;i<101;i++) 
 { 
 bar.setValue(i); 
 bar.repaint(); 
		}
 } 
 } 
 
 
 
 public static void main(String[]args) 
 { 
 
 ProgressBar pb = new ProgressBar(); 
 
 } 
 }

I can use a timer as you have mentioned, but, according to this code, is it necessary? And, won't it be a performance hit, because if I can do all the operations within 5 seconds, the Thread or Timer will take time to sleep..Please make this clear to me, honestly, this is the first time I am using JProgressBar.

OK..I copied a code from the internet to test the timer, and adjusted the napkin look and feel. But unfortunatly that is also not working. The code is below

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.text.html.*;
import napkin.NapkinLookAndFeel;

public class SwingProgressBar{
  final static int interval = 1000;
  int i;
  JLabel label;
  JProgressBar pb;
  Timer timer;
  JButton button;

public SwingProgressBar() {
  JFrame frame = new JFrame("Swing Progress Bar");
  button = new JButton("Start");
  button.addActionListener(new ButtonListener());

pb = new JProgressBar(0, 20);
  pb.setValue(0);
  pb.setStringPainted(true);

label = new JLabel("Roseindia.net");
  
  JPanel panel = new JPanel();
  panel.add(button);
  panel.add(pb);

JPanel panel1 = new JPanel();
  panel1.setLayout(new BorderLayout());
  panel1.add(panel, BorderLayout.NORTH);
  panel1.add(label, BorderLayout.CENTER);
  panel1.setBorder(BorderFactory.createEmptyBorder(20, 20, 20, 20));
  frame.setContentPane(panel1);
 
  frame.setUndecorated(false);
  
  try
  {
      UIManager.setLookAndFeel(new NapkinLookAndFeel());
  }
  catch(Exception e)
  {
      
  }
   SwingUtilities.updateComponentTreeUI(frame);
   
    
  frame.setVisible(true);
  frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

//Create a timer.
 timer = new Timer(interval, new ActionListener() {
 public void actionPerformed(ActionEvent evt) {
 if (i == 20){
 Toolkit.getDefaultToolkit().beep();
 timer.stop();
 button.setEnabled(true);
 pb.setValue(0);
 String str = "<html>" + "" + "" + 
"Downloading completed." + "" + "" + "</html>";
 label.setText(str);
 }
 i = i + 1;
 pb.setValue(i);
 }
 });
 }

class ButtonListener implements ActionListener {
 public void actionPerformed(ActionEvent ae) {
 button.setEnabled(false);
 i = 0;
 String str = "<html>" + "" + "" + 
"Downloading is in process......." + "" + "" + "</html>";
 label.setText(str);
 timer.start();
 }
 }
 
 public static void main(String[] args) {
 SwingProgressBar spb = new SwingProgressBar();
 
 
 }
}

This look and feel is a trouble maker anyway. So, is there any way to change the look and feel of the progressbar? I mean JUST the progress bar to System look and feel while all other components are napkin. Please help

just tested the code in the recent post, and it does actually work.
problem seems to be the napkin l&f paints everything the same color,
the text, the backround, the 'progress' etc - so you don't see any change.

try adding these lines immediately after the indicated line

if you get it working, then change to whatever colors you want

Michael Dunn wrote:just tested the code in the recent post, and it does actually work.
problem seems to be the napkin l&f paints everything the same color,
the text, the backround, the 'progress' etc - so you don't see any change.

try adding these lines immediately after the indicated line

UIManager.setLookAndFeel(new NapkinLookAndFeel());<-------------- UIManager.put("ProgressBar.selectionBackground",Color.BLACK); UIManager.put("ProgressBar.selectionForeground",Color.BLACK); UIManager.put("ProgressBar.foreground",Color.GRAY);

if you get it working, then change to whatever colors you want

wow, yeah!! It worked Michael!!! Thanks a lot for the help!!! That was great!!

Rob Spoor wrote:Your progress bar needs to show the workload, true, but how do you update it? Do you perform all the processing from an actionPerformed event or any other event on the EDT? If so, then this will block the EDT and therefore will let only the last progress bar value be shown.

Thanks a lot for the idea about the Timer, Rob!! Now I can stop, start, restart, and pause the workload!!! Amazing!!!

You're welcome.