diff --git a/app/javascript/flavours/glitch/features/account_gallery/components/media_item.js b/app/javascript/flavours/glitch/features/account_gallery/components/media_item.js
index b88f23aa4d..781bd4e039 100644
--- a/app/javascript/flavours/glitch/features/account_gallery/components/media_item.js
+++ b/app/javascript/flavours/glitch/features/account_gallery/components/media_item.js
@@ -61,73 +61,81 @@ export default class MediaItem extends ImmutablePureComponent {
const width = `${Math.floor((displayWidth - 4) / 3) - 4}px`;
const height = width;
const status = attachment.get('status');
- const title = status.get('spoiler_text') || attachment.get('description');
+ const title = status.get('spoiler_text') || attachment.get('description');
- let thumbnail = '';
+ let thumbnail, label, icon, content;
- if (attachment.get('type') === 'unknown') {
- // Skip
- } else if (attachment.get('type') === 'audio') {
- thumbnail = (
+ if (!visible) {
+ icon = (
-
+
);
- } else if (attachment.get('type') === 'image') {
- const focusX = attachment.getIn(['meta', 'focus', 'x']) || 0;
- const focusY = attachment.getIn(['meta', 'focus', 'y']) || 0;
- const x = ((focusX / 2) + .5) * 100;
- const y = ((focusY / -2) + .5) * 100;
+ } else {
+ if (['audio', 'video'].includes(attachment.get('type'))) {
+ content = (
+
+ );
- thumbnail = (
-
- );
- } else if (['gifv', 'video'].indexOf(attachment.get('type')) !== -1) {
- const autoPlay = !isIOS() && autoPlayGif;
- const label = attachment.get('type') === 'video' ? : 'GIF';
+ if (attachment.get('type') === 'audio') {
+ label = ;
+ } else {
+ label = ;
+ }
+ } else if (attachment.get('type') === 'image') {
+ const focusX = attachment.getIn(['meta', 'focus', 'x']) || 0;
+ const focusY = attachment.getIn(['meta', 'focus', 'y']) || 0;
+ const x = ((focusX / 2) + .5) * 100;
+ const y = ((focusY / -2) + .5) * 100;
- thumbnail = (
-
+ content = (
+
+ );
+ } else if (attachment.get('type') === 'gifv') {
+ content = (
+ );
+
+ label = 'GIF';
+ }
+
+ thumbnail = (
+
+ {content}
{label}
);
}
- const icon = (
-
-
-
- );
-
return (
diff --git a/app/javascript/flavours/glitch/features/account_gallery/index.js b/app/javascript/flavours/glitch/features/account_gallery/index.js
index f5fe6c930c..040741c2a6 100644
--- a/app/javascript/flavours/glitch/features/account_gallery/index.js
+++ b/app/javascript/flavours/glitch/features/account_gallery/index.js
@@ -113,9 +113,9 @@ class AccountGallery extends ImmutablePureComponent {
handleOpenMedia = attachment => {
if (attachment.get('type') === 'video') {
- this.props.dispatch(openModal('VIDEO', { media: attachment, status: attachment.get('status') }));
+ this.props.dispatch(openModal('VIDEO', { media: attachment, status: attachment.get('status'), options: { autoPlay: true } }));
} else if (attachment.get('type') === 'audio') {
- this.props.dispatch(openModal('AUDIO', { media: attachment, status: attachment.get('status') }));
+ this.props.dispatch(openModal('AUDIO', { media: attachment, status: attachment.get('status'), options: { autoPlay: true } }));
} else {
const media = attachment.getIn(['status', 'media_attachments']);
const index = media.findIndex(x => x.get('id') === attachment.get('id'));
diff --git a/app/javascript/flavours/glitch/features/audio/index.js b/app/javascript/flavours/glitch/features/audio/index.js
index 120a5ce1a5..9cb2307fda 100644
--- a/app/javascript/flavours/glitch/features/audio/index.js
+++ b/app/javascript/flavours/glitch/features/audio/index.js
@@ -37,6 +37,7 @@ class Audio extends React.PureComponent {
backgroundColor: PropTypes.string,
foregroundColor: PropTypes.string,
accentColor: PropTypes.string,
+ autoPlay: PropTypes.bool,
};
state = {
@@ -244,6 +245,14 @@ class Audio extends React.PureComponent {
this.setState({ hovered: false });
}
+ handleLoadedData = () => {
+ const { autoPlay } = this.props;
+
+ if (autoPlay) {
+ this.audio.play();
+ }
+ }
+
_initAudioContext () {
const context = new AudioContext();
const source = context.createMediaElementSource(this.audio);
@@ -321,7 +330,7 @@ class Audio extends React.PureComponent {
}
render () {
- const { src, intl, alt, editable } = this.props;
+ const { src, intl, alt, editable, autoPlay } = this.props;
const { paused, muted, volume, currentTime, duration, buffer, dragging } = this.state;
const progress = (currentTime / duration) * 100;
@@ -330,10 +339,11 @@ class Audio extends React.PureComponent {
diff --git a/app/javascript/flavours/glitch/features/ui/components/audio_modal.js b/app/javascript/flavours/glitch/features/ui/components/audio_modal.js
index f0c3b3bccb..f9d4bb2f30 100644
--- a/app/javascript/flavours/glitch/features/ui/components/audio_modal.js
+++ b/app/javascript/flavours/glitch/features/ui/components/audio_modal.js
@@ -2,16 +2,26 @@ import React from 'react';
import ImmutablePropTypes from 'react-immutable-proptypes';
import PropTypes from 'prop-types';
import Audio from 'flavours/glitch/features/audio';
+import { connect } from 'react-redux';
import ImmutablePureComponent from 'react-immutable-pure-component';
import { FormattedMessage } from 'react-intl';
import classNames from 'classnames';
import Icon from 'flavours/glitch/components/icon';
-export default class AudioModal extends ImmutablePureComponent {
+const mapStateToProps = (state, { status }) => ({
+ account: state.getIn(['accounts', status.get('account')]),
+});
+
+export default @connect(mapStateToProps)
+class AudioModal extends ImmutablePureComponent {
static propTypes = {
media: ImmutablePropTypes.map.isRequired,
status: ImmutablePropTypes.map,
+ options: PropTypes.shape({
+ autoPlay: PropTypes.bool,
+ }),
+ account: ImmutablePropTypes.map,
onClose: PropTypes.func.isRequired,
};
@@ -27,7 +37,8 @@ export default class AudioModal extends ImmutablePureComponent {
}
render () {
- const { media, status } = this.props;
+ const { media, status, account } = this.props;
+ const options = this.props.options || {};
return (
@@ -37,10 +48,11 @@ export default class AudioModal extends ImmutablePureComponent {
alt={media.get('description')}
duration={media.getIn(['meta', 'original', 'duration'], 0)}
height={150}
- poster={media.get('preview_url') || status.getIn(['account', 'avatar_static'])}
+ poster={media.get('preview_url') || account.get('avatar_static')}
backgroundColor={media.getIn(['meta', 'colors', 'background'])}
foregroundColor={media.getIn(['meta', 'colors', 'foreground'])}
accentColor={media.getIn(['meta', 'colors', 'accent'])}
+ autoPlay={options.autoPlay}
/>
diff --git a/app/javascript/flavours/glitch/selectors/index.js b/app/javascript/flavours/glitch/selectors/index.js
index 4a3303c362..bb9180d120 100644
--- a/app/javascript/flavours/glitch/selectors/index.js
+++ b/app/javascript/flavours/glitch/selectors/index.js
@@ -183,12 +183,13 @@ export const makeGetNotification = () => {
export const getAccountGallery = createSelector([
(state, id) => state.getIn(['timelines', `account:${id}:media`, 'items'], ImmutableList()),
state => state.get('statuses'),
-], (statusIds, statuses) => {
+ (state, id) => state.getIn(['accounts', id]),
+], (statusIds, statuses, account) => {
let medias = ImmutableList();
statusIds.forEach(statusId => {
const status = statuses.get(statusId);
- medias = medias.concat(status.get('media_attachments').map(media => media.set('status', status)));
+ medias = medias.concat(status.get('media_attachments').map(media => media.set('status', status).set('account', account)));
});
return medias;