seqeralabs / wave

On-demand containers provisioning service
https://seqera.io/wave/
GNU Affero General Public License v3.0
31 stars 4 forks source link

Prevent empty nodes to render expand icon #694

Open pditommaso opened 1 week ago

pditommaso commented 1 week ago

There's a minor inconsistency in the collapsible tree view rendered in the inspect page.

In some cases empty nodes are rendered with a plus icon even if there's no content, instead of being disabled (see below).

Likely this happens because there's empty object associated with it

Screenshot 2024-10-16 at 10 19 49
pditommaso commented 1 week ago

Link to validate it https://wave.seqera.io/view/inspect?image=oras://community.wave.seqera.io/library/bwa:0.7.18--07d074a4d1589c42

pditommaso commented 5 days ago

This is still happening, check for example "annotations" here

munishchouhan commented 3 days ago

This is still happening, check for example "annotations" here

ok, i will look into it

munishchouhan commented 3 days ago

PR to fix this issue: https://github.com/seqeralabs/wave/pull/706