Page MenuHomePhabricator

Impossible to submit "Done" inline comment state without adding a comment
Closed, ResolvedPublic


Steps to reproduce:

  1. open a commit (differential revision) you've made, where somebody else have added an inline comment
  2. check "Done" checkbox on that inline comment
  3. press "Submit" button with "Add Comment" action selection without entering any comment text

Right now nothing happens because comment text is empty. Instead the transaction for marking inline comments Done/Undone should be created regardless of:

  • selected action (Add Comment, Concern Raised, etc.)
  • presence/absence of comment text.

Event Timeline

aik099 raised the priority of this task from to Needs Triage.
aik099 updated the task description. (Show Details)
aik099 added projects: Differential, Audit.
aik099 added a subscriber: aik099.

Come across this issue and thought the functionality was broken. Should be able to mark done without any comment required.

angie added a project: Restricted Project.Jun 16 2015, 9:18 PM
angie added subscribers: angie, jhurwitz.

At HEAD, I get this:

Screen Shot 2015-09-09 at 5.29.29 PM.png (190×547 px, 21 KB)

Clicking "Apply Remaining Actions" marks the inline as done. This behavior isn't great, but also isn't "Impossible". Am I doing something wrong (or, uh, not doing something wrong, I guess?)

angie claimed this task.

Can confirm this now works for me.

angie moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board.Sep 10 2015, 5:03 PM

We should probably improve this behavior at some point, but it isn't entirely trivial, I'll likely do concurrently with T9132 anyway, and the current behavior is a little rough but doesn't seem too bad (it's inconsistent and needlessly indirect, but not technically incorrect).