Remove extraneous scrolled window in account editor

Fri, 23 Sep 2022 01:23:57 -0500

author
Elliott Sales de Andrade <quantum.analyst@gmail.com>
date
Fri, 23 Sep 2022 01:23:57 -0500
changeset 41737
657b634379ab
parent 41736
d78c0951ea2c
child 41738
5ff19c662e31

Remove extraneous scrolled window in account editor

The AdwPreferencesPage already contains a scrolled window.

Testing Done:
Opened the account editor, which was still scrollable.

Reviewed at https://reviews.imfreedom.org/r/1829/

pidgin/resources/Accounts/editor.ui file | annotate | diff | comparison | revisions
--- a/pidgin/resources/Accounts/editor.ui	Fri Sep 23 00:13:38 2022 -0500
+++ b/pidgin/resources/Accounts/editor.ui	Fri Sep 23 01:23:57 2022 -0500
@@ -36,204 +36,198 @@
       <object class="GtkBox">
         <property name="vexpand">1</property>
         <child>
-          <object class="GtkScrolledWindow">
+          <object class="AdwPreferencesPage">
             <property name="vexpand">1</property>
             <property name="hexpand">1</property>
-            <property name="child">
-              <object class="AdwPreferencesPage">
-                <property name="vexpand">1</property>
-                <property name="hexpand">1</property>
+            <child>
+              <object class="AdwPreferencesGroup" id="login_options">
+                <property name="title" translatable="1">Login Options</property>
                 <child>
-                  <object class="AdwPreferencesGroup" id="login_options">
-                    <property name="title" translatable="1">Login Options</property>
+                  <object class="PidginProtocolChooser" id="protocol">
+                    <property name="title" translatable="1">Pro_tocol</property>
+                    <property name="use-underline">1</property>
+                    <signal name="notify::protocol" handler="pidgin_account_editor_protocol_changed_cb" object="PidginAccountEditor" swapped="no"/>
+                  </object>
+                </child>
+                <child>
+                  <object class="AdwActionRow">
+                    <property name="activatable-widget">username</property>
+                    <property name="title" translatable="1">_Username</property>
+                    <property name="use-underline">1</property>
                     <child>
-                      <object class="PidginProtocolChooser" id="protocol">
-                        <property name="title" translatable="1">Pro_tocol</property>
-                        <property name="use-underline">1</property>
-                        <signal name="notify::protocol" handler="pidgin_account_editor_protocol_changed_cb" object="PidginAccountEditor" swapped="no"/>
-                      </object>
-                    </child>
-                    <child>
-                      <object class="AdwActionRow">
-                        <property name="activatable-widget">username</property>
-                        <property name="title" translatable="1">_Username</property>
-                        <property name="use-underline">1</property>
-                        <child>
-                          <object class="GtkEntry" id="username">
-                            <property name="hexpand">1</property>
-                            <property name="valign">center</property>
-                            <property name="truncate-multiline">1</property>
-                            <signal name="changed" handler="pidgin_account_editor_username_changed_cb" object="PidginAccountEditor" swapped="no"/>
-                          </object>
-                        </child>
+                      <object class="GtkEntry" id="username">
+                        <property name="hexpand">1</property>
+                        <property name="valign">center</property>
+                        <property name="truncate-multiline">1</property>
+                        <signal name="changed" handler="pidgin_account_editor_username_changed_cb" object="PidginAccountEditor" swapped="no"/>
                       </object>
                     </child>
                   </object>
                 </child>
+              </object>
+            </child>
+            <child>
+              <object class="AdwPreferencesGroup">
+                <property name="title" translatable="1">User Options</property>
                 <child>
-                  <object class="AdwPreferencesGroup">
-                    <property name="title" translatable="1">User Options</property>
-                    <child>
-                      <object class="AdwActionRow">
-                        <property name="activatable-widget">alias</property>
-                        <property name="title" translatable="1">_Local alias</property>
-                        <property name="use-underline">1</property>
-                        <child>
-                          <object class="GtkEntry" id="alias">
-                            <property name="truncate-multiline">1</property>
-                            <property name="hexpand">1</property>
-                            <property name="valign">center</property>
-                          </object>
-                        </child>
-                      </object>
-                    </child>
+                  <object class="AdwActionRow">
+                    <property name="activatable-widget">alias</property>
+                    <property name="title" translatable="1">_Local alias</property>
+                    <property name="use-underline">1</property>
                     <child>
