Bluesky app fork with some witchin' additions 💫 witchsky.app
bluesky fork client
122
fork

Configure Feed

Select the types of activity you want to include in your feed.

change pet account label description #27

open opened by 1f98a.net targeting main

Previously, the description that appeared when clicking the pet account icon read:

This account has been marked as a pet by its owner.

This is of course intended to match the automated account description (This account has been marked as automated by its owner.), which makes sense as 'automated' is a adjective, whereas 'pet' is a noun with pets usually having the property of 'having an owner', and can be slightly ambiguous as "it's owner" initially read to me as 'the owner of the pet' not the intended meaning of the 'the owner of the account', This patch changes the label to instead be

This account owner has marked this account as a pet.

Which makes it more clear that the account owner did the action to the account. Technically it's still not fully correct as the account itsself isn't the pet in question but it's clearer, especially if the account owner wants it to make very clear that the 'pet' tag may not have been applied by the owner of the pet itsself, if the pet even has one.

Labels

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:6vaoic6nysnlxow5mh265i4n/sh.tangled.repo.pull/3mlefhq7g5k22
+1 -1
Diff #0
+1 -1
src/components/PetAccountAlert.tsx
··· 24 24 const isSelf = profile.did === currentAccount?.did 25 25 const description = isSelf 26 26 ? l`You have marked this account as a pet. You can remove this label at any time from your account settings.` 27 - : l`This account has been marked as a pet by its owner.` 27 + : l`This account owner has marked this account as a pet.` 28 28 29 29 return ( 30 30 <Dialog.Outer control={control} nativeOptions={{preventExpansion: true}}>

History

1 round 0 comments
sign up or login to add to the discussion
1f98a.net submitted #0
1 commit
expand
85c684dd
change pet account label description
no conflicts, ready to merge
expand 0 comments