Merge pull request #865 from aws-lumberyard-dev/LYN-2139

LYN-2139 UX: Goto Position modal is unnecessarily massive

Code/Sandbox/Editor/GotoPositionDlg.cpp

@@ -34,6 +34,7 @@ CGotoPositionDlg::CGotoPositionDlg(QWidget* pParent /*=NULL*/)
 {
     m_ui->setupUi(this);
     setWindowFlags(windowFlags() & ~Qt::WindowContextHelpButtonHint);
+    setFixedSize(size());
     OnInitDialog();
 
     auto doubleValueChanged = static_cast<void(QDoubleSpinBox::*)(double)>(&QDoubleSpinBox::valueChanged);
@@ -98,6 +99,9 @@ void CGotoPositionDlg::OnInitDialog()
     m_ui->m_dymSegX->setVisible(false);
     m_ui->m_dymSegY->setVisible(false);
 
+    // Ensure the goto button is highlighted correctly.
+    m_ui->pushButton->setDefault(true);
+
     OnUpdateNumbers();
 }
 

Code/Sandbox/Editor/GotoPositionDlg.ui

@@ -6,28 +6,16 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>358</width>
-    <height>198</height>
+    <width>290</width>
+    <height>180</height>
    </rect>
   </property>
   <property name="windowTitle">
    <string>Go to Position</string>
   </property>
+   <layout class="QVBoxLayout" name="verticalLayout">
+     <item>
        <layout class="QGridLayout" name="gridLayout" columnstretch="0,1,0,0,1,0,0,1">
-   <item row="6" column="0" colspan="2">
-    <widget class="QPushButton" name="pushButton">
-     <property name="text">
-      <string>Go To</string>
-     </property>
-    </widget>
-   </item>
-   <item row="6" column="3" colspan="2">
-    <widget class="QPushButton" name="pushButton_2">
-     <property name="text">
-      <string>Cancel</string>
-     </property>
-    </widget>
-   </item>
          <item row="5" column="6" colspan="2">
            <spacer name="horizontalSpacer">
              <property name="orientation">
@@ -189,6 +177,39 @@
            <widget class="QSpinBox" name="m_dymSegY"/>
          </item>
        </layout>
+     </item>
+     <item>
+       <layout class="QHBoxLayout" name="buttonLayout">
+         <item>
+           <spacer name="horizontalSpacer_1">
+             <property name="orientation">
+               <enum>Qt::Horizontal</enum>
+             </property>
+             <property name="sizeHint" stdset="0">
+               <size>
+                 <width>0</width>
+                 <height>0</height>
+               </size>
+             </property>
+           </spacer>
+         </item>
+         <item>
+           <widget class="QPushButton" name="pushButton">
+             <property name="text">
+               <string>Go To</string>
+             </property>
+           </widget>
+         </item>
+         <item>
+           <widget class="QPushButton" name="pushButton_2">
+             <property name="text">
+               <string>Cancel</string>
+             </property>
+           </widget>
+         </item>
+       </layout>
+     </item>
+   </layout>
  </widget>
  <tabstops>
   <tabstop>m_posEdit</tabstop>