Replace PidginAccountsDisabledMenu with PidginInactiveAccountsMenu which is a GMenuModel subclass.
Review Request #1366 — Created March 22, 2022 and submitted
This gives us a bit more control as we are always in control of the attributes, so we don't need to modify the menu in place. This also made it trivial to add the protocol icons which are very helpful when you have the same user name on multiple protocols.
Enabled and disabled two accounts a lot while under valgrind.
Summary | ID |
---|---|
f54207e24fef2f6921106ca24ab72f419b02d28b |
Description | From | Last Updated |
---|---|---|
Should it be disabled? Or maybe inactive? |
|
|
Not a Menu any more; probably easier to keep the type out of the docs here. |
|
|
This leaked the existing menu->accounts list. |
|
|
The key doesn't seem to need to be strdup'd, unless that's required of the API? |
|
|
Should use the attribute name constants. |
|
|
I'm no sure if this should be g_variant_ref_sink instead. |
|
|
Add space. |
|
|
Update type in comment. |
|
- Change Summary:
-
Address issues and rename to PidginInactiveAccountsMenu
- Summary:
-
Reimplement the PidginAccountsDisabledMenu to be a GMenuModel subclass.Replace PidginAccountsDisabledMenu with PidginInactiveAccountsMenu which is a GMenuModel subclass.
- Commits:
-
Summary ID 2e74220169504805271bc92eb25f734a432e03d5 d0c53135a30fe6087a0f99367327a64a50f56349