New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 906667 link

Starred by 4 users
Status: Assigned
Owner:
█ elalish@chromium.org
Last visit 20 days ago
Cc:
vsupruniuk@google.com
dougman@chromium.org
█ bshe@chromium.org
█ asimjour@chromium.org
dbbrooks@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug
Proj-VR
Proj-XR
Proj-XR-VR



Sign in to add a comment

[VR Browsing] Clicking on Translation popup bar settings menu shows 2D UI

Project Member Reported by dbbrooks@chromium.org, Nov 19 
Chrome Version: 72.0.3615.0
VRCore: 1.19.219215888
OS: O
Device: S8+

What steps will reproduce the problem?
(1) Enter VR Browsing
(2) Go to a site where the automatic translation popup bar appears. E.g. https://vimeo.com/278174511
(3) Click on the vertical 3 dots menu inside the translation popup bar.

What is the expected result? All 2D UI should be blocked or rendered in VR.

What happens instead? 2D UI appears, breaking VR experience.

Screenshot attached.

Comment 1 by dbbrooks@chromium.org, Nov 19

translationPopup.jpg
871 KB View Download

Comment 2 by btebbs@google.com, Nov 27

Status: Available (was: Untriaged)

Comment 3 by elalish@chromium.org, Dec 19

Owner: elalish@chromium.org
Status: Assigned (was: Available)
The issue seems to be that TranslateMenuHelper.java calls ListPopupWindow, which is an android widget that wraps PopupWindow (another android widget). We want everyone to use Chrome's UiWidgetFactory, which has a VR wrapper for PopupWindow, but not ListPopupWindow. Not sure how to fix this besides copying android's UI widgets into Chrome's UI widget list, but considering android's list is much longer, that seems like a bad precedent. Any ideas? Also, the plans to automatically notify us of missing UI like this was never implemented, so we don't know how much there is.

Comment 4 by elalish@chromium.org, Dec 19

Cc: asimjour@chromium.org bshe@chromium.org dbbrooks@chromium.org
 Issue 827244  has been merged into this issue.
►

Sign in to add a comment