Add opt-in feature to anonymize tweets #173
Conversation
source/features/anonymize-tweets.js
Outdated
@@ -0,0 +1,7 @@ | |||
export default function () { | |||
$('.FullNameGroup .fullname').text('Jane Doe'); | |||
$('.js-retweet-text b').text('Jane Doe'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about Math.random() >= 0.5 ? 'Jane Doe' : 'John Doe'
? That way we stay totally neutral.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For sure, I changed it
Cool idea :) Can you also add it to the readme? |
da5d9a3
to
3f19293
Compare
source/features/index.js
Outdated
anonymizeTweets: { | ||
id: 'feature-anonymous-tweets', | ||
category: 'timeline', | ||
label: 'Transform your timeline in Jane Doe tweets timeline', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs a better description. More like the one in the readme.
@@ -42,6 +42,7 @@ We use Twitter a lot and notice many dumb annoyances we'd like to fix. So here b | |||
- Uses your personal color theme on all profiles. | |||
- Hides the header image on profile pages. | |||
- [Shows alternative image text below images when available.](https://user-images.githubusercontent.com/170270/40556400-b46c292c-6076-11e8-8241-f5c4e1a7a161.png) | |||
- Anonymizes tweets by replacing real usernames with "John Doe" or "Jane Doe" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Anonymizes tweets by replacing real usernames with "John Doe" or "Jane Doe" | |
- Anonymizes tweets by replacing real usernames and names with `John Doe` or `Jane Doe`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This made be realize, it shouldn't replace usernames with John Doe
. It should rather replace those with @johndoe
or @janedoe
. Right?
anonymizeTweets: { | ||
id: 'feature-anonymize-tweets', | ||
category: 'timeline', | ||
label: 'Anonymize tweets by replacing real usernames with "John Doe" or "Jane Doe" to focus on tweets\' content', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as the readme description comment.
#173 (comment) still needs to be fixed. |
@iGitScor Bump :) |
Closing for lack of response. Still happy to land this if you find time to finish it ;) |
Feature proposal: Anonymous tweets to encourage content