[Stats Refresh] Insights Tags and Categories: add drop-down rows for groups

[ScoutHarris]

Fixes #10832

To test:

---

On a site that has a post with multiple tags/categories:

• Tip: Hogwarts usually has some.
• Go to Stats > Insights > Tags and Categories.
• Verify:
  • The icon is folderMultiple.
  • The disclosure icon is pointed down.

---

Click on a row that has multiple tags/categories.

• Verify:
  • Individual rows appear for each tag/category.
    • Max of 10 "child" rows will be displayed.
    • NOTE: we don't currently have the totals for expanded rows. Ref Insights Tags and Categories: show totals for expanded rows #10945.
  • On the "parent" row, the disclosure icon is pointed up.
  • The row dividers above and below the expanded section are the full width of the row.
  • Selecting a "child" row displays a webview for that tag/category.

---

• Verify expanded rows persist on scrolling and rotation.
• Verify expanded rows are reset when Insights is first displayed.

[ScoutHarris]

@SylvesterWilmott -

---

---

---

---

[ScoutHarris]

@jklausa this is causing constraint warnings when going from Insights > Period > Insights. I'll address those in the next PR, which is branched from this one.

Update: I checked develop. The constraint warnings are there too. So I broke it long before this PR. I'll address it separately.

[SylvesterWilmott]

This looks great @ScoutHarris !

I'm a little concerned by the jumpiness of the expanding/collapsing. In the .gif it appears that the cells above the one that expands/collapses is moving too which makes it a little difficult to follow what is happening. Is there a way to keep the other cells completely static and animate only the group that is expanding?

[ScoutHarris]

Hey @SylvesterWilmott .

jumpiness of the expanding/collapsing

Hrm. That's being caused by the need to redraw the card to resize it to accommodate adding/removing rows. Without the redraw, the rows were just smushed into the original card size. I'm not sure how else I can accomplish that. Do you mind if I add this to my todo list for later so can experiment and hopefully come up with a better solution?

[jklausa]

I have one request for some comments, but other than that — looks good!

[jklausa]

Would you mind adding a comment why this is done this way here, instead of beginUpdates() at the beginning of this method and endUpdates() at the end?

[ScoutHarris]

Sure, I can. But for here - everything after applyTableUpdates is just updating an array that has no effect on the table. So I didn't think waiting to endUpdates was necessary.

[jklausa]

I also had similar issues with jumpiness in Plugins Detail view — I think I fought with it for a long time, and didn't manage to fix it a 100% (though I have an idea how to do it now...), but maybe you could take a peek there for inspiration :P

[ScoutHarris]

maybe you could take a peek there for inspiration

@jklausa thank you!