Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Copy button for GoalInfoScreen notes card #95

Merged
merged 9 commits into from
Apr 7, 2024

Conversation

mrghosti3
Copy link
Contributor

Adds a button for copying text from Note card in goal info screen.

Closes #94

Simple UI feature to copy notes for goal into a clipboard. Button can be
found after expanding the note card in the 'GoalInfoScreen'.
Displays taost message feedback if device API is 32 or below
(before Android 13)
@mrghosti3
Copy link
Contributor Author

May need extra edits for translations but otherwise think the main functionality is complete.

Copy link
Member

@starry-shivam starry-shivam left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Left a few comments.

- Renames current new string resources to match previous ones in Goal
  Info Screen;
- Adds another new resource for Icon description.
Copy link
Member

@starry-shivam starry-shivam left a comment

Choose a reason for hiding this comment

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

Thanks for addressing the review, just a couple more nitpicks.

@starry-shivam
Copy link
Member

Alright! Things look pretty good to me now. Please clarify what you want to do about translations.

@starry-shivam
Copy link
Member

Nvm, Got your response in the above comment.

@starry-shivam starry-shivam merged commit 0bcac27 into Pool-Of-Tears:main Apr 7, 2024
1 check passed
@starry-shivam
Copy link
Member

Merged! Thanks again for contributing 🎉

@mrghosti3
Copy link
Contributor Author

Thanks for accepting my contribution 🥳

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.

[Feature Request] Copy text in goal notes with long press
2 participants