Skip to content

Add more then one input placeholder in the compose view#867

Open
sahil-sahu-dev wants to merge 5 commits into
mastodon:developfrom
sahil-sahu-dev:mastodon#416
Open

Add more then one input placeholder in the compose view#867
sahil-sahu-dev wants to merge 5 commits into
mastodon:developfrom
sahil-sahu-dev:mastodon#416

Conversation

@sahil-sahu-dev
Copy link
Copy Markdown

-> Added two placeholder texts-
1. Express what's on your mind
2. Tell us what you think

-> Changed the compose view to randomly pick one of the three placeholder strings.
-> Added string localizations for all supported languages

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 7, 2023

CLA assistant check
All committers have signed the CLA.

@sahil-sahu-dev sahil-sahu-dev changed the title Mastodon #416 Add more then one input placeholder in the compose view Jan 7, 2023
Copy link
Copy Markdown
Contributor

@zeitschlag zeitschlag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your work @sahil-sahu-dev 🥳 It feels like having a nice, little easter-egg in the app.

I added some comments, please have a look at them 🙂 Can you also please take care of the merge-conflicts? Thank you!

Comment thread Mastodon.xcodeproj/xcshareddata/xcschemes/Mastodon.xcscheme Outdated
Comment thread MastodonSDK/Sources/MastodonUI/Scene/ComposeContent/View/ComposeContentView.swift Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants