Rename ariaLabel property of Dropdown to title (#5813)
DropdownMenu has ariaLabel property, but it is actually applied to title property of IconButton. Keep it consistent.
This commit is contained in:
		
							parent
							
								
									d07983b56d
								
							
						
					
					
						commit
						1266c66f79
					
				
					 3 changed files with 6 additions and 6 deletions
				
			
		| 
						 | 
				
			
			@ -110,7 +110,7 @@ export default class Dropdown extends React.PureComponent {
 | 
			
		|||
    icon: PropTypes.string.isRequired,
 | 
			
		||||
    items: PropTypes.array.isRequired,
 | 
			
		||||
    size: PropTypes.number.isRequired,
 | 
			
		||||
    ariaLabel: PropTypes.string,
 | 
			
		||||
    title: PropTypes.string,
 | 
			
		||||
    disabled: PropTypes.bool,
 | 
			
		||||
    status: ImmutablePropTypes.map,
 | 
			
		||||
    isUserTouching: PropTypes.func,
 | 
			
		||||
| 
						 | 
				
			
			@ -120,7 +120,7 @@ export default class Dropdown extends React.PureComponent {
 | 
			
		|||
  };
 | 
			
		||||
 | 
			
		||||
  static defaultProps = {
 | 
			
		||||
    ariaLabel: 'Menu',
 | 
			
		||||
    title: 'Menu',
 | 
			
		||||
  };
 | 
			
		||||
 | 
			
		||||
  state = {
 | 
			
		||||
| 
						 | 
				
			
			@ -186,14 +186,14 @@ export default class Dropdown extends React.PureComponent {
 | 
			
		|||
  }
 | 
			
		||||
 | 
			
		||||
  render () {
 | 
			
		||||
    const { icon, items, size, ariaLabel, disabled } = this.props;
 | 
			
		||||
    const { icon, items, size, title, disabled } = this.props;
 | 
			
		||||
    const { expanded } = this.state;
 | 
			
		||||
 | 
			
		||||
    return (
 | 
			
		||||
      <div onKeyDown={this.handleKeyDown}>
 | 
			
		||||
        <IconButton
 | 
			
		||||
          icon={icon}
 | 
			
		||||
          title={ariaLabel}
 | 
			
		||||
          title={title}
 | 
			
		||||
          active={expanded}
 | 
			
		||||
          disabled={disabled}
 | 
			
		||||
          size={size}
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -179,7 +179,7 @@ export default class StatusActionBar extends ImmutablePureComponent {
 | 
			
		|||
        {shareButton}
 | 
			
		||||
 | 
			
		||||
        <div className='status__action-bar-dropdown'>
 | 
			
		||||
          <DropdownMenuContainer disabled={anonymousAccess} status={status} items={menu} icon='ellipsis-h' size={18} direction='right' ariaLabel={intl.formatMessage(messages.more)} />
 | 
			
		||||
          <DropdownMenuContainer disabled={anonymousAccess} status={status} items={menu} icon='ellipsis-h' size={18} direction='right' title={intl.formatMessage(messages.more)} />
 | 
			
		||||
        </div>
 | 
			
		||||
      </div>
 | 
			
		||||
    );
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -120,7 +120,7 @@ export default class ActionBar extends React.PureComponent {
 | 
			
		|||
        {shareButton}
 | 
			
		||||
 | 
			
		||||
        <div className='detailed-status__action-bar-dropdown'>
 | 
			
		||||
          <DropdownMenuContainer size={18} icon='ellipsis-h' items={menu} direction='left' ariaLabel='More' />
 | 
			
		||||
          <DropdownMenuContainer size={18} icon='ellipsis-h' items={menu} direction='left' title='More' />
 | 
			
		||||
        </div>
 | 
			
		||||
      </div>
 | 
			
		||||
    );
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		
		Reference in a new issue