dombizita commented on code in PR #4043:
URL: https://github.com/apache/ozone/pull/4043#discussion_r1039525580
##########
hadoop-ozone/recon/src/main/resources/webapps/recon/ozone-recon-web/src/views/overview/overview.tsx:
##########
@@ -172,7 +176,12 @@ export class Overview extends
React.Component<Record<string, object>, IOverviewS
<span className='padded-text'>{containers -
missingContainersCount}/{containers}</span>
</span>
) :
- containers.toString();
+ <div>
+ <span>{containers.toString()} </span>
+ <Tooltip placement='bottom' title={openContainers > 0 ? `Open
Containers` : ``}>
Review Comment:
If we have 0 open containers we would still display it (eg. `3230 (0)`), so
I think we can leave the `Open Containers` tooltip there regardless the number
of open containers.
```suggestion
<Tooltip placement='bottom' title='Number of open containers'>
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
