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

Move atrribution option from editor screen to settings #303

Closed
wants to merge 4 commits into from

Conversation

spuday90
Copy link
Collaborator

@spuday90 spuday90 commented Jun 2, 2021

closes #301

@spuday90 spuday90 requested a review from adeekshith June 2, 2021 05:07
@adeekshith adeekshith changed the title Issue/301/add attrribution option to settings Move atrribution option from editor screen to settings Jun 3, 2021
@adeekshith
Copy link
Owner

  • Resolved conflicts.
  • Make Attribution ON by default

@spuday90
Copy link
Collaborator Author

spuday90 commented Jun 4, 2021

* [x]  Resolved conflicts.

* [ ]  Make Attribution ON by default

For fresh install it is working fine

@adeekshith
Copy link
Owner

adeekshith commented Jun 6, 2021

* [x]  Resolved conflicts.

* [ ]  Make Attribution ON by default

For fresh install it is working fine

Checked it on the other PR #305 which has these changes and it is not ON by default

@hegocre
Copy link
Collaborator

hegocre commented Jun 8, 2021

I see you removed the Append Watomatic attribution to help spread the word checkbox from CustomReplyEditorActivity, but as @spuday90 mentioned #301 (comment), I think that's the obvious place it should be. I suggest we leave it while also adding the switch on settings. Maybe moving the checkbox above the SAVE button will make it easier to find.

@adeekshith
Copy link
Owner

adeekshith commented Jun 8, 2021

I see you removed the Append Watomatic attribution to help spread the word checkbox from CustomReplyEditorActivity, but as @spuday90 mentioned #301 (comment), I think that's the obvious place it should be. I suggest we leave it while also adding the switch on settings. Maybe moving the checkbox above the SAVE button will make it easier to find.

  1. I thought it is the right place too but not many are finding it there. Somehow many are expecting it in the settings. Also for some, it gets hidden below the keyboard esp for small screens or when text is long.
  2. Moving it above "SAVE" will make it easy to find but it can hide the Save button below the keyboard which is not good.
  3. In fact, was thinking of moving the save button also to the action bar (top right) in the future so we can free up some space in the editor activity for reply rules feature (containing filters like if incoming has keywords, etc).
  4. Let us show one setting only at one place unless if it is critical.

But for the sake of transparency, we can show a preview of how it looks while saving the text and perhaps include the checkbox in the preview.

Also, guiding them to the setting in editor screen also is not easy as it requires multiple screenshots and lots of explanation like asking them to lower the keyboard if not visible, etc. Just asking them to check in the settings is easy. Lets see how users take this change and go forward accordingly.

I think this is one of those where we feel it is obvious once we already know where it is but rest of the folks are having trouble finding it.

One of many messages where users feel "Settings" is the right place for this:

some user: I realise now that the Automatic Watomatic attribution option is "buried" under the Auto reply text option. Would be nice if it was shown in settings.

@hegocre
Copy link
Collaborator

hegocre commented Jun 8, 2021

I think this is one of those where we feel it is obvious once we already know where it is but rest of the folks are having trouble finding it.

Yeah, it could be that!

But for the sake of transparency, we can show a preview of how it looks while saving the text and perhaps include the checkbox in the preview.

I think this will be very important in case we finally remove the checkbox. Some users might not check settings and simply enable the app from the main activity, and then find the attribution text is being added. I think that would be very confusing.

@adeekshith
Copy link
Owner

I think this will be very important in case we finally remove the checkbox. Some users might not check settings and simply enable the app from the main activity, and then find the attribution text is being added. I think that would be very confusing.

Okay, lets leave the checkbox as-is until we figure out a better way 😊

@spuday90
Copy link
Collaborator Author

* [x]  Resolved conflicts.

* [ ]  Make Attribution ON by default

For fresh install it is working fine

Checked it on the other PR #305 which has these changes and it is not ON by default

@adeekshith this is happening because on our devices as we have app installed from store its not considering as first install. Please check on other device.

@adeekshith
Copy link
Owner

@adeekshith this is happening because on our devices as we have app installed from store its not considering as first install. Please check on other device.

Seem to be working on the other PR but may need some more UI improvements before merging this. We can keep the branch but lets close this PR until we figure out the UI 😊

@adeekshith adeekshith closed this Jun 16, 2021
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.

Move "Add attribution" option to "Settings"
3 participants