communicate to user when analysis gives up [+2]

[pq]

For the second time in as many days folks have been hit by this issue (and pinged me about it).

Net-net: if we punt on analyzing an opened folder we need to tell the user that we've done so (and point them to the remedy).

Assigning to Luke for UX thoughts.

[alan-knight]

I'm one of the ones who got hit. Basically, I noticed that when I added the larger folder it was grey, but didn't have any idea why, or how to find out. Everything seemed to work at the time, so I ignored it, and didn't notice that things weren't being syntax highlighted. I'm not sure what would be a better mechanism.

Perhaps one thing that might be useful is if a warning was added to the list of problems, saying that a particular file, or directory, was not being analyzed, so problems could not be listed. That might also get lost, but when it was noticed at least would be concrete and could say what to do.

[stevemessick]

Let's talk about this. I'm sure we can create a reasonable UX.

---

cc @clayberg.
Set owner to @pq.

[clayberg]

All search results views (Search & Callers) should show a prominent warning to the user that the results are based on a partial analysis (e.g., incomplete and [potentially wrong!) and provide a link to a preference where the user could increase the analysis threshold. This came up today when Lars was showing me the editor and none of the search options were showing the correct results.

---

cc @danrubel.
Changed the title to: "communicate to user when analysis gives up [+2]".

[clayberg]

This is related to Issue #4387/

[lukechurch]

Lets talk at our Friday meeting

[danrubel]

Is this fixed now that we have the open folder action showing analysis progress to the user?

[pq]

This deserves more refinement but it's not M1-blocking. Moving to M2.

---

Removed this from the M1 milestone.
Added this to the M2 milestone.

[pq]

I think this is no longer an issue. Feel free to close unless you feel otherwise.

---

Set owner to @danrubel.

[danrubel]

No longer an issue now that we display progress to the user, and continue to analyze until the user tells us otherwise by canceling.

---

Removed this from the M2 milestone.
Added this to the M1 milestone.
Added Fixed label.