Page MenuHomePhabricator

Inline comments say they are "Not Visible" when they are.
Closed, ResolvedPublic

Description

To reproduce:

  • Start by viewing the "changes since last review" for some revision that has gone through a couple diffs. That is, be looking at the difference between some middle diff, and the last diff.
  • Add some inline comments. Don't submit them.
  • Switch to viewing all changes. That is, from the base, to the last diff.

Observed:

In the preview area, the inline comment preview says the inlines are "Not Visible":

{F151675}

Even though they totally are visible:

{F151677}

Clicking on "Not Visible" does nothing.

Interestingly, if you add another inline comment at this point, the problem goes away.

Expected:

"Not Visible" should say "View". Clicking it should scroll the view to the inline comment.

Revisions and Commits

Restricted Differential Revision

Event Timeline

• bitglue raised the priority of this task from to Needs Triage.
• bitglue updated the task description. (Show Details)
• bitglue added a project: Differential.
• bitglue added a subscriber: • bitglue.
epriestley added a revision: Restricted Differential Revision.Mar 9 2015, 10:59 PM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.Mar 10 2015, 12:27 AM
epriestley added a commit: Restricted Diffusion Commit.