mbien commented on pull request #3810:
URL: https://github.com/apache/netbeans/pull/3810#issuecomment-1073060301


   the label had a preferred size of 1k. It's now set to fill-with. The gaps 
are now set to default too (one side was 12, the other 18). I am not sure why 
it looked differently on your system (I am using cinnamon too).
   
   how it looked before for me (+ it did not resize properly):
   
![default](https://user-images.githubusercontent.com/114367/159134094-30cb23c7-c12d-4ee7-ace2-a1fee1ddb086.png)
   
   post patch:
   
![patched](https://user-images.githubusercontent.com/114367/159134109-88126803-7861-4ca6-89d9-e3cd3c5a073b.png)
   
   patch below:
   you can apply it with `git am <patch file>` - feel free to squash it into 
your changes.
   
   ```patch
   From 0a94ee498dc26dec4ee84e0f154c7867783a6a0d Mon Sep 17 00:00:00 2001
   From: Michael Bien <[email protected]>
   Date: Sat, 19 Mar 2022 19:19:19 +0100
   Subject: [PATCH] layout fixes
   
    - set label to fill width + hand cursor
    - set gaps to default
   ---
    .../core/ui/options/general/AdvancedProxyPanel.form       | 8 ++++----
    .../core/ui/options/general/AdvancedProxyPanel.java       | 6 +++---
    2 files changed, 7 insertions(+), 7 deletions(-)
   
   diff --git 
a/platform/core.ui/src/org/netbeans/core/ui/options/general/AdvancedProxyPanel.form
 
b/platform/core.ui/src/org/netbeans/core/ui/options/general/AdvancedProxyPanel.form
   index 342633efbb..84d4022034 100644
   --- 
a/platform/core.ui/src/org/netbeans/core/ui/options/general/AdvancedProxyPanel.form
   +++ 
b/platform/core.ui/src/org/netbeans/core/ui/options/general/AdvancedProxyPanel.form
   @@ -42,9 +42,9 @@
        <DimensionLayout dim="0">
          <Group type="103" groupAlignment="0" attributes="0">
              <Group type="102" alignment="0" attributes="0">
   -              <EmptySpace min="-2" pref="12" max="-2" attributes="0"/>
   +              <EmptySpace min="-2" max="-2" attributes="0"/>
                  <Group type="103" groupAlignment="0" attributes="0">
   -                  <Component id="lblBasicAuthNote" min="-2" pref="1000" 
max="-2" attributes="0"/>
   +                  <Component id="lblBasicAuthNote" pref="569" max="32767" 
attributes="0"/>
                      <Component id="cbUseProxyAuthentication" alignment="1" 
max="32767" attributes="0"/>
                      <Component id="sSeparator" alignment="1" max="32767" 
attributes="1"/>
                      <Group type="102" alignment="0" attributes="0">
   @@ -92,7 +92,7 @@
                          </Group>
                      </Group>
                  </Group>
   -              <EmptySpace min="-2" pref="18" max="-2" attributes="0"/>
   +              <EmptySpace min="-2" max="-2" attributes="0"/>
              </Group>
          </Group>
        </DimensionLayout>
   @@ -386,7 +386,7 @@
              <ResourceString 
bundle="org/netbeans/core/ui/options/general/Bundle.properties" 
key="AdvancedProxyPanel.lblBasicAuthNote.text" 
replaceFormat="org.openide.util.NbBundle.getMessage({sourceFileName}.class, 
&quot;{key}&quot;)"/>
            </Property>
            <Property name="cursor" type="java.awt.Cursor" 
editor="org.netbeans.modules.form.editors2.CursorEditor">
   -          <Color id="Cursor de Mano"/>
   +          <Color id="Hand Cursor"/>
            </Property>
          </Properties>
          <Events>
   diff --git 
a/platform/core.ui/src/org/netbeans/core/ui/options/general/AdvancedProxyPanel.java
 
b/platform/core.ui/src/org/netbeans/core/ui/options/general/AdvancedProxyPanel.java
   index 9c04164a1b..c80b8e981e 100644
   --- 
a/platform/core.ui/src/org/netbeans/core/ui/options/general/AdvancedProxyPanel.java
   +++ 
b/platform/core.ui/src/org/netbeans/core/ui/options/general/AdvancedProxyPanel.java
   @@ -351,9 +351,9 @@ public class AdvancedProxyPanel extends 
javax.swing.JPanel {
            layout.setHorizontalGroup(
                
layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                .addGroup(layout.createSequentialGroup()
   -                .addGap(12, 12, 12)
   +                .addContainerGap()
                    
.addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
   -                    .addComponent(lblBasicAuthNote, 
javax.swing.GroupLayout.PREFERRED_SIZE, 1000, 
javax.swing.GroupLayout.PREFERRED_SIZE)
   +                    .addComponent(lblBasicAuthNote, 
javax.swing.GroupLayout.DEFAULT_SIZE, 569, Short.MAX_VALUE)
                        .addComponent(cbUseProxyAuthentication, 
javax.swing.GroupLayout.Alignment.TRAILING, 
javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, 
Short.MAX_VALUE)
                        .addComponent(sSeparator, 
javax.swing.GroupLayout.Alignment.TRAILING)
                        .addGroup(layout.createSequentialGroup()
   @@ -390,7 +390,7 @@ public class AdvancedProxyPanel extends 
javax.swing.JPanel {
                            
.addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                                .addComponent(tfUserName)
                                .addComponent(pfUserPassword))))
   -                .addGap(18, 18, 18))
   +                .addContainerGap())
            );
            layout.setVerticalGroup(
                
layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
   -- 
   2.35.1
   
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to