Skip to content

Android WebView Bugfix: loadUrl() ignores default encoding#17394

Closed
adrianomelo wants to merge 1 commit into
react:masterfrom
adrianomelo:loadurl
Closed

Android WebView Bugfix: loadUrl() ignores default encoding#17394
adrianomelo wants to merge 1 commit into
react:masterfrom
adrianomelo:loadurl

Conversation

@adrianomelo

Copy link
Copy Markdown
Contributor

Closes #17385

The loadUrl() method ignores the default encoding on Android KitKat.

Motivation

I manage to reproduce the issue described by #17385 using Android KitKat (emulator).

Test Plan

I tested by modifying the RNTester project with the changes below. I have tested on Android KitKat (emulator) and Android Oreo 8.1 (device - Nexus 6p).

 <html>
   <head>
     <title>Hello Static World</title>
-    <meta http-equiv="content-type" content="text/html; charset=utf-8">
     <meta name="viewport" content="width=320, user-scalable=no">
     <style type="text/css">
       body {
@@ -391,7 +390,7 @@ const HTML = `
     </style>
   </head>
   <body>
-    <h1>Hello Static World</h1>
+    <h1>Hello Static World olá</h1>
   </body>
 </html>

Release Notes

[ANDROID] [BUGFIX] [WebView] - Default encoding for html content on older Android versions

Closes #17385

The loadUrl() method ignores the default encoding on Android KitKat.
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 30, 2017
@facebook-github-bot

Copy link
Copy Markdown
Contributor

@adrianomelo I tried to find reviewers for this pull request and wanted to ping them to take another look. However, based on the blame information for the files in this pull request I couldn't find any reviewers. This sometimes happens when the files in the pull request are new or don't exist on master anymore. Is this pull request still relevant? If yes could you please rebase? In case you know who has context on this code feel free to mention them in a comment (one person is fine). Thanks for reading and hope you will continue contributing to the project.

@react-native-bot react-native-bot added Android Ran Commands One of our bots successfully processed a command. Component: WebView Related to the WebView component. labels Mar 14, 2018
@react-native-bot react-native-bot added Platform: Android Android applications. Ran Commands One of our bots successfully processed a command. labels Mar 18, 2018
@hramos

hramos commented Aug 9, 2018

Copy link
Copy Markdown
Contributor

WebView PRs are on hold pending approval of the WebView proposal at react-native-community/discussions-and-proposals#3

@RSNara

RSNara commented Oct 12, 2018

Copy link
Copy Markdown
Contributor

WebViews have now been moved to react-native-community/react-native-webview. Would you like to relaunch this PR there? I'm sure the maintainers would love your contribution. 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Component: WebView Related to the WebView component. Platform: Android Android applications. Ran Commands One of our bots successfully processed a command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Font issue on Android 4.x WebView

6 participants