-                      <object class="AdwActionRow" id="avatar_row">
-                        <property name="activatable-widget">avatar</property>
-                        <property name="title" translatable="1">Use custom _avatar</property>
-                        <property name="use-underline">1</property>
-                        <property name="visible">0</property>
-                        <child type="prefix">
-                          <object class="GtkSwitch" id="use_custom_avatar">
-                            <property name="valign">center</property>
-                          </object>
-                        </child>
-                        <child>
-                          <object class="GtkButton">
-                            <property name="css-classes">flat</property>
-                            <property name="valign">center</property>
-                            <binding name="sensitive">
-                              <lookup name="active">use_custom_avatar</lookup>
-                            </binding>
-                            <child>
-                              <object class="GtkImage" id="avatar">
-                                <property name="icon-name">select-avatar</property>
-                                <property name="icon-size">large</property>
-                              </object>
-                            </child>
-                            <signal name="clicked" handler="pidgin_account_editor_avatar_set_clicked_cb" object="PidginAccountEditor" swapped="no"/>
-                          </object>
-                        </child>
-                        <child>
-                          <object class="GtkButton" id="remove">
-                            <property name="label" translatable="1">_Remove</property>
-                            <property name="use-underline">1</property>
-                            <property name="valign">center</property>
-                            <binding name="sensitive">
-                              <lookup name="active">use_custom_avatar</lookup>
-                            </binding>
-                            <signal name="clicked" handler="pidgin_account_editor_avatar_remove_clicked_cb" object="PidginAccountEditor" swapped="no"/>
-                          </object>
-                        </child>
-                      </object>
-                    </child>
-                  </object>
-                </child>
-                <child>
-                  <object class="AdwPreferencesGroup" id="advanced_group">
-                    <property name="title" translatable="1">Advanced Options</property>
-                    <property name="description" translatable="1">Additional options for this account.</property>
-                    <property name="visible">0</property>
-                    <child type="header-suffix">
-                      <object class="GtkSwitch" id="advanced_toggle">
-                        <property name="active">0</property>
+                      <object class="GtkEntry" id="alias">
+                        <property name="truncate-multiline">1</property>
+                        <property name="hexpand">1</property>
                         <property name="valign">center</property>
                       </object>
                     </child>
                   </object>
                 </child>
                 <child>
-                  <object class="AdwPreferencesGroup">
-                    <property name="title" translatable="1">Proxy</property>
+                  <object class="AdwActionRow" id="avatar_row">
+                    <property name="activatable-widget">avatar</property>
+                    <property name="title" translatable="1">Use custom _avatar</property>
+                    <property name="use-underline">1</property>
+                    <property name="visible">0</property>
+                    <child type="prefix">
+                      <object class="GtkSwitch" id="use_custom_avatar">
+                        <property name="valign">center</property>
+                      </object>
+                    </child>
                     <child>
-                      <object class="AdwComboRow" id="proxy_type">
-                        <property name="title" translatable="1">Proxy t_ype</property>
-                        <property name="use-underline">1</property>
-                        <property name="use-subtitle">1</property>
-                        <property name="model">
-                          <object class="GtkStringList">
-                            <items>
-                              <item>global</item>
-                              <item>none</item>
-                              <item>socks4</item>
-                              <item>socks5</item>
-                              <item>tor</item>
-                              <item>http</item>
-                              <item>envvar</item>
-                            </items>
+                      <object class="GtkButton">
+                        <property name="css-classes">flat</property>
+                        <property name="valign">center</property>
+                        <binding name="sensitive">
+                          <lookup name="active">use_custom_avatar</lookup>
+                        </binding>
+                        <child>
+                          <object class="GtkImage" id="avatar">
+                            <property name="icon-name">select-avatar</property>
+                            <property name="icon-size">large</property>
                           </object>
-                        </property>
-                        <property name="expression">
-                          <closure type="gchararray" function="pidgin_account_editor_proxy_type_expression_cb"/>
-                        </property>
-                        <signal name="notify::selected" handler="pidgin_account_editor_proxy_type_changed_cb" object="PidginAccountEditor" swapped="no"/>
+                        </child>
+                        <signal name="clicked" handler="pidgin_account_editor_avatar_set_clicked_cb" object="PidginAccountEditor" swapped="no"/>
                       </object>
                     </child>
                     <child>
-                      <object class="AdwPreferencesRow" id="proxy_options">
-                        <property name="visible">0</property>
+                      <object class="GtkButton" id="remove">
+                        <property name="label" translatable="1">_Remove</property>
+                        <property name="use-underline">1</property>
+                        <property name="valign">center</property>
+                        <binding name="sensitive">
+                          <lookup name="active">use_custom_avatar</lookup>
+                        </binding>
+                        <signal name="clicked" handler="pidgin_account_editor_avatar_remove_clicked_cb" object="PidginAccountEditor" swapped="no"/>
+                      </object>
+                    </child>
+                  </object>
+                </child>
+              </object>
+            </child>
+            <child>
+              <object class="AdwPreferencesGroup" id="advanced_group">
+                <property name="title" translatable="1">Advanced Options</property>
+                <property name="description" translatable="1">Additional options for this account.</property>
+                <property name="visible">0</property>
+                <child type="header-suffix">
+                  <object class="GtkSwitch" id="advanced_toggle">
+                    <property name="active">0</property>
+                    <property name="valign">center</property>
+                  </object>
+                </child>
+              </object>
+            </child>
+            <child>
+              <object class="AdwPreferencesGroup">
+                <property name="title" translatable="1">Proxy</property>
+                <child>
+                  <object class="AdwComboRow" id="proxy_type">
+                    <property name="title" translatable="1">Proxy t_ype</property>
+                    <property name="use-underline">1</property>
+                    <property name="use-subtitle">1</property>
+                    <property name="model">
+                      <object class="GtkStringList">
+                        <items>
+                          <item>global</item>
+                          <item>none</item>
+                          <item>socks4</item>
+                          <item>socks5</item>
+                          <item>tor</item>
+                          <item>http</item>
+                          <item>envvar</item>
+                        </items>
+                      </object>
+                    </property>
+                    <property name="expression">
+                      <closure type="gchararray" function="pidgin_account_editor_proxy_type_expression_cb"/>
+                    </property>
+                    <signal name="notify::selected" handler="pidgin_account_editor_proxy_type_changed_cb" object="PidginAccountEditor" swapped="no"/>
+                  </object>
+                </child>
+                <child>
+                  <object class="AdwPreferencesRow" id="proxy_options">
+                    <property name="visible">0</property>
+                    <child>
+                      <object class="GtkListBox">
                         <child>
