Skip to content

Commit

Permalink
fix(suiteheader): render data attr as (hidden) text
Browse files Browse the repository at this point in the history
  • Loading branch information
bryancboyd committed Dec 4, 2020
1 parent b3410c8 commit 189c745
Show file tree
Hide file tree
Showing 2 changed files with 33 additions and 6 deletions.
11 changes: 8 additions & 3 deletions src/components/SuiteHeader/SuiteHeader.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -222,9 +222,14 @@ const SuiteHeader = ({
<>
<span
className={`${settings.iotPrefix}--suite-header-data`}
data-workspace-id={routes.workspaceId}
data-domain={routes.domain}
/>
data-type="workspaceId">
{routes.workspaceId}
</span>
<span
className={`${settings.iotPrefix}--suite-header-data`}
data-type="domain">
{routes.domain}
</span>
<Header
className={[`${settings.iotPrefix}--suite-header`, className]
.filter((i) => i)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,11 @@ exports[`Storybook Snapshot tests and console checks Storyshots Watson IoT/Suite
</div>
<span
className="iot--suite-header-data"
data-type="workspaceId"
/>
<span
className="iot--suite-header-data"
data-type="domain"
/>
<header
aria-label="main header"
Expand Down Expand Up @@ -872,6 +877,11 @@ exports[`Storybook Snapshot tests and console checks Storyshots Watson IoT/Suite
</div>
<span
className="iot--suite-header-data"
data-type="workspaceId"
/>
<span
className="iot--suite-header-data"
data-type="domain"
/>
<header
aria-label="main header"
Expand Down Expand Up @@ -1832,6 +1842,11 @@ exports[`Storybook Snapshot tests and console checks Storyshots Watson IoT/Suite
</div>
<span
className="iot--suite-header-data"
data-type="workspaceId"
/>
<span
className="iot--suite-header-data"
data-type="domain"
/>
<header
aria-label="main header"
Expand Down Expand Up @@ -2393,9 +2408,16 @@ exports[`Storybook Snapshot tests and console checks Storyshots Watson IoT/Suite
</div>
<span
className="iot--suite-header-data"
data-domain="ibm.com"
data-workspace-id="workspace1"
/>
data-type="workspaceId"
>
workspace1
</span>
<span
className="iot--suite-header-data"
data-type="domain"
>
ibm.com
</span>
<header
aria-label="main header"
className="bx--header iot--suite-header"
Expand Down

0 comments on commit 189c745

Please sign in to comment.