Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Reinstate loading of local HTML files in WKWebViewRenderer #12029

Merged
merged 3 commits into from
Sep 11, 2020

Conversation

hartez
Copy link
Contributor

@hartez hartez commented Sep 5, 2020

Description of Change

The move to WKWebView on iOS broke the ability to load local html documents. These changes reinstate that ability.

Issues Resolved

  • (re)fixes Bugzilla 32033

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

Not applicable

Testing Procedure

Manual test 32033 (the first option).

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@hartez hartez added 4.4.0 regression on 4.4.0 a/webview p/iOS 🍎 labels Sep 5, 2020
Copy link
Member

@rmarinho rmarinho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Build Failing
Renderers\WkWebViewRenderer.cs(137,15): error CS0103: The name 'Path' does not exist in the current context [D:\agent\1\s\Xamarin.Forms.Platform.iOS\Xamarin.Forms.Platform.iOS.csproj]

@rmarinho rmarinho self-requested a review September 11, 2020 18:38
Copy link
Member

@rmarinho rmarinho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Failing tests not related

@rmarinho rmarinho merged commit d5f2d35 into 5.0.0 Sep 11, 2020
vNext+1 (5.0.0) automation moved this from In Progress to Done Sep 11, 2020
@rmarinho rmarinho deleted the ios-local-html branch September 11, 2020 18:39
@samhouts samhouts added this to Done in Sprint 176 Sep 13, 2020
@samhouts samhouts added the approved Has two approvals, no pending reviews, and no changes requested label Sep 13, 2020
@samhouts samhouts added this to the 5.0.0 milestone Sep 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4.4.0 regression on 4.4.0 a/webview approved Has two approvals, no pending reviews, and no changes requested p/iOS 🍎
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants