Author: pmouawad
Date: Sun Jul 15 12:40:08 2018
New Revision: 1835968

URL: http://svn.apache.org/viewvc?rev=1835968&view=rev
Log:
Bug 62535 - View Results Tree : XPath2 Tester not displaying correctly
Bugzilla Id: 62535

Modified:
    
jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsXPath2.java

Modified: 
jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsXPath2.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsXPath2.java?rev=1835968&r1=1835967&r2=1835968&view=diff
==============================================================================
--- 
jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsXPath2.java 
(original)
+++ 
jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsXPath2.java 
Sun Jul 15 12:40:08 2018
@@ -18,7 +18,6 @@
 
 package org.apache.jmeter.visualizers;
 
-import java.awt.BorderLayout;
 import java.awt.Color;
 import java.awt.Dimension;
 import java.awt.GridBagConstraints;
@@ -29,7 +28,6 @@ import java.awt.event.ActionListener;
 import java.util.ArrayList;
 import java.util.List;
 
-import javax.swing.Box;
 import javax.swing.JButton;
 import javax.swing.JCheckBox;
 import javax.swing.JLabel;
@@ -37,6 +35,7 @@ import javax.swing.JPanel;
 import javax.swing.JScrollPane;
 import javax.swing.JSplitPane;
 import javax.swing.JTabbedPane;
+import javax.swing.JTextField;
 
 import org.apache.commons.lang3.exception.ExceptionUtils;
 import org.apache.jmeter.extractor.XPath2Extractor;
@@ -45,7 +44,6 @@ import org.apache.jmeter.gui.util.JTextS
 import org.apache.jmeter.samplers.SampleResult;
 import org.apache.jmeter.util.JMeterUtils;
 import org.apache.jmeter.util.XPathUtil;
-import org.apache.jorphan.gui.JLabeledTextField;
 import org.fife.ui.rsyntaxtextarea.SyntaxConstants;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -66,7 +64,7 @@ public class RenderAsXPath2 implements R
 
     private JSyntaxTextArea xmlDataField;
 
-    private JLabeledTextField xpathExpressionField;
+    private JTextField xpathExpressionField;
 
     private JSyntaxTextArea xpathResultField;
 
@@ -210,14 +208,15 @@ public class RenderAsXPath2 implements R
         JScrollPane xmlDataPane = JTextScrollPane.getInstance(xmlDataField, 
true);
         xmlDataPane.setMinimumSize(new Dimension(0, 100));
         xmlDataPane.setPreferredSize(new Dimension(0, 200));
-
+        JPanel highPane = new JPanel(new GridLayout(1, 1));
+        highPane.add(xmlDataPane);
         JPanel pane = new JPanel(new GridLayout(1, 1));
 
         JSplitPane mainSplit = new JSplitPane(JSplitPane.VERTICAL_SPLIT,
-                xmlDataPane, createXpathExtractorTasksPanel());
+                highPane, createXpathExtractorTasksPanel());
         mainSplit.setDividerLocation(0.5d);
         mainSplit.setOneTouchExpandable(true);
-        pane.add(mainSplit, BorderLayout.CENTER);
+        pane.add(mainSplit);
         return pane;
     }
 
@@ -227,8 +226,9 @@ public class RenderAsXPath2 implements R
      * @return XPath task pane
      */
     private JPanel createXpathExtractorTasksPanel() {
-        Box selectorAndButton = Box.createHorizontalBox();
-        xpathExpressionField = new 
JLabeledTextField(JMeterUtils.getResString("xpath_tester_field")); // 
$NON-NLS-1$
+        xpathExpressionField = new JTextField(20);
+        xpathExpressionField.setMinimumSize(new Dimension(50, 20));
+        JLabel label = new 
JLabel(JMeterUtils.getResString("xpath_tester_field")); // $NON-NLS-1$
 
         JButton xpathTester = new 
JButton(JMeterUtils.getResString("xpath_tester_button_test")); // $NON-NLS-1$
         xpathTester.setActionCommand(XPATH_TESTER_COMMAND);
@@ -238,57 +238,56 @@ public class RenderAsXPath2 implements R
         xpathTesterNamespaces.setActionCommand(XPATH_NAMESPACES_COMMAND);
         xpathTesterNamespaces.addActionListener(this);
 
-        selectorAndButton.add(xpathExpressionField);
-        selectorAndButton.add(xpathTester);
-        selectorAndButton.add(xpathTesterNamespaces);
-
         JPanel panel = new JPanel(new GridBagLayout());
         GridBagConstraints gbc = new GridBagConstraints();
+
         initConstraints(gbc);
+        panel.add(label, gbc.clone());
+        gbc.gridx++;
+        gbc.gridwidth = 2;
+        panel.add(xpathExpressionField, gbc.clone());
+        gbc.gridx+=2;
+        gbc.gridwidth = 2;
+        panel.add(xpathTester, gbc.clone());
+        gbc.gridx+=2;
+        gbc.gridwidth = 2;
+        panel.add(xpathTesterNamespaces, gbc.clone());
+        gbc.weighty = 1;
+        gbc.gridx = 0;
+        gbc.gridy++;
         gbc.gridwidth = 2;
-        panel.add(selectorAndButton, gbc.clone());
-
-        resetContraints(gbc);
         panel.add(new 
JLabel(JMeterUtils.getResString("xpath_extractor_user_namespaces")), 
gbc.clone());
         gbc.gridx++;
-        gbc.weightx = 1;
-        gbc.fill = GridBagConstraints.HORIZONTAL;
-        namespacesTA = JSyntaxTextArea.getInstance(5, 80);
+        gbc.gridwidth = 6;
+        namespacesTA = JSyntaxTextArea.getInstance(5, 40);
         panel.add(JTextScrollPane.getInstance(namespacesTA, true), 
gbc.clone());
 
-        resetContraints(gbc);
-        gbc.gridwidth = 2;
+        gbc.gridx = 0;
+        gbc.gridy++;
+        gbc.gridwidth = 8;
         panel.add(getFragment, gbc.clone());
 
-        resetContraints(gbc);
-        xpathResultField = JSyntaxTextArea.getInstance(10, 80, true);
+        gbc.gridx = 0;
+        gbc.gridy++;
+        gbc.gridwidth = 8;
+        xpathResultField = JSyntaxTextArea.getInstance(10, 70, true);
         xpathResultField.setEditable(false);
         xpathResultField.setLineWrap(true);
         xpathResultField.setWrapStyleWord(true);
-        gbc.gridwidth = 2;
         gbc.fill = GridBagConstraints.HORIZONTAL;
         panel.add(JTextScrollPane.getInstance(xpathResultField, true), 
gbc.clone());
 
         return panel;
     }
 
-    private void resetContraints(GridBagConstraints gbc) {
-        gbc.gridx = 0;
-        gbc.gridy++;
-        gbc.gridwidth = 1;
-        gbc.weightx = 0;
-        gbc.fill = GridBagConstraints.NONE;
-    }
-
     private void initConstraints(GridBagConstraints gbc) {
-        gbc.anchor = GridBagConstraints.NORTHWEST;
-        gbc.fill = GridBagConstraints.NONE;
+        gbc.fill = GridBagConstraints.HORIZONTAL;
         gbc.gridheight = 1;
         gbc.gridwidth = 1;
         gbc.gridx = 0;
         gbc.gridy = 0;
-        gbc.weightx = 0;
-        gbc.weighty = 0;
+        gbc.weightx = 1;
+        gbc.weighty = 1;
     }
     
     /** {@inheritDoc} */


Reply via email to