From: Surinna Curtis Date: Thu, 16 Nov 2017 07:21:16 +0000 (-0600) Subject: Merge tootsuite/master at 30237259367a0ef2b20908518b86bbeb358999b5 X-Git-Url: https://git.xn--scling-oua.cat.family/?a=commitdiff_plain;h=35fbdc36f92b610e8a73e2acb220e87cf5fc83b0;p=mastodon.git Merge tootsuite/master at 30237259367a0ef2b20908518b86bbeb358999b5 --- 35fbdc36f92b610e8a73e2acb220e87cf5fc83b0 diff --cc app/controllers/api/v1/search_controller.rb index 997eed6e2,e183a71d7..d1b4e0402 --- a/app/controllers/api/v1/search_controller.rb +++ b/app/controllers/api/v1/search_controller.rb @@@ -1,9 -1,7 +1,9 @@@ # frozen_string_literal: true class Api::V1::SearchController < Api::BaseController + include Authorization + - RESULTS_LIMIT = 5 + RESULTS_LIMIT = 10 before_action -> { doorkeeper_authorize! :read } before_action :require_user! diff --cc app/javascript/glitch/components/account/header.js index 000000000,c94fb0851..7bc1a2189 mode 000000,100644..100644 --- a/app/javascript/glitch/components/account/header.js +++ b/app/javascript/glitch/components/account/header.js @@@ -1,0 -1,227 +1,227 @@@ + /* + + `` + ================= + + > For more information on the contents of this file, please contact: + > + > - kibigo! [@kibi@glitch.social] + + Original file by @gargron@mastodon.social et al as part of + tootsuite/mastodon. We've expanded it in order to handle user bio + frontmatter. + + The `` component provides the header for account + timelines. It is a fairly simple component which mostly just consists + of a `render()` method. + + __Props:__ + + - __`account` (`ImmutablePropTypes.map`) :__ + The account to render a header for. + + - __`me` (`PropTypes.number.isRequired`) :__ + The id of the currently-signed-in account. + + - __`onFollow` (`PropTypes.func.isRequired`) :__ + The function to call when the user clicks the "follow" button. + + - __`intl` (`PropTypes.object.isRequired`) :__ + Our internationalization object, inserted by `@injectIntl`. + + */ + + // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * + + /* + + Imports: + -------- + + */ + + // Package imports // + import React from 'react'; + import ImmutablePropTypes from 'react-immutable-proptypes'; + import PropTypes from 'prop-types'; + import { defineMessages, injectIntl, FormattedMessage } from 'react-intl'; + import ImmutablePureComponent from 'react-immutable-pure-component'; + + // Mastodon imports // + import emojify from '../../../mastodon/features/emoji/emoji'; + import IconButton from '../../../mastodon/components/icon_button'; + import Avatar from '../../../mastodon/components/avatar'; ++import { me } from '../../../mastodon/initial_state'; + + // Our imports // + import { processBio } from '../../util/bio_metadata'; + + // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * + + /* + + Inital setup: + ------------- + + The `messages` constant is used to define any messages that we need + from inside props. In our case, these are the `unfollow`, `follow`, and + `requested` messages used in the `title` of our buttons. + + */ + + const messages = defineMessages({ + unfollow: { id: 'account.unfollow', defaultMessage: 'Unfollow' }, + follow: { id: 'account.follow', defaultMessage: 'Follow' }, + requested: { id: 'account.requested', defaultMessage: 'Awaiting approval' }, + }); + + // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * + + /* + + Implementation: + --------------- + + */ + + @injectIntl + export default class AccountHeader extends ImmutablePureComponent { + + static propTypes = { + account : ImmutablePropTypes.map, - me : PropTypes.string.isRequired, + onFollow : PropTypes.func.isRequired, + intl : PropTypes.object.isRequired, + }; + + /* + + ### `render()` + + The `render()` function is used to render our component. + + */ + + render () { - const { account, me, intl } = this.props; ++ const { account, intl } = this.props; + + /* + + If no `account` is provided, then we can't render a header. Otherwise, + we get the `displayName` for the account, if available. If it's blank, + then we set the `displayName` to just be the `username` of the account. + + */ + + if (!account) { + return null; + } + + let displayName = account.get('display_name_html'); + let info = ''; + let actionBtn = ''; + let following = false; + + /* + + Next, we handle the account relationships. If the account follows the + user, then we add an `info` message. If the user has requested a + follow, then we disable the `actionBtn` and display an hourglass. + Otherwise, if the account isn't blocked, we set the `actionBtn` to the + appropriate icon. + + */ + + if (me !== account.get('id')) { + if (account.getIn(['relationship', 'followed_by'])) { + info = ( + + + + ); + } + if (account.getIn(['relationship', 'requested'])) { + actionBtn = ( +
+ +
+ ); + } else if (!account.getIn(['relationship', 'blocking'])) { + following = account.getIn(['relationship', 'following']); + actionBtn = ( +
+ +
+ ); + } + } + + /* + we extract the `text` and + `metadata` from our account's `note` using `processBio()`. + + */ + + const { text, metadata } = processBio(account.get('note')); + + /* + + Here, we render our component using all the things we've defined above. + + */ + + return ( +
+
+
+ + + + + + + + @{account.get('acct')} + {account.get('locked') ? : null} + +
+ + {info} + {actionBtn} +
+
+ + {metadata.length && ( + + + {(() => { + let data = []; + for (let i = 0; i < metadata.length; i++) { + data.push( + + + + + ); + } + return data; + })()} + +
+ ) || null} +
+ ); + } + + } diff --cc app/javascript/glitch/components/status/action_bar.js index 000000000,f4450d31b..34588b008 mode 000000,100644..100644 --- a/app/javascript/glitch/components/status/action_bar.js +++ b/app/javascript/glitch/components/status/action_bar.js @@@ -1,0 -1,188 +1,187 @@@ + // Package imports // + import React from 'react'; + import ImmutablePropTypes from 'react-immutable-proptypes'; + import PropTypes from 'prop-types'; + import { defineMessages, injectIntl } from 'react-intl'; + import ImmutablePureComponent from 'react-immutable-pure-component'; + + // Mastodon imports // + import RelativeTimestamp from '../../../mastodon/components/relative_timestamp'; + import IconButton from '../../../mastodon/components/icon_button'; + import DropdownMenuContainer from '../../../mastodon/containers/dropdown_menu_container'; ++import { me } from '../../../mastodon/initial_state'; + + const messages = defineMessages({ + delete: { id: 'status.delete', defaultMessage: 'Delete' }, + mention: { id: 'status.mention', defaultMessage: 'Mention @{name}' }, + mute: { id: 'account.mute', defaultMessage: 'Mute @{name}' }, + block: { id: 'account.block', defaultMessage: 'Block @{name}' }, + reply: { id: 'status.reply', defaultMessage: 'Reply' }, + share: { id: 'status.share', defaultMessage: 'Share' }, + replyAll: { id: 'status.replyAll', defaultMessage: 'Reply to thread' }, + reblog: { id: 'status.reblog', defaultMessage: 'Boost' }, + cannot_reblog: { id: 'status.cannot_reblog', defaultMessage: 'This post cannot be boosted' }, + favourite: { id: 'status.favourite', defaultMessage: 'Favourite' }, + open: { id: 'status.open', defaultMessage: 'Expand this status' }, + report: { id: 'status.report', defaultMessage: 'Report @{name}' }, + muteConversation: { id: 'status.mute_conversation', defaultMessage: 'Mute conversation' }, + unmuteConversation: { id: 'status.unmute_conversation', defaultMessage: 'Unmute conversation' }, + pin: { id: 'status.pin', defaultMessage: 'Pin on profile' }, + unpin: { id: 'status.unpin', defaultMessage: 'Unpin from profile' }, + embed: { id: 'status.embed', defaultMessage: 'Embed' }, + }); + + @injectIntl + export default class StatusActionBar extends ImmutablePureComponent { + + static contextTypes = { + router: PropTypes.object, + }; + + static propTypes = { + status: ImmutablePropTypes.map.isRequired, + onReply: PropTypes.func, + onFavourite: PropTypes.func, + onReblog: PropTypes.func, + onDelete: PropTypes.func, + onMention: PropTypes.func, + onMute: PropTypes.func, + onBlock: PropTypes.func, + onReport: PropTypes.func, + onEmbed: PropTypes.func, + onMuteConversation: PropTypes.func, + onPin: PropTypes.func, - me: PropTypes.string, + withDismiss: PropTypes.bool, + intl: PropTypes.object.isRequired, + }; + + // Avoid checking props that are functions (and whose equality will always + // evaluate to false. See react-immutable-pure-component for usage. + updateOnProps = [ + 'status', - 'me', + 'withDismiss', + ] + + handleReplyClick = () => { + this.props.onReply(this.props.status, this.context.router.history); + } + + handleShareClick = () => { + navigator.share({ + text: this.props.status.get('search_index'), + url: this.props.status.get('url'), + }); + } + + handleFavouriteClick = () => { + this.props.onFavourite(this.props.status); + } + + handleReblogClick = (e) => { + this.props.onReblog(this.props.status, e); + } + + handleDeleteClick = () => { + this.props.onDelete(this.props.status); + } + + handlePinClick = () => { + this.props.onPin(this.props.status); + } + + handleMentionClick = () => { + this.props.onMention(this.props.status.get('account'), this.context.router.history); + } + + handleMuteClick = () => { + this.props.onMute(this.props.status.get('account')); + } + + handleBlockClick = () => { + this.props.onBlock(this.props.status.get('account')); + } + + handleOpen = () => { + this.context.router.history.push(`/statuses/${this.props.status.get('id')}`); + } + + handleEmbed = () => { + this.props.onEmbed(this.props.status); + } + + handleReport = () => { + this.props.onReport(this.props.status); + } + + handleConversationMuteClick = () => { + this.props.onMuteConversation(this.props.status); + } + + render () { - const { status, me, intl, withDismiss } = this.props; ++ const { status, intl, withDismiss } = this.props; + + const mutingConversation = status.get('muted'); + const anonymousAccess = !me; + const publicStatus = ['public', 'unlisted'].includes(status.get('visibility')); + + let menu = []; + let reblogIcon = 'retweet'; + let replyIcon; + let replyTitle; + + menu.push({ text: intl.formatMessage(messages.open), action: this.handleOpen }); + + if (publicStatus) { + menu.push({ text: intl.formatMessage(messages.embed), action: this.handleEmbed }); + } + + menu.push(null); + + if (status.getIn(['account', 'id']) === me || withDismiss) { + menu.push({ text: intl.formatMessage(mutingConversation ? messages.unmuteConversation : messages.muteConversation), action: this.handleConversationMuteClick }); + menu.push(null); + } + + if (status.getIn(['account', 'id']) === me) { + if (publicStatus) { + menu.push({ text: intl.formatMessage(status.get('pinned') ? messages.unpin : messages.pin), action: this.handlePinClick }); + } + + menu.push({ text: intl.formatMessage(messages.delete), action: this.handleDeleteClick }); + } else { + menu.push({ text: intl.formatMessage(messages.mention, { name: status.getIn(['account', 'username']) }), action: this.handleMentionClick }); + menu.push(null); + menu.push({ text: intl.formatMessage(messages.mute, { name: status.getIn(['account', 'username']) }), action: this.handleMuteClick }); + menu.push({ text: intl.formatMessage(messages.block, { name: status.getIn(['account', 'username']) }), action: this.handleBlockClick }); + menu.push({ text: intl.formatMessage(messages.report, { name: status.getIn(['account', 'username']) }), action: this.handleReport }); + } + + if (status.get('in_reply_to_id', null) === null) { + replyIcon = 'reply'; + replyTitle = intl.formatMessage(messages.reply); + } else { + replyIcon = 'reply-all'; + replyTitle = intl.formatMessage(messages.replyAll); + } + + const shareButton = ('share' in navigator) && status.get('visibility') === 'public' && ( + + ); + + return ( +
+ + + + {shareButton} + +
+ +
+ + +
+ ); + } + + } diff --cc app/javascript/glitch/components/status/container.js index 000000000,24261e763..0054abd14 mode 000000,100644..100644 --- a/app/javascript/glitch/components/status/container.js +++ b/app/javascript/glitch/components/status/container.js @@@ -1,0 -1,265 +1,263 @@@ + /* + + `` + =================== + + Original file by @gargron@mastodon.social et al as part of + tootsuite/mastodon. Documentation by @kibi@glitch.social. The code + detecting reblogs has been moved here from . + + */ + + /* * * * */ + + /* + + Imports: + -------- + + */ + + // Package imports // + import React from 'react'; + import { connect } from 'react-redux'; + import { + defineMessages, + injectIntl, + FormattedMessage, + } from 'react-intl'; + + // Mastodon imports // + import { makeGetStatus } from '../../../mastodon/selectors'; + import { + replyCompose, + mentionCompose, + } from '../../../mastodon/actions/compose'; + import { + reblog, + favourite, + unreblog, + unfavourite, + pin, + unpin, + } from '../../../mastodon/actions/interactions'; + import { blockAccount } from '../../../mastodon/actions/accounts'; + import { initMuteModal } from '../../../mastodon/actions/mutes'; + import { + muteStatus, + unmuteStatus, + deleteStatus, + } from '../../../mastodon/actions/statuses'; + import { initReport } from '../../../mastodon/actions/reports'; + import { openModal } from '../../../mastodon/actions/modal'; + + // Our imports // + import Status from '.'; + + /* * * * */ + + /* + + Inital setup: + ------------- + + The `messages` constant is used to define any messages that we will + need in our component. In our case, these are the various confirmation + messages used with statuses. + + */ + + const messages = defineMessages({ + deleteConfirm : { + id : 'confirmations.delete.confirm', + defaultMessage : 'Delete', + }, + deleteMessage : { + id : 'confirmations.delete.message', + defaultMessage : 'Are you sure you want to delete this status?', + }, + blockConfirm : { + id : 'confirmations.block.confirm', + defaultMessage : 'Block', + }, + }); + + /* * * * */ + + /* + + State mapping: + -------------- + + The `mapStateToProps()` function maps various state properties to the + props of our component. We wrap this in a `makeMapStateToProps()` + function to give us closure and preserve `getStatus()` across function + calls. + + */ + + const makeMapStateToProps = () => { + const getStatus = makeGetStatus(); + + const mapStateToProps = (state, ownProps) => { + + let status = getStatus(state, ownProps.id); + + if(status === null) { + console.error(`ERROR! NULL STATUS! ${ownProps.id}`); + // work-around: find first good status + for (let k of state.get('statuses').keys()) { + status = getStatus(state, k); + if (status !== null) break; + } + } + + let reblogStatus = status.get('reblog', null); + let account = undefined; + let prepend = undefined; + + /* + + Here we process reblogs. If our status is a reblog, then we create a + `prependMessage` to pass along to our `` along with the + reblogger's `account`, and set `coreStatus` (the one we will actually + render) to the status which has been reblogged. + + */ + + if (reblogStatus !== null && typeof reblogStatus === 'object') { + account = status.get('account'); + status = reblogStatus; + prepend = 'reblogged_by'; + } + + /* + + Here are the props we pass to ``. + + */ + + return { + status : status, + account : account || ownProps.account, - me : state.getIn(['meta', 'me']), + settings : state.get('local_settings'), + prepend : prepend || ownProps.prepend, + reblogModal : state.getIn(['meta', 'boost_modal']), + deleteModal : state.getIn(['meta', 'delete_modal']), - autoPlayGif : state.getIn(['meta', 'auto_play_gif']), + }; + }; + + return mapStateToProps; + }; + + /* * * * */ + + /* + + Dispatch mapping: + ----------------- + + The `mapDispatchToProps()` function maps dispatches to our store to the + various props of our component. We need to provide dispatches for all + of the things you can do with a status: reply, reblog, favourite, et + cetera. + + For a few of these dispatches, we open up confirmation modals; the rest + just immediately execute their corresponding actions. + + */ + + const mapDispatchToProps = (dispatch, { intl }) => ({ + + onReply (status, router) { + dispatch(replyCompose(status, router)); + }, + + onModalReblog (status) { + dispatch(reblog(status)); + }, + + onReblog (status, e) { + if (status.get('reblogged')) { + dispatch(unreblog(status)); + } else { + if (e.shiftKey || !this.reblogModal) { + this.onModalReblog(status); + } else { + dispatch(openModal('BOOST', { status, onReblog: this.onModalReblog })); + } + } + }, + + onFavourite (status) { + if (status.get('favourited')) { + dispatch(unfavourite(status)); + } else { + dispatch(favourite(status)); + } + }, + + onPin (status) { + if (status.get('pinned')) { + dispatch(unpin(status)); + } else { + dispatch(pin(status)); + } + }, + + onEmbed (status) { + dispatch(openModal('EMBED', { url: status.get('url') })); + }, + + onDelete (status) { + if (!this.deleteModal) { + dispatch(deleteStatus(status.get('id'))); + } else { + dispatch(openModal('CONFIRM', { + message: intl.formatMessage(messages.deleteMessage), + confirm: intl.formatMessage(messages.deleteConfirm), + onConfirm: () => dispatch(deleteStatus(status.get('id'))), + })); + } + }, + + onMention (account, router) { + dispatch(mentionCompose(account, router)); + }, + + onOpenMedia (media, index) { + dispatch(openModal('MEDIA', { media, index })); + }, + + onOpenVideo (media, time) { + dispatch(openModal('VIDEO', { media, time })); + }, + + onBlock (account) { + dispatch(openModal('CONFIRM', { + message: @{account.get('acct')} }} />, + confirm: intl.formatMessage(messages.blockConfirm), + onConfirm: () => dispatch(blockAccount(account.get('id'))), + })); + }, + + onReport (status) { + dispatch(initReport(status.get('account'), status)); + }, + + onMute (account) { + dispatch(initMuteModal(account)); + }, + + onMuteConversation (status) { + if (status.get('muted')) { + dispatch(unmuteStatus(status.get('id'))); + } else { + dispatch(muteStatus(status.get('id'))); + } + }, + }); + + export default injectIntl( + connect(makeMapStateToProps, mapDispatchToProps)(Status) + ); diff --cc app/javascript/glitch/components/status/index.js index 000000000,6bd95b051..33a9730e5 mode 000000,100644..100644 --- a/app/javascript/glitch/components/status/index.js +++ b/app/javascript/glitch/components/status/index.js @@@ -1,0 -1,770 +1,760 @@@ + /* + + `` + ========== + + Original file by @gargron@mastodon.social et al as part of + tootsuite/mastodon. *Heavily* rewritten (and documented!) by + @kibi@glitch.social as a part of glitch-soc/mastodon. The following + features have been added: + + - Better separating the "guts" of statuses from their wrapper(s) + - Collapsing statuses + - Moving images inside of CWs + + A number of aspects of this original file have been split off into + their own components for better maintainance; for these, see: + + - + - + + …And, of course, the other -related components as well. + + */ + + /* * * * */ + + /* + + Imports: + -------- + + */ + + // Package imports // + import React from 'react'; + import PropTypes from 'prop-types'; + import ImmutablePropTypes from 'react-immutable-proptypes'; + import ImmutablePureComponent from 'react-immutable-pure-component'; + + // Mastodon imports // + import scheduleIdleTask from '../../../mastodon/features/ui/util/schedule_idle_task'; ++import { autoPlayGif } from '../../../mastodon/initial_state'; + + // Our imports // + import StatusPrepend from './prepend'; + import StatusHeader from './header'; + import StatusContent from './content'; + import StatusActionBar from './action_bar'; + import StatusGallery from './gallery'; + import StatusPlayer from './player'; + import NotificationOverlayContainer from '../notification/overlay/container'; + + /* * * * */ + + /* + + The `` component: + ------------------------- + + The `` component is a container for statuses. It consists of a + few parts: + + - The ``, which contains tangential information about + the status, such as who reblogged it. + - The ``, which contains the avatar and username of the + status author, as well as a media icon and the "collapse" toggle. + - The ``, which contains the content of the status. + - The ``, which provides actions to be performed + on statuses, like reblogging or sending a reply. + + ### Context + + - __`router` (`PropTypes.object`) :__ + We need to get our router from the surrounding React context. + + ### Props + + - __`id` (`PropTypes.number`) :__ + The id of the status. + + - __`status` (`ImmutablePropTypes.map`) :__ + The status object, straight from the store. + + - __`account` (`ImmutablePropTypes.map`) :__ + Don't be confused by this one! This is **not** the account which + posted the status, but the associated account with any further + action (eg, a reblog or a favourite). + + - __`settings` (`ImmutablePropTypes.map`) :__ + These are our local settings, fetched from our store. We need this + to determine how best to collapse our statuses, among other things. + - - __`me` (`PropTypes.number`) :__ - This is the id of the currently-signed-in user. - + - __`onFavourite`, `onReblog`, `onModalReblog`, `onDelete`, + `onMention`, `onMute`, `onMuteConversation`, onBlock`, `onReport`, + `onOpenMedia`, `onOpenVideo` (`PropTypes.func`) :__ + These are all functions passed through from the + ``. We don't deal with them directly here. + + - __`reblogModal`, `deleteModal` (`PropTypes.bool`) :__ + These tell whether or not the user has modals activated for + reblogging and deleting statuses. They are used by the `onReblog` + and `onDelete` functions, but we don't deal with them here. + - - __`autoPlayGif` (`PropTypes.bool`) :__ - This tells the frontend whether or not to autoplay gifs! - + - __`muted` (`PropTypes.bool`) :__ + This has nothing to do with a user or conversation mute! "Muted" is + what Mastodon internally calls the subdued look of statuses in the + notifications column. This should be `true` for notifications, and + `false` otherwise. + + - __`collapse` (`PropTypes.bool`) :__ + This prop signals a directive from a higher power to (un)collapse + a status. Most of the time it should be `undefined`, in which case + we do nothing. + + - __`prepend` (`PropTypes.string`) :__ + The type of prepend: `'reblogged_by'`, `'reblog'`, or + `'favourite'`. + + - __`withDismiss` (`PropTypes.bool`) :__ + Whether or not the status can be dismissed. Used for notifications. + + - __`intersectionObserverWrapper` (`PropTypes.object`) :__ + This holds our intersection observer. In Mastodon parlance, + an "intersection" is just when the status is viewable onscreen. + + ### State + + - __`isExpanded` :__ + Should be either `true`, `false`, or `null`. The meanings of + these values are as follows: + + - __`true` :__ The status contains a CW and the CW is expanded. + - __`false` :__ The status is collapsed. + - __`null` :__ The status is not collapsed or expanded. + + - __`isIntersecting` :__ + This boolean tells us whether or not the status is currently + onscreen. + + - __`isHidden` :__ + This boolean tells us if the status has been unrendered to save + CPUs. + + */ + + export default class Status extends ImmutablePureComponent { + + static contextTypes = { + router : PropTypes.object, + }; + + static propTypes = { + id : PropTypes.string, + status : ImmutablePropTypes.map, + account : ImmutablePropTypes.map, + settings : ImmutablePropTypes.map, + notification : ImmutablePropTypes.map, - me : PropTypes.string, + onFavourite : PropTypes.func, + onReblog : PropTypes.func, + onModalReblog : PropTypes.func, + onDelete : PropTypes.func, + onPin : PropTypes.func, + onMention : PropTypes.func, + onMute : PropTypes.func, + onMuteConversation : PropTypes.func, + onBlock : PropTypes.func, + onEmbed : PropTypes.func, + onHeightChange : PropTypes.func, + onReport : PropTypes.func, + onOpenMedia : PropTypes.func, + onOpenVideo : PropTypes.func, + reblogModal : PropTypes.bool, + deleteModal : PropTypes.bool, - autoPlayGif : PropTypes.bool, + muted : PropTypes.bool, + collapse : PropTypes.bool, + prepend : PropTypes.string, + withDismiss : PropTypes.bool, + intersectionObserverWrapper : PropTypes.object, + }; + + state = { + isExpanded : null, + isIntersecting : true, + isHidden : false, + markedForDelete : false, + } + + /* + + ### Implementation + + #### `updateOnProps` and `updateOnStates`. + + `updateOnProps` and `updateOnStates` tell the component when to update. + We specify them explicitly because some of our props are dynamically= + generated functions, which would otherwise always trigger an update. + Of course, this means that if we add an important prop, we will need + to remember to specify it here. + + */ + + updateOnProps = [ + 'status', + 'account', + 'settings', + 'prepend', - 'me', + 'boostModal', - 'autoPlayGif', + 'muted', + 'collapse', + 'notification', + ] + + updateOnStates = [ + 'isExpanded', + 'markedForDelete', + ] + + /* + + #### `componentWillReceiveProps()`. + + If our settings have changed to disable collapsed statuses, then we + need to make sure that we uncollapse every one. We do that by watching + for changes to `settings.collapsed.enabled` in + `componentWillReceiveProps()`. + + We also need to watch for changes on the `collapse` prop---if this + changes to anything other than `undefined`, then we need to collapse or + uncollapse our status accordingly. + + */ + + componentWillReceiveProps (nextProps) { + if (!nextProps.settings.getIn(['collapsed', 'enabled'])) { + if (this.state.isExpanded === false) { + this.setExpansion(null); + } + } else if ( + nextProps.collapse !== this.props.collapse && + nextProps.collapse !== undefined + ) this.setExpansion(nextProps.collapse ? false : null); + } + + /* + + #### `componentDidMount()`. + + When mounting, we just check to see if our status should be collapsed, + and collapse it if so. We don't need to worry about whether collapsing + is enabled here, because `setExpansion()` already takes that into + account. + + The cases where a status should be collapsed are: + + - The `collapse` prop has been set to `true` + - The user has decided in local settings to collapse all statuses. + - The user has decided to collapse all notifications ('muted' + statuses). + - The user has decided to collapse long statuses and the status is + over 400px (without media, or 650px with). + - The status is a reply and the user has decided to collapse all + replies. + - The status contains media and the user has decided to collapse all + statuses with media. + + We also start up our intersection observer to monitor our statuses. + `componentMounted` lets us know that everything has been set up + properly and our intersection observer is good to go. + + */ + + componentDidMount () { + const { node, handleIntersection } = this; + const { + status, + settings, + collapse, + muted, + id, + intersectionObserverWrapper, + prepend, + } = this.props; + const autoCollapseSettings = settings.getIn(['collapsed', 'auto']); + + if ( + collapse || + autoCollapseSettings.get('all') || ( + autoCollapseSettings.get('notifications') && muted + ) || ( + autoCollapseSettings.get('lengthy') && + node.clientHeight > ( + status.get('media_attachments').size && !muted ? 650 : 400 + ) + ) || ( + autoCollapseSettings.get('reblogs') && + prepend === 'reblogged_by' + ) || ( + autoCollapseSettings.get('replies') && + status.get('in_reply_to_id', null) !== null + ) || ( + autoCollapseSettings.get('media') && + !(status.get('spoiler_text').length) && + status.get('media_attachments').size + ) + ) this.setExpansion(false); + + if (!intersectionObserverWrapper) return; + else intersectionObserverWrapper.observe( + id, + node, + handleIntersection + ); + + this.componentMounted = true; + } + + /* + + #### `shouldComponentUpdate()`. + + If the status is about to be both offscreen (not intersecting) and + hidden, then we only need to update it if it's not that way currently. + If the status is moving from offscreen to onscreen, then we *have* to + re-render, so that we can unhide the element if necessary. + + If neither of these cases are true, we can leave it up to our + `updateOnProps` and `updateOnStates` arrays. + + */ + + shouldComponentUpdate (nextProps, nextState) { + switch (true) { + case !nextState.isIntersecting && nextState.isHidden: + return this.state.isIntersecting || !this.state.isHidden; + case nextState.isIntersecting && !this.state.isIntersecting: + return true; + default: + return super.shouldComponentUpdate(nextProps, nextState); + } + } + + /* + + #### `componentDidUpdate()`. + + If our component is being rendered for any reason and an update has + triggered, this will save its height. + + This is, frankly, a bit overkill, as the only instance when we + actually *need* to update the height right now should be when the + value of `isExpanded` has changed. But it makes for more readable + code and prevents bugs in the future where the height isn't set + properly after some change. + + */ + + componentDidUpdate () { + if ( + this.state.isIntersecting || !this.state.isHidden + ) this.saveHeight(); + } + + /* + + #### `componentWillUnmount()`. + + If our component is about to unmount, then we'd better unset + `this.componentMounted`. + + */ + + componentWillUnmount () { + this.componentMounted = false; + } + + /* + + #### `handleIntersection()`. + + `handleIntersection()` either hides the status (if it is offscreen) or + unhides it (if it is onscreen). It's called by + `intersectionObserverWrapper.observe()`. + + If our status isn't intersecting, we schedule an idle task (using the + aptly-named `scheduleIdleTask()`) to hide the status at the next + available opportunity. + + tootsuite/mastodon left us with the following enlightening comment + regarding this function: + + > Edge 15 doesn't support isIntersecting, but we can infer it + + It then implements a polyfill (intersectionRect.height > 0) which isn't + actually sufficient. The short answer is, this behaviour isn't really + supported on Edge but we can get kinda close. + + */ + + handleIntersection = (entry) => { + const isIntersecting = ( + typeof entry.isIntersecting === 'boolean' ? + entry.isIntersecting : + entry.intersectionRect.height > 0 + ); + this.setState( + (prevState) => { + if (prevState.isIntersecting && !isIntersecting) { + scheduleIdleTask(this.hideIfNotIntersecting); + } + return { + isIntersecting : isIntersecting, + isHidden : false, + }; + } + ); + } + + /* + + #### `hideIfNotIntersecting()`. + + This function will hide the status if we're still not intersecting. + Hiding the status means that it will just render an empty div instead + of actual content, which saves RAMS and CPUs or some such. + + */ + + hideIfNotIntersecting = () => { + if (!this.componentMounted) return; + this.setState( + (prevState) => ({ isHidden: !prevState.isIntersecting }) + ); + } + + /* + + #### `saveHeight()`. + + `saveHeight()` saves the height of our status so that when whe hide it + we preserve its dimensions. We only want to store our height, though, + if our status has content (otherwise, it would imply that it is + already hidden). + + */ + + saveHeight = () => { + if (this.node && this.node.children.length) { + this.height = this.node.getBoundingClientRect().height; + } + } + + /* + + #### `setExpansion()`. + + `setExpansion()` sets the value of `isExpanded` in our state. It takes + one argument, `value`, which gives the desired value for `isExpanded`. + The default for this argument is `null`. + + `setExpansion()` automatically checks for us whether toot collapsing + is enabled, so we don't have to. + + We use a `switch` statement to simplify our code. + + */ + + setExpansion = (value) => { + switch (true) { + case value === undefined || value === null: + this.setState({ isExpanded: null }); + break; + case !value && this.props.settings.getIn(['collapsed', 'enabled']): + this.setState({ isExpanded: false }); + break; + case !!value: + this.setState({ isExpanded: true }); + break; + } + } + + /* + + #### `handleRef()`. + + `handleRef()` just saves a reference to our status node to `this.node`. + It also saves our height, in case the height of our node has changed. + + */ + + handleRef = (node) => { + this.node = node; + this.saveHeight(); + } + + /* + + #### `parseClick()`. + + `parseClick()` takes a click event and responds appropriately. + If our status is collapsed, then clicking on it should uncollapse it. + If `Shift` is held, then clicking on it should collapse it. + Otherwise, we open the url handed to us in `destination`, if + applicable. + + */ + + parseClick = (e, destination) => { + const { router } = this.context; + const { status } = this.props; + const { isExpanded } = this.state; + if (!router) return; + if (destination === undefined) { + destination = `/statuses/${ + status.getIn(['reblog', 'id'], status.get('id')) + }`; + } + if (e.button === 0) { + if (isExpanded === false) this.setExpansion(null); + else if (e.shiftKey) { + this.setExpansion(false); + document.getSelection().removeAllRanges(); + } else router.history.push(destination); + e.preventDefault(); + } + } + + /* + + #### `render()`. + + `render()` actually puts our element on the screen. The particulars of + this operation are further explained in the code below. + + */ + + render () { + const { + parseClick, + setExpansion, + saveHeight, + handleRef, + } = this; + const { router } = this.context; + const { + status, + account, + settings, + collapsed, + muted, + prepend, + intersectionObserverWrapper, + onOpenVideo, + onOpenMedia, - autoPlayGif, + notification, + ...other + } = this.props; + const { isExpanded, isIntersecting, isHidden } = this.state; + let background = null; + let attachments = null; + let media = null; + let mediaIcon = null; + + /* + + If we don't have a status, then we don't render anything. + + */ + + if (status === null) { + return null; + } + + /* + + If our status is offscreen and hidden, then we render an empty
in + its place. We fill it with "content" but note that opacity is set to 0. + + */ + + if (!isIntersecting && isHidden) { + return ( +
+ { + status.getIn(['account', 'display_name']) || + status.getIn(['account', 'username']) + } + {status.get('content')} +
+ ); + } + + /* + + If user backgrounds for collapsed statuses are enabled, then we + initialize our background accordingly. This will only be rendered if + the status is collapsed. + + */ + + if ( + settings.getIn(['collapsed', 'backgrounds', 'user_backgrounds']) + ) background = status.getIn(['account', 'header']); + + /* + + This handles our media attachments. Note that we don't show media on + muted (notification) statuses. If the media type is unknown, then we + simply ignore it. + + After we have generated our appropriate media element and stored it in + `media`, we snatch the thumbnail to use as our `background` if media + backgrounds for collapsed statuses are enabled. + + */ + + attachments = status.get('media_attachments'); + if (attachments.size && !muted) { + if (attachments.some((item) => item.get('type') === 'unknown')) { + + } else if ( + attachments.getIn([0, 'type']) === 'video' + ) { + media = ( // Media type is 'video' + + ); + mediaIcon = 'video-camera'; + } else { // Media type is 'image' or 'gifv' + media = ( + + ); + mediaIcon = 'picture-o'; + } + + if ( + !status.get('sensitive') && + !(status.get('spoiler_text').length > 0) && + settings.getIn(['collapsed', 'backgrounds', 'preview_images']) + ) background = attachments.getIn([0, 'preview_url']); + } + + /* + + Here we prepare extra data-* attributes for CSS selectors. + Users can use those for theming, hiding avatars etc via UserStyle + + */ + + const selectorAttribs = { + 'data-status-by': `@${status.getIn(['account', 'acct'])}`, + }; + + if (prepend && account) { + const notifKind = { + favourite: 'favourited', + reblog: 'boosted', + reblogged_by: 'boosted', + }[prepend]; + + selectorAttribs[`data-${notifKind}-by`] = `@${account.get('acct')}`; + } + + /* + + Finally, we can render our status. We just put the pieces together + from above. We only render the action bar if the status isn't + collapsed. + + */ + + return ( +
+ {prepend && account ? ( + + ) : null} + + + {isExpanded !== false ? ( + + ) : null} + {notification ? ( + + ) : null} +
+ ); + + } + + } diff --cc app/javascript/mastodon/components/icon_button.js index 06f53841d,76b0da12f..d0c1b049f --- a/app/javascript/mastodon/components/icon_button.js +++ b/app/javascript/mastodon/components/icon_button.js @@@ -59,6 -65,6 +65,7 @@@ export default class IconButton extend expanded, icon, inverted, ++ flip, overlay, pressed, tabIndex, @@@ -72,27 -78,23 +79,42 @@@ overlayed: overlay, }); - const flipDeg = this.props.flip ? -180 : -360; - const rotateDeg = this.props.active ? flipDeg : 0; ++ const flipDeg = flip ? -180 : -360; ++ const rotateDeg = active ? flipDeg : 0; + + const motionDefaultStyle = { + rotate: rotateDeg, + }; + + const springOpts = { + stiffness: this.props.flip ? 60 : 120, + damping: 7, + }; + const motionStyle = { + rotate: animate ? spring(rotateDeg, springOpts) : 0, + }; + + if (!animate) { + // Perf optimization: avoid unnecessary components unless + // we actually need to animate. + return ( +