Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Mobile] Waypoints and links are not working for other users if the “Create and move objects” setting is disabled #4691

Closed
Softvision-GeluHaiduc opened this issue Sep 30, 2021 · 3 comments
Labels
android bug jira-hubs needs triage For bugs that have not yet been assigned a fix priority P2 Address when able [QA]:Normal-issue Label for QA to mark normal issues logged [QA]:Verified-fixed Label for QA to mark verified fixed issues

Comments

@Softvision-GeluHaiduc
Copy link
Contributor

Softvision-GeluHaiduc commented Sep 30, 2021

[Affected Versions]:

  • Firefox for Android 92.1.1
  • Chrome 94.0.4606.61

[Affected Platforms]:

  • Android 11

[Prerequisites]:

[Steps to reproduce]:

  1. Enter the room from preconditions.
  2. Try using a waypoint.
  3. Try opening a link to another website.
  4. Observe the behavior.

[Expected results]:

  • The waypoint and links work as expected for other users.

[Actual results]:

  • The waypoint and links don’t work for other users.

[Notes]:

  • The issue is not reproducible on PC.

┆Issue is synchronized with this Jira Task

@Softvision-GeluHaiduc Softvision-GeluHaiduc added bug needs triage For bugs that have not yet been assigned a fix priority [QA]:Normal-issue Label for QA to mark normal issues logged jira-hubs labels Sep 30, 2021
@takahirox takahirox added P2 Address when able and removed P3 Backlog labels Oct 11, 2021
@johnshaughnessy
Copy link
Contributor

johnshaughnessy commented Oct 14, 2021

I think the canMove here is too restrictive. Will test something like this later today:

Unstaged changes (1)
modified   src/systems/userinput/devices/app-aware-touchscreen.js
@@ -36,6 +36,10 @@ const getPlayerCamera = (() => {
   };
 })();
 
+function isSingleActionButton(el) {
+  return el.components.tags && el.components.tags.data.singleActionButton;
+}
+
 function shouldMoveCursor(touch, rect, raycaster) {
   const isCursorGrabbing = !!AFRAME.scenes[0].systems.interaction.state.rightRemote.held;
   if (isCursorGrabbing) {
@@ -74,7 +78,7 @@ function shouldMoveCursor(touch, rect, raycaster) {
     (isFrozen || !isPinned) &&
     !isStaticControlledMedia &&
     !isStaticMedia &&
-    (remoteHoverTarget && canMove(remoteHoverTarget))
+    (remoteHoverTarget && (canMove(remoteHoverTarget) || isSingleActionButton(remoteHoverTarget)))
   );
 }

@takahirox
Copy link
Contributor

@Softvision-GeluHaiduc

#4740 should fix the problem. Just in case, would you do double check on your end, too?

@Softvision-RemusDranca
Copy link
Contributor

I have verified the issue and it's no longer reproducible. The links and waypoints are now working while the “Create and move objects” setting is disabled.
Tested on Windows 10 using Chrome(94) and Firefox(93), macOS(Big Sur 11.6) using Safari(15.0), iOS (15.0.1) with Safari Android (11) using Chrome(94) and Firefox(92.1.1).

@Softvision-RemusDranca Softvision-RemusDranca added the [QA]:Verified-fixed Label for QA to mark verified fixed issues label Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android bug jira-hubs needs triage For bugs that have not yet been assigned a fix priority P2 Address when able [QA]:Normal-issue Label for QA to mark normal issues logged [QA]:Verified-fixed Label for QA to mark verified fixed issues
Projects
None yet
Development

No branches or pull requests

4 participants