Do not hide notifications without prompt when window has no focus

[SteveBenz]

Issue Type: Bug

If an extension posts a notification while VSCode is, say, minimized, the user has to actively search for the notification in order to see it.

While the 15-second timer on notifications is generally a good thing, the 15 seconds should only tick while VSCode is in-focus.

Here are some examples of cases where notifications might pop up while VSCode is not focused:

1. An extension that's monitoring for changes to the branch that the user is coding on.
2. An extension that monitors for work items or pull requests assigned to the user.
3. An extension that checks for updates for itself.

VS Code version: Code 1.37.1 (f06011a, 2019-08-15T16:17:55.855Z)
OS version: Windows_NT x64 10.0.18975

System Info

Item	Value
CPUs	Intel(R) Xeon(R) CPU E5-1650 v3 @ 3.50GHz (12 x 3492)
GPU Status	2d_canvas: enabled flash_3d: enabled flash_stage3d: enabled flash_stage3d_baseline: enabled gpu_compositing: enabled multiple_raster_threads: enabled_on native_gpu_memory_buffers: disabled_software oop_rasterization: disabled_off protected_video_decode: unavailable_off rasterization: enabled skia_deferred_display_list: disabled_off skia_renderer: disabled_off surface_synchronization: enabled_on video_decode: enabled viz_display_compositor: disabled_off webgl: enabled webgl2: enabled
Load (avg)	undefined
Memory (System)	15.92GB (10.38GB free)
Process Argv	.
Screen Reader	no
VM	0%

Extensions (4)

Extension	Author (truncated)	Version
tslint	eg2	1.0.44
vscode-nmake-tools	Mic	2.0.190702001
cpptools	ms-	0.25.1
quokka-vscode	Wal	1.0.240

[vscodebot]

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

• update clear all notification action status (#61983)
• add the ability to silent notifications for vscode 😴 (#78671)

[bpasero]

Actually that should be the case given this line:

vscode/src/vs/workbench/browser/parts/notifications/notificationsToasts.ts

Line 244 in 109add7

if ((item.sticky || item.hasPrompt()) && !this.windowService.hasFocus) {

Can you demonstrate where this does not work? A notification should not start to hide automatically if the window does not have focus.

[SteveBenz]

Thanks for having a look @bpasero ! That line might save us if the notification were sticky or prompting. For for stuff like what I'm talking about, you wouldn't want a notification to be that heavyweight - you'd just want it to tell you what it's got to tell you and disappear. For example, you go to a meeting and come back to your desk and re-open VSCode - you want that notification that the branch head moved half an hour ago to appear, get its 15 seconds of screen-time, and disappear.

To be sure I'm clear, here are some repro-steps:

1. Create a trivial extension with a single command, whose implementation is:

   setTimeout(() => vscode.window.showInformationMessage("Zow!"), 5000);

2. Run the command - observe that it pops up the notification after 5 seconds, and the notification dies after 15 seconds. All is well.

3. Run the command, and immediately minimize VSCode, get coffee. When you come back and re-open VSCode - observe that there's no notification to be seen, but if you dig into your notifications, you can see it.

4. Run the command, alt-tab to a web browser. Again the notification pops up, and pops down after 15 seconds even though VSCode was not in focus.

[bpasero]

Got it. It looks like we did an explicit decision (in 29ab698) to only keep notifications around that ask the user for input, even if the window does not have focus. I think we can leave this open as feature request, but I am not sure if we want to change this in the near term. We are sensitive to not showing too many notifications at once. For that purpose we have the notification center to always access notifications as needed.