If application has no URL, render text only (#25138)
Fixes #25124 Signed-off-by: Stan Silvert <ssilvert@redhat.com>
This commit is contained in:
parent
386e4ea8d4
commit
8a9b83d016
1 changed files with 16 additions and 9 deletions
|
@ -128,15 +128,22 @@ const Applications = () => {
|
||||||
className="pf-u-align-items-center"
|
className="pf-u-align-items-center"
|
||||||
dataListCells={[
|
dataListCells={[
|
||||||
<DataListCell width={2} key={`client${application.clientId}`}>
|
<DataListCell width={2} key={`client${application.clientId}`}>
|
||||||
<Button
|
{application.effectiveUrl && (
|
||||||
className="pf-u-pl-0 title-case"
|
<Button
|
||||||
component="a"
|
className="pf-u-pl-0 title-case"
|
||||||
variant="link"
|
component="a"
|
||||||
onClick={() => window.open(application.effectiveUrl)}
|
variant="link"
|
||||||
>
|
onClick={() => window.open(application.effectiveUrl)}
|
||||||
{application.clientName || application.clientId}{" "}
|
>
|
||||||
<ExternalLinkAltIcon />
|
{application.clientName || application.clientId}{" "}
|
||||||
</Button>
|
<ExternalLinkAltIcon />
|
||||||
|
</Button>
|
||||||
|
)}
|
||||||
|
{!application.effectiveUrl && (
|
||||||
|
<span>
|
||||||
|
{application.clientName || application.clientId}
|
||||||
|
</span>
|
||||||
|
)}
|
||||||
</DataListCell>,
|
</DataListCell>,
|
||||||
<DataListCell
|
<DataListCell
|
||||||
width={2}
|
width={2}
|
||||||
|
|
Loading…
Reference in a new issue