Skip to content

Commit d644fdf

Browse files
authored
Make the 'View Engagement' link right clickable (#589)
1 parent c9526fb commit d644fdf

File tree

4 files changed

+24
-25
lines changed

4 files changed

+24
-25
lines changed

src/components/engagement_list/__tests__/__snapshots__/engagement_list.spec.tsx.snap

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -221,17 +221,17 @@ Object {
221221
<div
222222
class="pf-l-grid__item pf-m-12-col-on-sm pf-m-4-col-on-md pf-m-2-col-on-lg"
223223
>
224-
<button
224+
<a
225225
aria-disabled="false"
226226
class="pf-c-button pf-m-secondary"
227227
data-cy="view_engagement_button"
228228
data-ouia-component-id="OUIA-Generated-Button-secondary-1"
229229
data-ouia-component-type="PF4/Button"
230230
data-ouia-safe="true"
231-
type="button"
231+
href="/app/engagements/uuid"
232232
>
233233
View Engagement
234-
</button>
234+
</a>
235235
</div>
236236
</div>
237237
</div>
@@ -455,17 +455,17 @@ Object {
455455
<div
456456
class="pf-l-grid__item pf-m-12-col-on-sm pf-m-4-col-on-md pf-m-2-col-on-lg"
457457
>
458-
<button
458+
<a
459459
aria-disabled="false"
460460
class="pf-c-button pf-m-secondary"
461461
data-cy="view_engagement_button"
462462
data-ouia-component-id="OUIA-Generated-Button-secondary-1"
463463
data-ouia-component-type="PF4/Button"
464464
data-ouia-safe="true"
465-
type="button"
465+
href="/app/engagements/uuid"
466466
>
467467
View Engagement
468-
</button>
468+
</a>
469469
</div>
470470
</div>
471471
</div>
@@ -746,17 +746,17 @@ Object {
746746
<div
747747
class="pf-l-grid__item pf-m-12-col-on-sm pf-m-4-col-on-md pf-m-2-col-on-lg"
748748
>
749-
<button
749+
<a
750750
aria-disabled="false"
751751
class="pf-c-button pf-m-secondary"
752752
data-cy="view_engagement_button"
753753
data-ouia-component-id="OUIA-Generated-Button-secondary-1"
754754
data-ouia-component-type="PF4/Button"
755755
data-ouia-safe="true"
756-
type="button"
756+
href="/app/engagements/uuid"
757757
>
758758
View Engagement
759-
</button>
759+
</a>
760760
</div>
761761
</div>
762762
</div>

src/components/engagement_list_item/__tests__/__snapshots__/engagement_list_item.spec.tsx.snap

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -221,17 +221,17 @@ Object {
221221
<div
222222
class="pf-l-grid__item pf-m-12-col-on-sm pf-m-4-col-on-md pf-m-2-col-on-lg"
223223
>
224-
<button
224+
<a
225225
aria-disabled="false"
226226
class="pf-c-button pf-m-secondary"
227227
data-cy="view_engagement_button"
228228
data-ouia-component-id="OUIA-Generated-Button-secondary-1"
229229
data-ouia-component-type="PF4/Button"
230230
data-ouia-safe="true"
231-
type="button"
231+
href="/app/engagements/uuid"
232232
>
233233
View Engagement
234-
</button>
234+
</a>
235235
</div>
236236
</div>
237237
</div>
@@ -455,17 +455,17 @@ Object {
455455
<div
456456
class="pf-l-grid__item pf-m-12-col-on-sm pf-m-4-col-on-md pf-m-2-col-on-lg"
457457
>
458-
<button
458+
<a
459459
aria-disabled="false"
460460
class="pf-c-button pf-m-secondary"
461461
data-cy="view_engagement_button"
462462
data-ouia-component-id="OUIA-Generated-Button-secondary-1"
463463
data-ouia-component-type="PF4/Button"
464464
data-ouia-safe="true"
465-
type="button"
465+
href="/app/engagements/uuid"
466466
>
467467
View Engagement
468-
</button>
468+
</a>
469469
</div>
470470
</div>
471471
</div>

src/components/engagement_list_item/__tests__/__snapshots__/engagement_list_item_card.spec.tsx.snap

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -221,17 +221,17 @@ Object {
221221
<div
222222
class="pf-l-grid__item pf-m-12-col-on-sm pf-m-4-col-on-md pf-m-2-col-on-lg"
223223
>
224-
<button
224+
<a
225225
aria-disabled="false"
226226
class="pf-c-button pf-m-secondary"
227227
data-cy="view_engagement_button"
228228
data-ouia-component-id="OUIA-Generated-Button-secondary-2"
229229
data-ouia-component-type="PF4/Button"
230230
data-ouia-safe="true"
231-
type="button"
231+
href="/app/engagements/uuid"
232232
>
233233
View Engagement
234-
</button>
234+
</a>
235235
</div>
236236
</div>
237237
</div>
@@ -455,17 +455,17 @@ Object {
455455
<div
456456
class="pf-l-grid__item pf-m-12-col-on-sm pf-m-4-col-on-md pf-m-2-col-on-lg"
457457
>
458-
<button
458+
<a
459459
aria-disabled="false"
460460
class="pf-c-button pf-m-secondary"
461461
data-cy="view_engagement_button"
462462
data-ouia-component-id="OUIA-Generated-Button-secondary-2"
463463
data-ouia-component-type="PF4/Button"
464464
data-ouia-safe="true"
465-
type="button"
465+
href="/app/engagements/uuid"
466466
>
467467
View Engagement
468-
</button>
468+
</a>
469469
</div>
470470
</div>
471471
</div>

src/components/engagement_list_item/engagement_list_item.tsx

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -68,11 +68,10 @@ export function EngagementListItem(props: DataCardProps) {
6868
</GridItem>
6969
<GridItem sm={12} md={4} lg={2}>
7070
<Button
71-
onClick={() => {
72-
history.push(url);
73-
}}
71+
href = { url }
7472
variant="secondary"
75-
data-cy={'view_engagement_button'}
73+
data-cy="view_engagement_button"
74+
component="a"
7675
>
7776
View Engagement
7877
</Button>

0 commit comments

Comments
 (0)