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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants