Skip to content

Commit

Permalink
fix: [M3-7717] - Fix leading whitespace in list of Linode and Nodebal…
Browse files Browse the repository at this point in the history
…ancer links for Firewall Services (#10527)

* Fix leading whitespace

* Add changeset
  • Loading branch information
mjac0bs authored May 31, 2024
1 parent 7a5f3b7 commit cbb668a
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 14 deletions.
5 changes: 5 additions & 0 deletions packages/manager/.changeset/pr-10527-fixed-1717019227934.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@linode/manager": Fixed
---

Leading whitespace in list of Firewall Services ([#10527](https://github.com/linode/manager/pull/10527))
Original file line number Diff line number Diff line change
Expand Up @@ -105,21 +105,19 @@ export const getDeviceLinks = (data: FirewallDevice[]): JSX.Element => {
return (
<>
{firstThree.map((thisDevice, idx) => (
<Link
className="link secondaryLink"
data-testid="firewall-row-link"
key={thisDevice.id}
to={`/${thisDevice.entity.type}s/${thisDevice.entity.id}`}
>
{idx > 0 && `, `}
{thisDevice.entity.label}
</Link>
<>
{idx > 0 && ', '}
<Link
className="link secondaryLink"
data-testid="firewall-row-link"
key={thisDevice.id}
to={`/${thisDevice.entity.type}s/${thisDevice.entity.id}`}
>
{thisDevice.entity.label}
</Link>
</>
))}
{data.length > 3 && (
<span>
{`, `}plus {data.length - 3} more.
</span>
)}
{data.length > 3 && <span>, plus {data.length - 3} more.</span>}
</>
);
};

0 comments on commit cbb668a

Please sign in to comment.