react/jsx-a11y eslint 为 control/label 抛出意外错误

Question

This code:

      <PopupContent>
        <label htmlFor="popup" style={{ margintop : '0px', marginbottom : '0px' }}>log out</label>
        <button type="button" id="popup" onClick={this.logUserOut} />
      </PopupContent>

throws these two errors:

 84:9  error  A form label must be associated with a control  jsx-a11y/label-has-associated-control
 85:9  error  A control must be associated with a text label  jsx-a11y/control-has-associated-label

What am I missing?

Answer 1

看起来这个规则还没有完全成熟： 当它变得更稳定时，请参阅考虑添加jsx-a11y/control-has-associated-label

Answer 2

You don't typically use a element to label a button. More commonly, the text inside the button serves as its label. Try something along the lines of:

<button type="button id="popup" onClick={this.logUserOut}>log out</button>

Other options could be any one of the following:

<button type="button" id="popup" onClick={this.logUserOut} aria-label="log out"/>

or

<span id="button-label" style={{ margintop : '0px', marginbottom : '0px' }}>log out</span>
<button type="button" id="popup" onClick={this.logUserOut} aria-labelledby="button-label" />

or

<button type="button id="popup" onClick={this.logUserOut}>
    <img src="icon.png" alt="log out"/>
</button>

or

<button type="button id="popup" onClick={this.logUserOut}>
    <span className="off-screen">log out</span>
</button>

where the CSS class of .off-screen hides the text off-screen in an accessible way eg do not use display: none; or visibility: hidden;