Skip to content

Commit fe21397

Browse files
authored
fix(WebView): Fix bug with onLoadingStart event (microsoft#1495)
Mistakenly set DOM load starting event to `topLoadStart` instead of `topLoadingStart`. The direct events for `WebView` are mapped in the UIManager; the `topLoadStart` event is not mapped for the `WebView` whereas the latter is.
1 parent 5c164ce commit fe21397

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

ReactWindows/ReactNative/Views/Web/Events/WebViewLoadEvent.cs

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
1-
using Newtonsoft.Json.Linq;
1+
using Newtonsoft.Json.Linq;
22
using ReactNative.UIManager.Events;
33

44
namespace ReactNative.Views.Web.Events
55
{
66
class WebViewLoadEvent : Event
77
{
8-
public const string TopLoadStart = "topLoadStart";
8+
public const string TopLoadingStart = "topLoadingStart";
99
public const string TopLoadingFinish = "topLoadingFinish";
1010

1111
private readonly string _url;

ReactWindows/ReactNative/Views/Web/ReactWebViewManager.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,7 @@ private void OnNavigationStarting(object sender, WebViewNavigationStartingEventA
347347
.DispatchEvent(
348348
new WebViewLoadEvent(
349349
tag,
350-
WebViewLoadEvent.TopLoadStart,
350+
WebViewLoadEvent.TopLoadingStart,
351351
e.Uri?.ToString(),
352352
true,
353353
webView.DocumentTitle,

0 commit comments

Comments
 (0)