Convert GtkTreeView in status editor to GtkColumnView

Review Request #2199 — Created Jan. 17, 2023 and submitted

Information

pidgin/pidgin
default

Reviewers

There is a (big or small, up to you) hack in this implementation: PidginSavedStatus is a boxed type and so cannot be placed in a GListModel. Thus the model uses a dummy GObject instance for each row that works similar to each GtkTreeModel row. This is annoying, but means the rest of the .ui remains similar to how it would be with a non-boxed type.

Compared to before, this is only missing search.

Like the account manager, we might want to redesign this entirely, but this is just a straight move away from GtkTreeView due to its impending deprecation. Also, now I have some idea how GtkColumnView works at least.

Double-clicked a few saved statuses to confirm the editor dialogs opened, then clicked Modify to confirm that it opened the existing editor dialog.
Added a new saved status and confirmed that they were in the view.
Deleted a saved status and confirmed it was no longer in the view.
Closed the manager window with editor dialogs opened and confirmed all the editor dialogs closed as they used to.
Confirmed that Remove button was disabled when selecting an in-use status.
Clicked the column headers and confirmed that the statuses were sorted by them.

Summary ID
Convert GtkTreeView in status editor to GtkColumnView
6e4caa0eaedf534b4adf0f84dbc665e55f2a0b44
Description From Last Updated

We determined we don't need to set object if this the parent widget we're defining correct?

grimgrim
grim
  1. 
      
  2. pidgin/resources/Status/manager.ui (Diff revision 1)
     
     

    We determined we don't need to set object if this the parent widget we're defining correct?

    1. Yes, these were just renamed from before so I didn't remove them.

  3. 
      
QuLogic
grim
  1. Ship It!
  2. Great work, thanks!!

  3. 
      
grim
Review request changed

Status: Closed (submitted)

Loading...