Comments: missing operation in progress indicator

[folletto]

This is not technically a bug, but experience wise might lead to experience the app as buggy:

1. Get on a real or simulated slow connection
2. Write a comment
3. Send
4. Wait
5. Wait
6. Is this thing working?
7. Wait
8. Sent

We should add some kind of indication that the comment is being sent. We need a better design, but a quick fix (hopefully?) could be to have a different view on the "You replied to this comment" line:

• A spinner instead of the icon
• "Sending comment..." as text

A better design would be to optimistically post the comment, and then just notify if it failed — but to work well that component should actually show the comment, so it would require a deeper redesign.

[stale]

This issue has been marked as stale and will be automatically closed. This happened because:

• It has been inactive for the past year.
• It isn't in a project or a milestone.
• It hasn’t been labeled `[Pri] Blocker`, `[Pri] High`, or `good first issue`.

However, discussion is still welcome! If the issue is still valid, please leave a comment with a brief explanation so the issue can be reopened.

[folletto]

I think it's still an important thing to add to make the experience more fluid.

[stale]

This issue has been marked as stale because:

• It has been inactive for the past year.
• It isn't in a project or a milestone.
• It hasn’t been labeled [Pri] Blocker, [Pri] High, or good first issue.

Please comment with an update if you believe this issue is still valid or if it can be closed. This issue will also be reviewed for validity and priority (cc @designsimply).

[designsimply]

Tested and confirmed that no indicator is shown when replying to comments from the notifications screen. (28s)

^{Tested with WPiOS 13.8.0.0 TestFlight beta on iPhone 6S iOS 13.2.3.}

@mattmiklic can Mobile Design Groundskeeping give us a design solution here, or some direction if there's already a design elsewhere that can just be implemented in this screen?

[aerych]

Confirmed this is still an area for improvement.

[stale]

This issue has been marked as stale because:

• It has been inactive for the past year.
• It isn't in a project or a milestone.
• It hasn’t been labeled [Pri] Blocker, [Pri] High, or good first issue.

Please comment with an update if you believe this issue is still valid or if it can be closed. This issue will also be reviewed for validity and priority during regularly scheduled triage sessions.

[dangermattic]

Thanks for reporting! 👍