pidgin/pidgin

Remove extraneous scrolled window in account editor

19 months ago, Elliott Sales de Andrade
657b634379ab
Parents d78c0951ea2c
Children 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/
--- 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>