New issue
Advanced search Search tips

Issue 678871 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Webview copy dialog UI overlap in RTL language when AP don't support RTL

Reported by hjchangl...@gmail.com, Jan 6 2017

Issue description

Steps to reproduce the problem:
1. Change Android device to RTL language (ex.Arabic)
2. Open a AP (include webview) which don't support RTL in manifest 
3. Long press text in webview content to open the copy/paste dialog
4. In dialog, "Copy" is overlapping with "More..." button

What is the expected behavior?
Dialog UI should be OK whatever AP support RTL layout or not.

What went wrong?
As screenshot,
"Copy" button is overlapping with "More...",
The only way to fix it is add android:supportsRtl="true" to AP manifest,
But whatever AP support RTL layout or not,
Webview dialog UI should be OK.

Did this work before? N/A 

Chrome version: 54.0.2840.85  Channel: stable
OS Version: 7.0
Flash Version: N/A
 
copy_dialog_UI_overlap.png
134 KB View Download
Update.
Did this work before?
Yes, when it run on Android M (6.0) it will be OK,
Only happen on Android N(7.0).
Screenshot_android_m.png
18.7 KB View Download
Components: Mobile>WebView
Labels: Needs-Feedback
did you find similar issue in chrome Android?
No, Chrome app won't happen this,
Because Chrome app support RTL,
But default value of android:supportsRtl is false,
So all app which don't write android:supportsRtl="true" will meet this issue.
Project Member

Comment 5 by sheriffbot@chromium.org, Jan 16 2017

Labels: -Needs-Feedback Needs-Review
Owner: michaelbai@chromium.org
Thank you for providing more feedback. Adding requester "michaelbai@chromium.org" for another review and adding "Needs-Review" label for tracking.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Owner: ----
Assigned to WebView bug cop

Comment 7 by cda...@chromium.org, Mar 13 2017

Cleaning up "Needs-Review" label as we are not using this label for triage. Ref  bug 684919 

Comment 8 by cda...@chromium.org, Mar 13 2017

Labels: -Needs-Review
Cc: ntfschr@chromium.org
Labels: Needs-Feedback
> So all app which don't write android:supportsRtl="true" will meet this issue.

hjchangliao@, if I understand you correctly, this sounds like it's an app issue. From what you've described, it sounds like it's the app's responsibility to add `android:supportsRtl="true"`, and WebView can't do anything here. Please correct me if I'm misunderstanding.
No, it's not app's responsibility.

Support RTL or not is optional for developer,
and default value of android:supportsRtl is false.

https://developer.android.com/guide/topics/manifest/application-element.html#supportsrtl
Project Member

Comment 11 by sheriffbot@chromium.org, May 12 2017

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "ntfschr@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: Needs-Feedback
I can't reproduce this on my device: Nexus 5X Android NRD91R 3563105, with WebView 54.0.2840.85. I've included a screenshot of what it looks like (I have no clue what the content on the screenshot says, but I hope it's not offensive or inappropriate).

The steps I tried:
 1. Install simple WebView app which loads Google.com but does not change default "android:supportsRtl" setting
 2. Set device language to arabic
 3. Open my simple app, do any search, find some text on the results page, and highlight it by long pressing
 4. Look at the selection menu, observe that no words are overlapping.

I even tried on 53, 55, 57, and 58. All of them look fine.

hjchangliao@, can you please provide your device and Android version information?
2017-05-12_18-33-25_droidshot_EUih6.png
337 KB View Download
I try with Pixel Large,
It won't happen on Android 7.1

Android N 7.0 : NMD63C (happen)
Android N7.1.2: N2G22 (not happen)
(sorry for I can only get development ROM)

I pack my test app to attachment,
or you can try below app on google play:
https://play.google.com/store/apps/details?id=com.piti.webviewtester
https://play.google.com/store/apps/details?id=com.html5test.webview
https://play.google.com/store/apps/details?id=com.webviewbrowser
pixel_7.0.png
133 KB View Download
pixel_7.1.png
255 KB View Download
HJTest.zip
2.6 MB Download
Project Member

Comment 14 by sheriffbot@chromium.org, May 16 2017

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "ntfschr@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 15 by ctzsm@chromium.org, Oct 19 2017

Status: WontFix (was: Unconfirmed)
IIUC, this looks like an Android issue since we have no control of this in WebView. And it should be fixed in later Android version.

Sign in to add a comment