[Glitch] Fix hidden label causing accessibility issue for search inputs
Port 903e5a3f45 to glitch-soc
Signed-off-by: Claire <claire.github-309c@sitedethib.com>
			
			
This commit is contained in:
		
							parent
							
								
									8cd7b95559
								
							
						
					
					
						commit
						aad42cfc35
					
				| 
						 | 
				
			
			@ -144,28 +144,20 @@ class Search extends React.PureComponent {
 | 
			
		|||
 | 
			
		||||
    return (
 | 
			
		||||
      <div className='search'>
 | 
			
		||||
        <label>
 | 
			
		||||
          <span style={{ display: 'none' }}>{intl.formatMessage(messages.placeholder)}</span>
 | 
			
		||||
          <input
 | 
			
		||||
            ref={this.setRef}
 | 
			
		||||
            className='search__input'
 | 
			
		||||
            type='text'
 | 
			
		||||
            placeholder={intl.formatMessage(signedIn ? messages.placeholderSignedIn : messages.placeholder)}
 | 
			
		||||
            value={value || ''}
 | 
			
		||||
            onChange={this.handleChange}
 | 
			
		||||
            onKeyUp={this.handleKeyUp}
 | 
			
		||||
            onFocus={this.handleFocus}
 | 
			
		||||
            onBlur={this.handleBlur}
 | 
			
		||||
          />
 | 
			
		||||
        </label>
 | 
			
		||||
        <input
 | 
			
		||||
          ref={this.setRef}
 | 
			
		||||
          className='search__input'
 | 
			
		||||
          type='text'
 | 
			
		||||
          placeholder={intl.formatMessage(signedIn ? messages.placeholderSignedIn : messages.placeholder)}
 | 
			
		||||
          aria-label={intl.formatMessage(signedIn ? messages.placeholderSignedIn : messages.placeholder)}
 | 
			
		||||
          value={value || ''}
 | 
			
		||||
          onChange={this.handleChange}
 | 
			
		||||
          onKeyUp={this.handleKeyUp}
 | 
			
		||||
          onFocus={this.handleFocus}
 | 
			
		||||
          onBlur={this.handleBlur}
 | 
			
		||||
        />
 | 
			
		||||
 | 
			
		||||
        <div
 | 
			
		||||
          aria-label={intl.formatMessage(messages.placeholder)}
 | 
			
		||||
          className='search__icon'
 | 
			
		||||
          onClick={this.handleClear}
 | 
			
		||||
          role='button'
 | 
			
		||||
          tabIndex='0'
 | 
			
		||||
        >
 | 
			
		||||
        <div role='button' tabIndex='0' className='search__icon' onClick={this.handleClear}>
 | 
			
		||||
          <Icon id='search' className={hasValue ? '' : 'active'} />
 | 
			
		||||
          <Icon id='times-circle' className={hasValue ? 'active' : ''} />
 | 
			
		||||
        </div>
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue