diff options
author | samogot <samogot@gmail.com> | 2016-05-28 02:31:02 +0300 |
---|---|---|
committer | Harrison Healey <harrisonmhealey@gmail.com> | 2016-05-27 19:31:02 -0400 |
commit | e1bebb2d776b12d6e1461e06f318e79fcb93ea2c (patch) | |
tree | 9cac8e57867a3b16589162f620f991c0ea6ea94d /webapp/components/youtube_video.jsx | |
parent | c7615920df6b25d0e8d59e0aaf00d2f772ef55e4 (diff) | |
download | chat-e1bebb2d776b12d6e1461e06f318e79fcb93ea2c.tar.gz chat-e1bebb2d776b12d6e1461e06f318e79fcb93ea2c.tar.bz2 chat-e1bebb2d776b12d6e1461e06f318e79fcb93ea2c.zip |
PLT-2321 Move the toggle icon next to the link being previewed (#3071)
* PLT-2321 Move the toggle icon next to the link being previewed
only applicable to one-line messages started with link
* remove useless "Youtube" header when no title is available
* allow breaking long links instead of toggle
* simplify "/" to "/<wbr />" replacing
fix empty post without additional content body
* discard buggy "simplification"
Diffstat (limited to 'webapp/components/youtube_video.jsx')
-rw-r--r-- | webapp/components/youtube_video.jsx | 30 |
1 files changed, 16 insertions, 14 deletions
diff --git a/webapp/components/youtube_video.jsx b/webapp/components/youtube_video.jsx index dc2d368d7..42b193612 100644 --- a/webapp/components/youtube_video.jsx +++ b/webapp/components/youtube_video.jsx @@ -144,9 +144,22 @@ export default class YoutubeVideo extends React.Component { return <div className='video-loading'/>; } - let header = 'Youtube'; + let header; if (this.state.title) { - header = header + ' - '; + header = ( + <h4> + <span className='video-type'>{'Youtube - '}</span> + <span className='video-title'> + <a + href={this.props.link} + target='blank' + rel='noopener noreferrer' + > + {this.state.title} + </a> + </span> + </h4> + ); } let content; @@ -190,18 +203,7 @@ export default class YoutubeVideo extends React.Component { return ( <div> - <h4> - <span className='video-type'>{header}</span> - <span className='video-title'> - <a - href={this.props.link} - target='blank' - rel='noopener noreferrer' - > - {this.state.title} - </a> - </span> - </h4> + {header} <div className='video-div embed-responsive-item' onClick={this.play} |