diff --git a/images/movie.svg b/images/movie.svg new file mode 100644 index 000000000..11700796f --- /dev/null +++ b/images/movie.svg @@ -0,0 +1 @@ + \ No newline at end of file diff --git a/stylesheets/_conversation.scss b/stylesheets/_conversation.scss index bec5b49a1..2ece4aa95 100644 --- a/stylesheets/_conversation.scss +++ b/stylesheets/_conversation.scss @@ -788,15 +788,18 @@ span.status { .close-container { position: absolute; - top: 0px; - right: 0px; - height: 18px; - width: 18px; + top: 4px; + right: 4px; + height: 16px; + width: 16px; + + background-color: rgba(255, 255, 255, 0.75); + border-radius: 50%; .close-button { width: 100%; height: 100%; - @include color-svg('../images/x.svg', white); + @include color-svg('../images/x.svg', $grey); } } @@ -840,6 +843,9 @@ span.status { &.play { @include color-svg('../images/play.svg', white); } + &.movie { + @include color-svg('../images/movie.svg', white); + } @include avatar-colors; } @@ -853,9 +859,12 @@ span.status { align-items: center; justify-content: center; + object-fit: cover; + img { - max-width: 100%; - max-height: 100%; + width: 100%; + height: 100%; + object-fit: cover; } } } diff --git a/stylesheets/_ios.scss b/stylesheets/_ios.scss index 4b7ee8e12..c5f398e0e 100644 --- a/stylesheets/_ios.scss +++ b/stylesheets/_ios.scss @@ -194,6 +194,9 @@ $ios-border-color: rgba(0,0,0,0.1); height: 48px; position: relative; + top: auto; + right: auto; + display: flex; align-items: center; justify-content: center; diff --git a/ts/components/conversation/Quote.md b/ts/components/conversation/Quote.md index 838135fce..e97482679 100644 --- a/ts/components/conversation/Quote.md +++ b/ts/components/conversation/Quote.md @@ -183,9 +183,8 @@ const View = Whisper.MessageView; #### A lot of text in quotation, with image ```jsx -const quotedMessage = { - imageUrl: util.gifObjectUrl, - id: '3234-23423-2342', +const thumbnail = { + objectUrl: util.gifObjectUrl, }; const outgoing = new Whisper.Message({ type: 'outgoing', @@ -218,8 +217,8 @@ const incoming = new Whisper.Message(Object.assign({}, outgoing.attributes, { }), })); -outgoing.quotedMessage = quotedMessage; -incoming.quotedMessage = quotedMessage; +outgoing.quoteThumbnail = thumbnail; +incoming.quoteThumbnail = thumbnail; const View = Whisper.MessageView; @@ -237,8 +236,8 @@ const View = Whisper.MessageView; #### Image with caption ```jsx -const quotedMessage = { - imageUrl: util.gifObjectUrl, +const thumbnail = { + objectUrl: util.gifObjectUrl, id: '3234-23423-2342', }; const outgoing = new Whisper.Message({ @@ -268,8 +267,8 @@ const incoming = new Whisper.Message(Object.assign({}, outgoing.attributes, { }), })); -outgoing.quotedMessage = quotedMessage; -incoming.quotedMessage = quotedMessage; +outgoing.quoteThumbnail = thumbnail; +incoming.quoteThumbnail = thumbnail; const View = Whisper.MessageView; @@ -287,8 +286,8 @@ const View = Whisper.MessageView; #### Image ```jsx -const quotedMessage = { - imageUrl: util.gifObjectUrl, +const thumbnail = { + objectUrl: util.gifObjectUrl, }; const outgoing = new Whisper.Message({ @@ -317,8 +316,8 @@ const incoming = new Whisper.Message(Object.assign({}, outgoing.attributes, { }), })); -outgoing.quotedMessage = quotedMessage; -incoming.quotedMessage = quotedMessage; +outgoing.quoteThumbnail = thumbnail; +incoming.quoteThumbnail = thumbnail; const View = Whisper.MessageView; @@ -375,8 +374,8 @@ const View = Whisper.MessageView; #### Video with caption ```jsx -const quotedMessage = { - imageUrl: util.gifObjectUrl, +const thumbnail = { + objectUrl: util.gifObjectUrl, }; const outgoing = new Whisper.Message({ @@ -406,8 +405,8 @@ const incoming = new Whisper.Message(Object.assign({}, outgoing.attributes, { }), })); -outgoing.quotedMessage = quotedMessage; -incoming.quotedMessage = quotedMessage; +outgoing.quoteThumbnail = thumbnail; +incoming.quoteThumbnail = thumbnail; const View = Whisper.MessageView; @@ -425,8 +424,8 @@ const View = Whisper.MessageView; #### Video ```jsx -const quotedMessage = { - imageUrl: util.gifObjectUrl, +const thumbnail = { + objectUrl: util.gifObjectUrl, }; const outgoing = new Whisper.Message({ @@ -456,8 +455,8 @@ const incoming = new Whisper.Message(Object.assign({}, outgoing.attributes, { }), })); -outgoing.quotedMessage = quotedMessage; -incoming.quotedMessage = quotedMessage; +outgoing.quoteThumbnail = thumbnail; +incoming.quoteThumbnail = thumbnail; const View = Whisper.MessageView; diff --git a/ts/components/conversation/Quote.tsx b/ts/components/conversation/Quote.tsx index 678435371..22342665b 100644 --- a/ts/components/conversation/Quote.tsx +++ b/ts/components/conversation/Quote.tsx @@ -95,7 +95,7 @@ export class Quote extends React.Component { if (Mime.isVideo(contentType)) { return objectUrl ? this.renderImage(objectUrl, 'play') - : this.renderIcon('play'); + : this.renderIcon('movie'); } if (Mime.isImage(contentType)) { return objectUrl