This change has been marked as completed.

Describe the completed change (optional):

Summary

Convert PidginAccountsEnabledMenu to the list model

Review Request #3954 — Created April 1, 2025 and submitted 1 week, 1 day ago

Information

Owner

QuLogic*

Repository

pidgin/pidgin

Branch

default

Bugs

PIDGIN-18066

Depends On

~~3951~~

Blocks

3955

Reviewers

Groups

pidgin

People

Description*

Again, not exactly what PIDGIN-18066 requested, but should be a reasonable version of it.

Testing Done

Opened Pidgin with no accounts enabled, and the menu was empty. Enabled an account and it moved there, disable the account and it was removed.

Also, started Pidgin with an account enabled, and it appeared in the lower menu as expected.

Commits

Summary	ID
Convert PidginAccountsEnabledMenu to the list model	bc74b90650447ecb539bf1d91b8f0322b8be3f12

Description	From	Last Updated
There are no open issues

pidgin/pidginaccountsenabledmenu.c (Diff revision 1)

Note, I changed this from 0, 0 to 1, 1 since we had some issues with GListModel::items-changed. I don't know if that affects GMenuModel, but better to be safe.

Ship it!

```
Great work, thanks!!
```

Status:: Completed