mirror of
https://github.com/mastodon/mastodon.git
synced 2025-05-14 13:41:15 +00:00
Fix filters not applying in detailed view (#34259)
This commit is contained in:
parent
653868bb0c
commit
d6442b5455
|
@ -15,6 +15,7 @@ import AlternateEmailIcon from '@/material-icons/400-24px/alternate_email.svg?re
|
||||||
import { AnimatedNumber } from 'mastodon/components/animated_number';
|
import { AnimatedNumber } from 'mastodon/components/animated_number';
|
||||||
import { ContentWarning } from 'mastodon/components/content_warning';
|
import { ContentWarning } from 'mastodon/components/content_warning';
|
||||||
import EditedTimestamp from 'mastodon/components/edited_timestamp';
|
import EditedTimestamp from 'mastodon/components/edited_timestamp';
|
||||||
|
import { FilterWarning } from 'mastodon/components/filter_warning';
|
||||||
import type { StatusLike } from 'mastodon/components/hashtag_bar';
|
import type { StatusLike } from 'mastodon/components/hashtag_bar';
|
||||||
import { getHashtagBarForStatus } from 'mastodon/components/hashtag_bar';
|
import { getHashtagBarForStatus } from 'mastodon/components/hashtag_bar';
|
||||||
import { Icon } from 'mastodon/components/icon';
|
import { Icon } from 'mastodon/components/icon';
|
||||||
|
@ -68,6 +69,7 @@ export const DetailedStatus: React.FC<{
|
||||||
}) => {
|
}) => {
|
||||||
const properStatus = status?.get('reblog') ?? status;
|
const properStatus = status?.get('reblog') ?? status;
|
||||||
const [height, setHeight] = useState(0);
|
const [height, setHeight] = useState(0);
|
||||||
|
const [showDespiteFilter, setShowDespiteFilter] = useState(false);
|
||||||
const nodeRef = useRef<HTMLDivElement>();
|
const nodeRef = useRef<HTMLDivElement>();
|
||||||
|
|
||||||
const handleOpenVideo = useCallback(
|
const handleOpenVideo = useCallback(
|
||||||
|
@ -80,6 +82,10 @@ export const DetailedStatus: React.FC<{
|
||||||
[onOpenVideo, status],
|
[onOpenVideo, status],
|
||||||
);
|
);
|
||||||
|
|
||||||
|
const handleFilterToggle = useCallback(() => {
|
||||||
|
setShowDespiteFilter(!showDespiteFilter);
|
||||||
|
}, [showDespiteFilter, setShowDespiteFilter]);
|
||||||
|
|
||||||
const handleExpandedToggle = useCallback(() => {
|
const handleExpandedToggle = useCallback(() => {
|
||||||
if (onToggleHidden) onToggleHidden(status);
|
if (onToggleHidden) onToggleHidden(status);
|
||||||
}, [onToggleHidden, status]);
|
}, [onToggleHidden, status]);
|
||||||
|
@ -290,8 +296,12 @@ export const DetailedStatus: React.FC<{
|
||||||
const { statusContentProps, hashtagBar } = getHashtagBarForStatus(
|
const { statusContentProps, hashtagBar } = getHashtagBarForStatus(
|
||||||
status as StatusLike,
|
status as StatusLike,
|
||||||
);
|
);
|
||||||
|
|
||||||
|
const matchedFilters = status.get('matched_filters');
|
||||||
|
|
||||||
const expanded =
|
const expanded =
|
||||||
!status.get('hidden') || status.get('spoiler_text').length === 0;
|
(!matchedFilters || showDespiteFilter) &&
|
||||||
|
(!status.get('hidden') || status.get('spoiler_text').length === 0);
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div style={outerStyle}>
|
<div style={outerStyle}>
|
||||||
|
@ -328,17 +338,26 @@ export const DetailedStatus: React.FC<{
|
||||||
)}
|
)}
|
||||||
</Link>
|
</Link>
|
||||||
|
|
||||||
{status.get('spoiler_text').length > 0 && (
|
{matchedFilters && (
|
||||||
<ContentWarning
|
<FilterWarning
|
||||||
text={
|
title={matchedFilters.join(', ')}
|
||||||
status.getIn(['translation', 'spoilerHtml']) ||
|
expanded={showDespiteFilter}
|
||||||
status.get('spoilerHtml')
|
onClick={handleFilterToggle}
|
||||||
}
|
|
||||||
expanded={expanded}
|
|
||||||
onClick={handleExpandedToggle}
|
|
||||||
/>
|
/>
|
||||||
)}
|
)}
|
||||||
|
|
||||||
|
{status.get('spoiler_text').length > 0 &&
|
||||||
|
(!matchedFilters || showDespiteFilter) && (
|
||||||
|
<ContentWarning
|
||||||
|
text={
|
||||||
|
status.getIn(['translation', 'spoilerHtml']) ||
|
||||||
|
status.get('spoilerHtml')
|
||||||
|
}
|
||||||
|
expanded={expanded}
|
||||||
|
onClick={handleExpandedToggle}
|
||||||
|
/>
|
||||||
|
)}
|
||||||
|
|
||||||
{expanded && (
|
{expanded && (
|
||||||
<>
|
<>
|
||||||
<StatusContent
|
<StatusContent
|
||||||
|
|
|
@ -138,7 +138,7 @@ const makeMapStateToProps = () => {
|
||||||
});
|
});
|
||||||
|
|
||||||
const mapStateToProps = (state, props) => {
|
const mapStateToProps = (state, props) => {
|
||||||
const status = getStatus(state, { id: props.params.statusId });
|
const status = getStatus(state, { id: props.params.statusId, contextType: 'detailed' });
|
||||||
|
|
||||||
let ancestorsIds = Immutable.List();
|
let ancestorsIds = Immutable.List();
|
||||||
let descendantsIds = Immutable.List();
|
let descendantsIds = Immutable.List();
|
||||||
|
|
|
@ -15,9 +15,10 @@ export const makeGetStatus = () => {
|
||||||
(state, { id }) => state.getIn(['accounts', state.getIn(['statuses', id, 'account'])]),
|
(state, { id }) => state.getIn(['accounts', state.getIn(['statuses', id, 'account'])]),
|
||||||
(state, { id }) => state.getIn(['accounts', state.getIn(['statuses', state.getIn(['statuses', id, 'reblog']), 'account'])]),
|
(state, { id }) => state.getIn(['accounts', state.getIn(['statuses', state.getIn(['statuses', id, 'reblog']), 'account'])]),
|
||||||
getFilters,
|
getFilters,
|
||||||
|
(_, { contextType }) => contextType === 'detailed',
|
||||||
],
|
],
|
||||||
|
|
||||||
(statusBase, statusReblog, accountBase, accountReblog, filters) => {
|
(statusBase, statusReblog, accountBase, accountReblog, filters, warnInsteadOfHide) => {
|
||||||
if (!statusBase || statusBase.get('isLoading')) {
|
if (!statusBase || statusBase.get('isLoading')) {
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
@ -31,7 +32,7 @@ export const makeGetStatus = () => {
|
||||||
let filtered = false;
|
let filtered = false;
|
||||||
if ((accountReblog || accountBase).get('id') !== me && filters) {
|
if ((accountReblog || accountBase).get('id') !== me && filters) {
|
||||||
let filterResults = statusReblog?.get('filtered') || statusBase.get('filtered') || ImmutableList();
|
let filterResults = statusReblog?.get('filtered') || statusBase.get('filtered') || ImmutableList();
|
||||||
if (filterResults.some((result) => filters.getIn([result.get('filter'), 'filter_action']) === 'hide')) {
|
if (!warnInsteadOfHide && filterResults.some((result) => filters.getIn([result.get('filter'), 'filter_action']) === 'hide')) {
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
filterResults = filterResults.filter(result => filters.has(result.get('filter')));
|
filterResults = filterResults.filter(result => filters.has(result.get('filter')));
|
||||||
|
|
|
@ -6,6 +6,8 @@ export const toServerSideType = (columnType: string) => {
|
||||||
case 'thread':
|
case 'thread':
|
||||||
case 'account':
|
case 'account':
|
||||||
return columnType;
|
return columnType;
|
||||||
|
case 'detailed':
|
||||||
|
return 'thread';
|
||||||
default:
|
default:
|
||||||
if (columnType.includes('list:')) {
|
if (columnType.includes('list:')) {
|
||||||
return 'home';
|
return 'home';
|
||||||
|
|
Loading…
Reference in New Issue
Block a user