-                          <object class="GtkListBox">
+                          <object class="AdwActionRow">
+                            <property name="activatable-widget">proxy_host</property>
+                            <property name="title" translatable="1">_Host</property>
+                            <property name="use-underline">1</property>
                             <child>
-                              <object class="AdwActionRow">
-                                <property name="activatable-widget">proxy_host</property>
-                                <property name="title" translatable="1">_Host</property>
-                                <property name="use-underline">1</property>
-                                <child>
-                                  <object class="GtkEntry" id="proxy_host">
-                                    <property name="hexpand">1</property>
-                                    <property name="valign">center</property>
-                                  </object>
-                                </child>
+                              <object class="GtkEntry" id="proxy_host">
+                                <property name="hexpand">1</property>
+                                <property name="valign">center</property>
                               </object>
                             </child>
+                          </object>
+                        </child>
+                        <child>
+                          <object class="AdwActionRow">
+                            <property name="activatable-widget">proxy_port</property>
+                            <property name="title" translatable="1">P_ort</property>
+                            <property name="use-underline">1</property>
                             <child>
-                              <object class="AdwActionRow">
-                                <property name="activatable-widget">proxy_port</property>
-                                <property name="title" translatable="1">P_ort</property>
-                                <property name="use-underline">1</property>
-                                <child>
-                                  <object class="GtkSpinButton" id="proxy_port">
-                                    <property name="text" translatable="1">0</property>
-                                    <property name="adjustment">
-                                      <object class="GtkAdjustment">
-                                        <property name="upper">65535</property>
-                                        <property name="step-increment">1</property>
-                                        <property name="page-increment">10</property>
-                                      </object>
-                                    </property>
-                                    <property name="numeric">1</property>
-                                    <property name="hexpand">1</property>
-                                    <property name="valign">center</property>
+                              <object class="GtkSpinButton" id="proxy_port">
+                                <property name="text" translatable="1">0</property>
+                                <property name="adjustment">
+                                  <object class="GtkAdjustment">
+                                    <property name="upper">65535</property>
+                                    <property name="step-increment">1</property>
+                                    <property name="page-increment">10</property>
                                   </object>
-                                </child>
+                                </property>
+                                <property name="numeric">1</property>
+                                <property name="hexpand">1</property>
+                                <property name="valign">center</property>
                               </object>
                             </child>
+                          </object>
+                        </child>
+                        <child>
+                          <object class="AdwActionRow">
+                            <property name="activatable-widget">proxy_username</property>
+                            <property name="title" translatable="1">User_name</property>
+                            <property name="use-underline">1</property>
                             <child>
-                              <object class="AdwActionRow">
-                                <property name="activatable-widget">proxy_username</property>
-                                <property name="title" translatable="1">User_name</property>
-                                <property name="use-underline">1</property>
-                                <child>
-                                  <object class="GtkEntry" id="proxy_username">
-                                    <property name="hexpand">1</property>
-                                    <property name="valign">center</property>
-                                  </object>
-                                </child>
+                              <object class="GtkEntry" id="proxy_username">
+                                <property name="hexpand">1</property>
+                                <property name="valign">center</property>
                               </object>
                             </child>
+                          </object>
+                        </child>
+                        <child>
+                          <object class="AdwActionRow">
+                            <property name="activatable-widget">proxy_password</property>
+                            <property name="title" translatable="1">Pa_ssword</property>
+                            <property name="use-underline">1</property>
                             <child>
-                              <object class="AdwActionRow">
-                                <property name="activatable-widget">proxy_password</property>
-                                <property name="title" translatable="1">Pa_ssword</property>
-                                <property name="use-underline">1</property>
-                                <child>
-                                  <object class="GtkPasswordEntry" id="proxy_password">
-                                    <property name="hexpand">1</property>
-                                    <property name="show-peek-icon">1</property>
-                                    <property name="valign">center</property>
-                                  </object>
-                                </child>
+                              <object class="GtkPasswordEntry" id="proxy_password">
+                                <property name="hexpand">1</property>
+                                <property name="show-peek-icon">1</property>
+                                <property name="valign">center</property>
                               </object>
                             </child>
                           </object>
@@ -243,7 +237,7 @@
                   </object>
                 </child>
               </object>
-            </property>
+            </child>
           </object>
         </child>
       </object>

mercurial