Skip to content

Commit 5fed7da

Browse files
committed
missing type="button" + aria-pressed="false"
1 parent 01b9a58 commit 5fed7da

File tree

5 files changed

+14
-12
lines changed

5 files changed

+14
-12
lines changed

site/data/components-details.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,7 @@ export const componentsDetails: ComponentCardData[] = [
211211
<div class="input-container">
212212
<input type="password" id="passwordInput[[id_prefix]]" class="text-input-field" placeholder=" " value="xxxxxx">
213213
</div>
214-
<button class="btn btn-minimal btn-icon">
214+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
215215
<svg aria-hidden="true">
216216
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
217217
</svg>

site/src/assets/partials/snippets.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -277,9 +277,11 @@ export default () => {
277277
if (type === 'text') {
278278
iconUse.setAttribute('xlink:href', iconUse.getAttribute('xlink:href').replace('accessibility-vision', 'hide'))
279279
visuallyHiddenText.textContent = 'Hide password'
280+
togglePasswordButton.setAttribute('aria-pressed', 'true')
280281
} else {
281282
iconUse.setAttribute('xlink:href', iconUse.getAttribute('xlink:href').replace('hide', 'accessibility-vision'))
282283
visuallyHiddenText.textContent = 'Show password'
284+
togglePasswordButton.setAttribute('aria-pressed', 'false')
283285
}
284286
})
285287
}

site/src/content/docs/components/password-input.mdx

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ You will need to write some additional JavaScript code to make the show/hide pas
3737
<div class="text-input-container">
3838
<label for="inputPassword">Password</label>
3939
<input type="password" id="inputPassword" class="text-input-field" placeholder=" ">
40-
<button class="btn btn-minimal btn-icon">
40+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
4141
<svg aria-hidden="true">
4242
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
4343
</svg>
@@ -52,7 +52,7 @@ You will need to write some additional JavaScript code to make the show/hide pas
5252
</svg>
5353
<label for="inputPasswordIcon">Password</label>
5454
<input type="password" id="inputPasswordIcon" class="text-input-field" placeholder="Minimum 8 characters">
55-
<button class="btn btn-minimal btn-icon">
55+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
5656
<svg aria-hidden="true">
5757
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
5858
</svg>
@@ -69,7 +69,7 @@ You will need to write some additional JavaScript code to make the show/hide pas
6969
<div class="input-container" data-bs-prefix="DEV-">
7070
<input type="password" id="inputPasswordPrefix" aria-describedby="inputPasswordPrefixHelper" class="text-input-field" placeholder=" ">
7171
</div>
72-
<button class="btn btn-minimal btn-icon">
72+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
7373
<svg aria-hidden="true">
7474
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
7575
</svg>
@@ -85,7 +85,7 @@ You will need to write some additional JavaScript code to make the show/hide pas
8585
</svg>
8686
<label for="inputPasswordOutlined">Password</label>
8787
<input type="password" id="inputPasswordOutlined" aria-describedby="inputPasswordOutlinedHelper" class="text-input-field" placeholder=" ">
88-
<button class="btn btn-minimal btn-icon">
88+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
8989
<svg aria-hidden="true">
9090
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
9191
</svg>
@@ -101,7 +101,7 @@ You will need to write some additional JavaScript code to make the show/hide pas
101101
</svg>
102102
<label for="inputPasswordOutlinedValue">Password</label>
103103
<input type="password" id="inputPasswordOutlinedValue" aria-describedby="inputPasswordOutlinedValueHelper" class="text-input-field" placeholder=" " value="My$ecureP@ssw0rd">
104-
<button class="btn btn-minimal btn-icon">
104+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
105105
<svg aria-hidden="true">
106106
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
107107
</svg>
@@ -116,7 +116,7 @@ You will need to write some additional JavaScript code to make the show/hide pas
116116
<div class="input-container" data-bs-prefix="DEV-">
117117
<input type="password" id="inputPasswordInvalid" class="text-input-field is-invalid" aria-describedby="inputPasswordInvalidError" placeholder=" " value="12345">
118118
</div>
119-
<button class="btn btn-minimal btn-icon">
119+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
120120
<svg aria-hidden="true">
121121
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
122122
</svg>
@@ -135,7 +135,7 @@ You will need to write some additional JavaScript code to make the show/hide pas
135135
<div class="input-container" data-bs-prefix="DEV-">
136136
<input type="password" id="inputPasswordOutlinedInvalid" class="text-input-field is-invalid" aria-describedby="inputPasswordOutlinedInvalidError" placeholder=" " value="12345">
137137
</div>
138-
<button class="btn btn-minimal btn-icon">
138+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
139139
<svg aria-hidden="true">
140140
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
141141
</svg>
@@ -156,7 +156,7 @@ Find here an example of how to implement a password input with a show password b
156156
<div class="text-input-container">
157157
<label for="liveInputPassword">Password</label>
158158
<input type="password" id="liveInputPassword" class="text-input-field" placeholder=" ">
159-
<button class="btn btn-minimal btn-icon" id="togglePassword" type="button">
159+
<button class="btn btn-minimal btn-icon" id="togglePassword" type="button" aria-pressed="false">
160160
<svg aria-hidden="true">
161161
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
162162
</svg>

site/src/content/docs/components/skeleton.mdx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ To set components in skeleton state, you can wrap any of them in an `[aria-busy=
174174
<div class="input-container" data-bs-prefix="DEV-">
175175
<input type="password" id="inputPasswordPrefix" class="text-input-field" placeholder=" ">
176176
</div>
177-
<button class="btn btn-minimal btn-icon">
177+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
178178
<svg aria-hidden="true">
179179
<use xlink:href="${getVersionedDocsPath('/assets/img/ouds-web-sprite.svg#accessibility-vision')}"/>
180180
</svg>

site/src/content/docs/foundation/form-validation.mdx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ Try to submit the form below; our JavaScript will intercept the submit event, an
9797
<div class="text-input-container">
9898
<label for="inputPassword">Password</label>
9999
<input type="password" id="inputPassword" class="text-input-field" aria-describedby="inputPasswordPrefixHelper" data-errormessage="passwordFeedback" autocomplete="new-password" placeholder=" " required minlength="8">
100-
<button class="btn btn-minimal btn-icon">
100+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
101101
<svg aria-hidden="true">
102102
<use xlink:href="${getVersionedDocsPath("/assets/img/ouds-web-sprite.svg#accessibility-vision")}"/>
103103
</svg>
@@ -231,7 +231,7 @@ For invalid fields, ensure that the invalid feedback/error message is associated
231231
<div class="text-input-container">
232232
<label for="inputPassword2">Password</label>
233233
<input type="password" id="inputPassword2" class="text-input-field" aria-invalid="true" aria-describedby="inputPassword2PrefixHelper password2Feedback" autocomplete="new-password" placeholder=" " required minlength="8">
234-
<button class="btn btn-minimal btn-icon">
234+
<button class="btn btn-minimal btn-icon" type="button" aria-pressed="false">
235235
<svg aria-hidden="true">
236236
<use xlink:href="${getVersionedDocsPath("/assets/img/ouds-web-sprite.svg#accessibility-vision")}"/>
237237
</svg>

0 commit comments

Comments
 (0)