improve html output

[bendichter]

@edeno

current:

Summary expandable is rendered for fields that are empty. See the expanded groups at the bottom that are empty:

option 1: render them as empty

option 2: don't render them

I have a slight preference for option 2, but I think option 1 would be fine and would be an improvement.

[bendichter]

Vote for Option 1

[bendichter]

Vote for Option 2

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -0.02 ⚠️

Comparison is base (6043e77) 88.17% compared to head (972b15a) 88.15%.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #886      +/-   ##
==========================================
- Coverage   88.17%   88.15%   -0.02%     
==========================================
  Files          44       44              
  Lines        9067     9069       +2     
  Branches     2595     2596       +1     
==========================================
  Hits         7995     7995              
- Misses        757      758       +1     
- Partials      315      316       +1     

Impacted Files	Coverage Δ
src/hdmf/container.py	90.39% <0.00%> (-0.24%)	⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[edeno]

I don't personally mind the way it is because those exist, but are just empty. I guess there's the expectation that if something has the triangle that the expansion leads to an output. But hard for me to tell if that would freak people out or they would just understand that it could be expanded were there something there.

So Option 1 (or how it is) is fine, but I would not advocate for Option 2 (unless there's the situation in which there are a lot of these) because it lets the user know that something could be there but isn't.

[bendichter]

Option 3: #887 (could be combined with 1 or 2)

[edeno]

Option 3 is the best I think.

[rly]

Closed in favor of #887