LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Control tags (#7433)

Browse source 
* Add prefix to all internal tags

* Strip internal tag prefix from form labels

Co-authored-by: Dan Peterson <dan@dan-peterson.ca>
This commit is contained in:
jessopb 2022-01-21 20:43:45 -05:00 committed by GitHub
parent f06b3bd877
commit c1a54f9707
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 19 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
ui/component/tagsSearch/view.jsx
View file

@ -5,7 +5,7 @@ import Tag from 'component/tag';
import { setUnion, setDifference } from 'util/set-operations';
import I18nMessage from 'component/i18nMessage';
import analytics from 'analytics';
import { UTILITY_TAGS } from 'constants/tags';
import { CONTROL_TAGS, INTERNAL_TAGS, INTERNAL_TAG_PREFIX } from 'constants/tags';
type Props = {
tagsPassedIn: Array<Tag>,
@ -79,7 +79,7 @@ export default function TagsSearch(props: Props) {
const remainingUnfollowedTagsSet = setDifference(unfollowedTagsSet, selectedTagsSet);
const suggestedTagsSet = setUnion(remainingFollowedTagsSet, remainingUnfollowedTagsSet);
const SPECIAL_TAGS = [...UTILITY_TAGS, 'lbry-first', 'mature'];
const SPECIAL_TAGS = [...INTERNAL_TAGS, 'mature'];
let countWithoutSpecialTags = selectedTagsSet.size;
SPECIAL_TAGS.forEach((t) => {
@ -179,7 +179,7 @@ export default function TagsSearch(props: Props) {
{countWithoutSpecialTags === 0 && <Tag key={`placeholder-tag`} name={'example'} disabled type={'remove'} />}
{Boolean(tagsPassedIn.length) &&
tagsPassedIn
.filter((t) => !UTILITY_TAGS.includes(t.name))
.filter((t) => !INTERNAL_TAGS.includes(t.name))
.map((tag) => (
<Tag
key={`passed${tag.name}`}
@ -234,7 +234,7 @@ export default function TagsSearch(props: Props) {
onSelect && ( // onSelect ensures this does not appear on TagFollow
<fieldset-section>
<label>{__('Control Tags')}</label>
{UTILITY_TAGS.map((t) => (
{CONTROL_TAGS.map((t) => (
<FormField
key={t}
name={t}
@ -242,6 +242,7 @@ export default function TagsSearch(props: Props) {
blockWrap={false}
label={__(
t
.replace(INTERNAL_TAG_PREFIX, '')
.split('-')
.map((word) => word.charAt(0).toUpperCase() + word.slice(1))
.join(' ')

18
ui/constants/tags.js
View file

@ -13,11 +13,21 @@ export const DEFAULT_FOLLOWED_TAGS = [
'technology',
];
export const DISABLE_COMMENTS_TAG = 'disable-comments';
export const DISABLE_SUPPORT_TAG = 'disable-support';
export const PREFERENCE_EMBED = 'preference-embed';
export const INTERNAL_TAG_PREFIX = 'c:';
export const UTILITY_TAGS = [DISABLE_COMMENTS_TAG, DISABLE_SUPPORT_TAG, PREFERENCE_EMBED];
export const DISABLE_COMMENTS_TAG = 'c:disable-comments';
export const DISABLE_SUPPORT_TAG = 'disable-support';
export const PREFERENCE_EMBED = 'c:preference-embed';
export const SCHEDULED_LIVESTREAM_TAG = 'c:scheduled-livestream';
export const LBRY_FIRST_TAG = 'c:lbry-first';
// Control tags are special tags that are available to the user in some situations.
export const CONTROL_TAGS = [DISABLE_COMMENTS_TAG, DISABLE_SUPPORT_TAG, PREFERENCE_EMBED];
// System tags are special tags that are not available to the user.
export const SYSTEM_TAGS = [SCHEDULED_LIVESTREAM_TAG, LBRY_FIRST_TAG];
export const INTERNAL_TAGS = [...CONTROL_TAGS, ...SYSTEM_TAGS];
export const MATURE_TAGS = [
'porn',