wordpress-mobile / wordpress-mobile/WordPress-Android
[Post Settings] Use screen to edit String fields instead of modal dialog
Nobody has claimed this yet.
- Dominant language
- Kotlin
- Stars
- 3.2k
- Forks
- 1.4k
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 69
Description
After implementation of the separate full-screen Activity to edit the Share Message field of the Jetpack Social Sharing section, @osullivanchris suggested also reusing that for other fields, such as slug and excerpt which are also fields containing a String defined by the user, instead of the modal dialog we use now.
slug / excerpt dialog |
share message screen |
|---|---|
The Activity (EditJetpackSocialShareMessageActivity) currently has specific logic inside it so reusing it will require a bit of work for proper parametrization of custom needs of each field (such as max length and specific rules).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with EditJetpackSocialShareMessageActivity and review its current field-specific logic. Then locate the existing slug and excerpt modal-dialog entry points and compare their custom requirements, including maximum length and validation rules. Done means both fields use the full-screen editing flow while preserving their field-specific behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- kotlin
- Domain
- mobile
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100