-
-
pidgin/pidginaccountmanager.c (Diff revision 1) Should we have
show_overview
callset_account(editor, NULL
and not do it in both places?
Fix a number of bugs in the account editor
Review Request #2540 — Created Aug. 4, 2023 and submitted
This fixes the protocol not being the first one in the list as well as the username still being populate from a previous account when a new account is being created. All login options are now set to insensitive when an account is online and are properly made sensitive when the account is offline. However, these properties aren't bound yet, because there's no property to bind to. See PIDGIN-17815.
Opened the account editor for an existing online account and verified that the protocol, username, and all user splits were insensitive. Hit the back button, hit the add button and verified that the protocol was at Bonjour (the first in the list) and that the username was empty and that those 2 and all of the user splits were sensitive.
Summary | ID |
---|---|
7460ad45b61a271353c8a5816084742fa0d77445 |
Change Summary:
set the editor's account to null when we show the overview page in the manager.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+88 -48) |
-
Ship It!
-
pidgin/pidginaccountmanager.c (Diff revision 2) Minor thing; I'd change the stack first so that the editor doesn't appear to clear before hiding (though I'm pretty sure at this point, nothing runs the event loop, so it shouldn't do that, but just in case that changes in the future.)
Change Summary:
switch the stack before nulling out the account.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+88 -48) |