Change Summary:
make sure we disconnect our purple signals.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+1222 -4) |
Review Request #1322 — Created March 10, 2022 and submitted
Behavorial Changes: * Confirmation dialog for remove has been removed. * Remove button is disabled if the currently selected status is the active status. Previously this scenario was silently ignored after confirmation. Errata: It is possible to open multiple modify windows for a status. Previously the editor was reaching into the manager and controlling this pointer. I was going to fix this, but the editor isn't a widget yet and I don't like doing duplicate work.
Basically tried everything I could think of with editing and selecting and so on.
Summary | ID |
---|---|
b108bd2f846428e47c4896ce95f80b36f7284e91 |
make sure we disconnect our purple signals.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+1222 -4) |
pidgin/pidginstatusmanager.h (Diff revision 2) |
---|
Should this be
_get_default
or similar if it never returns a second new one?
pidgin/pidginstatusmanager.c (Diff revision 2) |
---|
Kind of like the below function, this could be changed to:
sensitive = status != purple_savedstatus_get_current();
and one
set_sensitive
call after theif
.
address review issues
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+1206 -4) |
forgot to update the docs.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 4 (+1206 -4) |