Skip to content

Commit 6c4ec4d

Browse files
Apply suggestions from code review
Co-authored-by: Tim Meier <[email protected]>
1 parent 1615ab9 commit 6c4ec4d

File tree

2 files changed

+12
-7
lines changed

2 files changed

+12
-7
lines changed

src/frontend/packages/core/src/shared/components/page-header/page-header.component.html

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -43,15 +43,15 @@
4343
<mat-menu #token="matMenu" [overlapTrigger]="false" class="page-header__menu">
4444
<div class="page-header__token">
4545
<h3 class="page-header__token-title">UAA Tokens</h3>
46-
<div (click)="copy(authToken$)" matTooltip="Click to copy">
47-
<h4>Auth Token</h4>
46+
<button mat-stroked-button (click)="copy(authToken$)" matTooltip="Click to copy auth token">
47+
<span>Copy Auth Token</span>
4848
<div class="page-header__token-text" hidden>{{ authToken$ | async }}</div>
49-
</div>
50-
<div (click)="copy(refreshToken$)" matTooltip="Click to copy">
51-
<h4>Refresh Token</h4>
49+
</button>
50+
<button mat-stroked-button (click)="copy(refreshToken$)" matTooltip="Click to copy refresh token">
51+
<span>Copy Refresh Token</span>
5252
<div class="page-header__token-text" hidden>{{ refreshToken$ | async }}</div>
53-
</div>
54-
<h4>Token Expiry</h4>
53+
</button>
54+
<h3>Token Expiry</h3>
5555
<div class="page-header__token-text">{{ tokenExpiry$ | async }}</div>
5656
</div>
5757
</mat-menu>

src/frontend/packages/core/src/shared/components/page-header/page-header.component.scss

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,11 @@ $bottom-index: $top-index - 1;
134134
&__token {
135135
min-width: 200px;
136136
padding: 2px 10px 10px;
137+
138+
button {
139+
width: 100%;
140+
margin: 2px 0 2px 0;
141+
}
137142
}
138143
&__token-title {
139144
margin: 0 0 10px;

0 commit comments

Comments
 (0)