search

h3x4d3c1m4l / BlazorProgressIndicator

Simple loading indicator for ASP.NET Blazor
GNU Lesser General Public License v3.0
86 stars 15 forks source link

Upadte UI was sent to all clients #20

Open gaffkins opened 4 years ago

gaffkins commented 4 years ago

Open the same page on two devices or more devices or browser tabs, Loading Indicator is show on every clients. I attached video with examples.

examples_video.zip

gaffkins commented 4 years ago

It happens on different pages too. Video in attachement. example3.mp4.zip

h3x4d3c1m4l commented 4 years ago

Can you provide me with a sample just so I can quickly reproduce the issue?

cakmaksevgi commented 4 years ago

blazor-loading-overlay-master.zip just open https://localhost:44318/piloadingdemo link two different tab. I changed here like this in IServiceCollectionExtension.cs

services.AddTransient<IIndicatorService, IndicatorService>(_ => new IndicatorService { Options = _options });

and it doesn't work also

h3x4d3c1m4l commented 4 years ago

Hi @cakmaksevgi thank you for the sample project. I'm a bit busy at the moment due to some things happening in my life the moment, but I'll take a look soon.

The issue seems to be that IndicatorService currently has no knowledge of different clients. For my own project I mostly use Blazor WebAssembly in which this does not apply. Fixing it should be possibly easily I think.