[RFC][Icedtea-Web]: Hide unused components in itw-settings.

Deepak Bhole dbhole at redhat.com
Mon Feb 28 07:21:46 PST 2011


* Andrew Su <asu at redhat.com> [2011-02-27 22:47]:
> Hello,
> 
>  The attached patch will hide the unused components in itw-settings.
>  Unused:
>    Setting cache size.
>    Selecting another JRE for plugin to bind to.
> 
> 
> Questions? Comments? Concerns?
> 
> Cheers,
>   Andrew


Indentations are off. In the interest of keeping it looking clean, can you
please fix it so that the // are right beside the code, and aligned? e.g.


> -                new SettingsPanel(Translator.R("CPTabRuntimes"), createRuntimesSettingsPanel()),
> +                //                new SettingsPanel(Translator.R("CPTabRuntimes"), createRuntimesSettingsPanel()),

to

> -                new SettingsPanel(Translator.R("CPTabRuntimes"), createRuntimesSettingsPanel()),
> +                // new SettingsPanel(Translator.R("CPTabRuntimes"), createRuntimesSettingsPanel()),

Also, please add a comment in the code stating why it is disabled, so that it
is not forgotten in the future.

Cheers,
Deepak

> diff -r 02ef9ba4d8a2 netx/net/sourceforge/jnlp/controlpanel/ControlPanel.java
> --- a/netx/net/sourceforge/jnlp/controlpanel/ControlPanel.java	Fri Feb 25 18:16:48 2011 -0500
> +++ b/netx/net/sourceforge/jnlp/controlpanel/ControlPanel.java	Sun Feb 27 21:44:06 2011 -0500
> @@ -225,7 +225,7 @@
>                  new SettingsPanel(Translator.R("CPTabDebugging"), createDebugSettingsPanel()),
>                  new SettingsPanel(Translator.R("CPTabDesktopIntegration"), createDesktopSettingsPanel()),
>                  new SettingsPanel(Translator.R("CPTabNetwork"), createNetworkSettingsPanel()),
> -                new SettingsPanel(Translator.R("CPTabRuntimes"), createRuntimesSettingsPanel()),
> +                //                new SettingsPanel(Translator.R("CPTabRuntimes"), createRuntimesSettingsPanel()),
>                  new SettingsPanel(Translator.R("CPTabSecurity"), createSecuritySettingsPanel()), };
>  
>          // Add panels.
> diff -r 02ef9ba4d8a2 netx/net/sourceforge/jnlp/controlpanel/TemporaryInternetFilesPanel.java
> --- a/netx/net/sourceforge/jnlp/controlpanel/TemporaryInternetFilesPanel.java	Fri Feb 25 18:16:48 2011 -0500
> +++ b/netx/net/sourceforge/jnlp/controlpanel/TemporaryInternetFilesPanel.java	Sun Feb 27 21:44:06 2011 -0500
> @@ -152,15 +152,15 @@
>          });
>  
>          // This is to work with how much space is available for caching.
> -        JLabel lCacheSize = new JLabel(Translator.R("TIFPCacheSize") + ":");
> -        slCacheSize = new JSlider(minSize, maxSize, Integer.parseInt(this.config.getProperty(properties[2])));
> -        slCacheSize.setMinorTickSpacing(50);
> -        slCacheSize.setPaintTicks(true);
> -        SpinnerNumberModel snmCacheSize = new SpinnerNumberModel(Integer.parseInt(this.config.getProperty(properties[2])), minSize, maxSize, 1);
> -        spCacheSize = new JSpinner(snmCacheSize);
> -
> -        slCacheSize.addChangeListener(this);
> -        spCacheSize.addChangeListener(this);
> +//        JLabel lCacheSize = new JLabel(Translator.R("TIFPCacheSize") + ":");
> +//        slCacheSize = new JSlider(minSize, maxSize, Integer.parseInt(this.config.getProperty(properties[2])));
> +//        slCacheSize.setMinorTickSpacing(50);
> +//        slCacheSize.setPaintTicks(true);
> +//        SpinnerNumberModel snmCacheSize = new SpinnerNumberModel(Integer.parseInt(this.config.getProperty(properties[2])), minSize, maxSize, 1);
> +//        spCacheSize = new JSpinner(snmCacheSize);
> +//
> +//        slCacheSize.addChangeListener(this);
> +//        spCacheSize.addChangeListener(this);
>  
>          c.gridy = 0;
>          c.gridx = 0;
> @@ -169,16 +169,16 @@
>          c.gridx = 1;
>          c.weightx = 0;
>          diskSpacePanel.add(cbCompression, c);
> -        c.gridy = 1;
> -        c.gridx = 0;
> -        c.gridwidth = GridBagConstraints.REMAINDER;
> +//        c.gridy = 1;
> +//        c.gridx = 0;
> +//        c.gridwidth = GridBagConstraints.REMAINDER;
>          c.weightx = 1;
> -        diskSpacePanel.add(lCacheSize, c);
> -        c.gridwidth = 1;
> -        c.gridy = 2;
> -        diskSpacePanel.add(slCacheSize, c);
> -        c.gridx = 1;
> -        diskSpacePanel.add(spCacheSize, c);
> +//        diskSpacePanel.add(lCacheSize, c);
> +//        c.gridwidth = 1;
> +//        c.gridy = 2;
> +//        diskSpacePanel.add(slCacheSize, c);
> +//        c.gridx = 1;
> +//        diskSpacePanel.add(spCacheSize, c);
>  
>          JPanel buttonDeleteRestore = new JPanel(new FlowLayout(FlowLayout.TRAILING));
>          JButton bViewFiles = new JButton(Translator.R("TIFPViewFiles"));




More information about the distro-pkg-dev mailing list