From 16fef9633898854448e3545515af41c11286a52e Mon Sep 17 00:00:00 2001
From: Oliwia Rogala <oliwia.rogala@smartbear.com>
Date: Thu, 15 Feb 2024 11:17:11 +0100
Subject: [PATCH] fix: associate HTML labels with inputs (#9599)

Refs #8879

Co-authored-by: morsko1 <morssko1@gmail.com>
---
 src/core/components/auth/api-key-auth.jsx     | 11 ++++-
 src/core/components/auth/basic-auth.jsx       | 28 +++++++++---
 src/core/components/param-body.jsx            | 10 ++++-
 .../components/parameter-include-empty.jsx    | 16 ++++---
 src/core/components/parameters/parameters.jsx |  9 +++-
 .../oas3/components/auth/http-auth.jsx        | 45 ++++++++++++++-----
 6 files changed, 90 insertions(+), 29 deletions(-)

diff --git a/src/core/components/auth/api-key-auth.jsx b/src/core/components/auth/api-key-auth.jsx
index 5b3527e8c06..f85c6014c05 100644
--- a/src/core/components/auth/api-key-auth.jsx
+++ b/src/core/components/auth/api-key-auth.jsx
@@ -66,10 +66,17 @@ export default class ApiKeyAuth extends React.Component {
           <p>In: <code>{ schema.get("in") }</code></p>
         </Row>
         <Row>
-          <label>Value:</label>
+          <label htmlFor="api_key_value">Value:</label>
           {
             value ? <code> ****** </code>
-                  : <Col><Input type="text" onChange={ this.onChange } autoFocus/></Col>
+                  : <Col>
+                      <Input 
+                        id="api_key_value" 
+                        type="text" 
+                        onChange={ this.onChange } 
+                        autoFocus
+                      />
+                    </Col>
           }
         </Row>
         {
diff --git a/src/core/components/auth/basic-auth.jsx b/src/core/components/auth/basic-auth.jsx
index baa5268ffc9..f982f71f624 100644
--- a/src/core/components/auth/basic-auth.jsx
+++ b/src/core/components/auth/basic-auth.jsx
@@ -65,20 +65,34 @@ export default class BasicAuth extends React.Component {
           <Markdown source={ schema.get("description") } />
         </Row>
         <Row>
-          <label>Username:</label>
+          <label htmlFor="auth_username">Username:</label>
           {
             username ? <code> { username } </code>
-                     : <Col><Input type="text" required="required" name="username" onChange={ this.onChange } autoFocus/></Col>
+                     : <Col>
+                          <Input 
+                            id="auth_username" 
+                            type="text" 
+                            required="required" 
+                            name="username" 
+                            onChange={ this.onChange } 
+                            autoFocus
+                          />
+                        </Col>
           }
         </Row>
         <Row>
-          <label>Password:</label>
+          <label htmlFor="auth_password">Password:</label>
             {
               username ? <code> ****** </code>
-                       : <Col><Input autoComplete="new-password"
-                                     name="password"
-                                     type="password"
-                                     onChange={ this.onChange }/></Col>
+                       : <Col>
+                            <Input 
+                              id="auth_password"
+                              autoComplete="new-password"
+                              name="password"
+                              type="password"
+                              onChange={ this.onChange }
+                            />
+                          </Col>
             }
         </Row>
         {
diff --git a/src/core/components/param-body.jsx b/src/core/components/param-body.jsx
index 3ac5a21e88e..11eeef9f7b9 100644
--- a/src/core/components/param-body.jsx
+++ b/src/core/components/param-body.jsx
@@ -2,6 +2,7 @@ import React, { PureComponent } from "react"
 import PropTypes from "prop-types"
 import { fromJS, List } from "immutable"
 import { getKnownSyntaxHighlighterLanguage } from "core/utils/jsonParse"
+import createHtmlReadyId from "core/utils/create-html-ready-id"
 
 const NOOP = Function.prototype
 
@@ -118,6 +119,9 @@ export default class ParamBody extends PureComponent {
       language = "json"
     }
 
+    const regionId = createHtmlReadyId(`${pathMethod[1]}${pathMethod[0]}_parameters`)
+    const controlId = `${regionId}_select`
+
     return (
       <div className="body-param" data-param-name={param.get("name")} data-param-in={param.get("in")}>
         {
@@ -137,14 +141,16 @@ export default class ParamBody extends PureComponent {
                          </Button>
                          </div>
           }
-          <label htmlFor="">
+          <label htmlFor={controlId}>
             <span>Parameter content type</span>
             <ContentType
               value={ consumesValue }
               contentTypes={ consumes }
               onChange={onChangeConsumes}
               className="body-param-content-type"
-              ariaLabel="Parameter content type" />
+              ariaLabel="Parameter content type" 
+              controlId={controlId}
+            />
           </label>
         </div>
 
diff --git a/src/core/components/parameter-include-empty.jsx b/src/core/components/parameter-include-empty.jsx
index 6656574c07c..35bc286514e 100644
--- a/src/core/components/parameter-include-empty.jsx
+++ b/src/core/components/parameter-include-empty.jsx
@@ -38,13 +38,19 @@ export default class ParameterIncludeEmpty extends Component {
 
     return (
       <div>
-        <label className={cx("parameter__empty_value_toggle", {
-          "disabled": isDisabled
-        })}>
-          <input type="checkbox" 
+        <label 
+          htmlFor="include_empty_value" 
+          className={cx("parameter__empty_value_toggle", {
+            "disabled": isDisabled
+          })}
+        >
+          <input 
+            id="include_empty_value"
+            type="checkbox" 
             disabled={isDisabled}
             checked={!isDisabled && isIncluded}
-            onChange={this.onCheckboxChange} />
+            onChange={this.onCheckboxChange} 
+          />
           Send empty value
         </label>
       </div>
diff --git a/src/core/components/parameters/parameters.jsx b/src/core/components/parameters/parameters.jsx
index d76b23a56f6..ee3e46a85fb 100644
--- a/src/core/components/parameters/parameters.jsx
+++ b/src/core/components/parameters/parameters.jsx
@@ -2,6 +2,7 @@ import React, { Component } from "react"
 import PropTypes from "prop-types"
 import { Map, List } from "immutable"
 import ImPropTypes from "react-immutable-proptypes"
+import createHtmlReadyId from "core/utils/create-html-ready-id"
 
 export default class Parameters extends Component {
 
@@ -120,6 +121,8 @@ export default class Parameters extends Component {
     const isExecute = tryItOutEnabled && allowTryItOut
     const isOAS3 = specSelectors.isOAS3()
 
+    const regionId = createHtmlReadyId(`${pathMethod[1]}${pathMethod[0]}_requests`)
+    const controlId = `${regionId}_select`
 
     const requestBody = operation.get("requestBody")
 
@@ -215,7 +218,7 @@ export default class Parameters extends Component {
             <div className="opblock-section-header">
               <h4 className={`opblock-title parameter__name ${requestBody.get("required") && "required"}`}>Request
                 body</h4>
-              <label>
+              <label id={controlId}>
                 <ContentType
                   value={oas3Selectors.requestContentType(...pathMethod)}
                   contentTypes={requestBody.get("content", List()).keySeq()}
@@ -223,7 +226,9 @@ export default class Parameters extends Component {
                     this.onChangeMediaType({ value, pathMethod })
                   }}
                   className="body-param-content-type"
-                  ariaLabel="Request content type" />
+                  ariaLabel="Request content type" 
+                  controlId={controlId}
+                />
               </label>
             </div>
             <div className="opblock-description-wrapper">
diff --git a/src/core/plugins/oas3/components/auth/http-auth.jsx b/src/core/plugins/oas3/components/auth/http-auth.jsx
index 069be3ff519..9599fa8b1a8 100644
--- a/src/core/plugins/oas3/components/auth/http-auth.jsx
+++ b/src/core/plugins/oas3/components/auth/http-auth.jsx
@@ -71,22 +71,37 @@ export default class HttpAuth extends React.Component {
           <Markdown source={ schema.get("description") } />
         </Row>
         <Row>
-          <label>Username:</label>
+          <label htmlFor="auth-basic-username">Username:</label>
           {
             username ? <code> { username } </code>
-              : <Col><Input type="text" required="required" name="username" aria-label="auth-basic-username" onChange={ this.onChange } autoFocus/></Col>
+              : <Col>
+                  <Input 
+                    id="auth-basic-username"
+                    type="text"
+                    required="required"
+                    name="username"
+                    aria-label="auth-basic-username"
+                    onChange={ this.onChange }
+                    autoFocus
+                  />
+                </Col>
           }
         </Row>
         <Row>
-          <label>Password:</label>
+          <label htmlFor="auth-basic-password">Password:</label>
             {
               username ? <code> ****** </code>
-                       : <Col><Input autoComplete="new-password"
-                                     name="password"
-                                     type="password"
-                                     aria-label="auth-basic-password"
-                                     onChange={ this.onChange }/></Col>
-            }
+                       : <Col>
+                            <Input 
+                              id="auth-basic-password"
+                              autoComplete="new-password"
+                              name="password"
+                              type="password"
+                              aria-label="auth-basic-password"
+                              onChange={ this.onChange }
+                            />
+                          </Col>
+          }
         </Row>
         {
           errors.valueSeq().map( (error, key) => {
@@ -110,10 +125,18 @@ export default class HttpAuth extends React.Component {
               <Markdown source={ schema.get("description") } />
             </Row>
             <Row>
-              <label>Value:</label>
+              <label htmlFor="auth-bearer-value">Value:</label>
               {
                 value ? <code> ****** </code>
-              : <Col><Input type="text" aria-label="auth-bearer-value" onChange={ this.onChange } autoFocus/></Col>
+              : <Col>
+                  <Input
+                    id="auth-bearer-value"
+                    type="text"
+                    aria-label="auth-bearer-value"
+                    onChange={ this.onChange }
+                    autoFocus
+                  />
+                </Col>
           }
         </Row>
         {