[Glitch] Fix design issues with sensitive preview cards
Port 1d2b0d2121 to glitch-soc
Signed-off-by: Thibaut Girka <thib@sitedethib.com>
			
			
This commit is contained in:
		
							parent
							
								
									9a26d2e127
								
							
						
					
					
						commit
						327f6309e4
					
				
					 2 changed files with 6 additions and 4 deletions
				
			
		| 
						 | 
					@ -156,7 +156,9 @@ export default class Card extends React.PureComponent {
 | 
				
			||||||
    this.setState({ previewLoaded: true });
 | 
					    this.setState({ previewLoaded: true });
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  handleReveal = () => {
 | 
					  handleReveal = e => {
 | 
				
			||||||
 | 
					    e.preventDefault();
 | 
				
			||||||
 | 
					    e.stopPropagation();
 | 
				
			||||||
    this.setState({ revealed: true });
 | 
					    this.setState({ revealed: true });
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -244,7 +246,7 @@ export default class Card extends React.PureComponent {
 | 
				
			||||||
      }
 | 
					      }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
      return (
 | 
					      return (
 | 
				
			||||||
        <div className={className} ref={this.setRef}>
 | 
					        <div className={className} ref={this.setRef} onClick={revealed ? null : this.handleReveal} role={revealed ? 'button' : null}>
 | 
				
			||||||
          {embed}
 | 
					          {embed}
 | 
				
			||||||
          {!compact && description}
 | 
					          {!compact && description}
 | 
				
			||||||
        </div>
 | 
					        </div>
 | 
				
			||||||
| 
						 | 
					@ -254,14 +256,12 @@ export default class Card extends React.PureComponent {
 | 
				
			||||||
        <div className='status-card__image'>
 | 
					        <div className='status-card__image'>
 | 
				
			||||||
          {canvas}
 | 
					          {canvas}
 | 
				
			||||||
          {thumbnail}
 | 
					          {thumbnail}
 | 
				
			||||||
          {!revealed && spoilerButton}
 | 
					 | 
				
			||||||
        </div>
 | 
					        </div>
 | 
				
			||||||
      );
 | 
					      );
 | 
				
			||||||
    } else {
 | 
					    } else {
 | 
				
			||||||
      embed = (
 | 
					      embed = (
 | 
				
			||||||
        <div className='status-card__image'>
 | 
					        <div className='status-card__image'>
 | 
				
			||||||
          <Icon id='file-text' />
 | 
					          <Icon id='file-text' />
 | 
				
			||||||
          {!revealed && spoilerButton}
 | 
					 | 
				
			||||||
        </div>
 | 
					        </div>
 | 
				
			||||||
      );
 | 
					      );
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
| 
						 | 
					@ -270,6 +270,7 @@ export default class Card extends React.PureComponent {
 | 
				
			||||||
      <a href={card.get('url')} className={className} target='_blank' rel='noopener noreferrer' ref={this.setRef}>
 | 
					      <a href={card.get('url')} className={className} target='_blank' rel='noopener noreferrer' ref={this.setRef}>
 | 
				
			||||||
        {embed}
 | 
					        {embed}
 | 
				
			||||||
        {description}
 | 
					        {description}
 | 
				
			||||||
 | 
					        {!revealed && spoilerButton}
 | 
				
			||||||
      </a>
 | 
					      </a>
 | 
				
			||||||
    );
 | 
					    );
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -776,6 +776,7 @@ a.status__display-name,
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
.status-card {
 | 
					.status-card {
 | 
				
			||||||
 | 
					  position: relative;
 | 
				
			||||||
  display: flex;
 | 
					  display: flex;
 | 
				
			||||||
  font-size: 14px;
 | 
					  font-size: 14px;
 | 
				
			||||||
  border: 1px solid lighten($ui-base-color, 8%);
 | 
					  border: 1px solid lighten($ui-base-color, 8%);
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue