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

Places app: Portal Spawner and Copy URL #1296

Merged
merged 5 commits into from
Feb 10, 2025

Conversation

AleziaKurdis
Copy link
Contributor

@AleziaKurdis AleziaKurdis commented Jan 17, 2025

This PR adds 2 new functionalities to the Places app. - Place details panel:

  • Portal spawner
  • Copy URL

image

Copy URL:
We can now, from the Place details panel, click on a button "Copy URL" to copy the place url into the clipboard.
This will allow the user to paste the hifiurl anywhere else, including to the chat app. for sharing the link.

When the "Copy URL" button is click, a notification is trigger to confirm the copy.

Portal Spawner:
We can now, from the Place details panel, click on a button "Portal" to spawn in-world a 3D portal leading to that place.

The portal is rendered to everyone in a range of 100m from the requester.
The portal get automatically deleted after 45 sec.

The portal is render in the same scale as the avatar's user to be functional for himself.
(yes, users will not see a same size portal but it will be at the same position)

The portal is built with local Entities (since avatar entity are not allowed everywhere). When the Portal is requested to be spawn, this send by message to everyone and each user will render its own local version from their own Places app.

image001

There is a limit of N (currently 15) portals that can be spawn. This is to limit the risk to see this used negatively by overwhelming people with tonnes of portal ruining the performance.

Each places has a persistent portal color, the color is computed based on the placeID.

ezgif-2-5bee7d55be

Add Portal spawner and Copy URL
Adding Portal Sound and Maximum Spawn protection
@AleziaKurdis
Copy link
Contributor Author

This is addressing Issue #1292.

Using Sound Entity for the portal, this simplify the termination of the looping sound.
Fix Documentation Link for EntityProperties-Sound
In The documentation, the syntax wasn't right for Picks.PICK_AVATARS
Previously we could read: PICK_AVATATRS
But it was supposed to be be: PICK_AVATARS
@AleziaKurdis
Copy link
Contributor Author

Added 2 documentations fixes.

@ksuprynowicz
Copy link
Member

I just tested it and portals works perfectly across multiple clients.
The portal particle effect looks very cool :)

@ksuprynowicz ksuprynowicz added enhancement New feature or request needs CR This pull request needs to be code reviewed QA approved This pull request has been successfully tested labels Feb 10, 2025
Copy link
Member

@ksuprynowicz ksuprynowicz left a comment

Choose a reason for hiding this comment

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

Everything looks good :)

@ksuprynowicz ksuprynowicz added CR approved This pull request has been successfully code reviewed and removed needs CR This pull request needs to be code reviewed labels Feb 10, 2025
@ksuprynowicz ksuprynowicz merged commit ccdac96 into overte-org:master Feb 10, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CR approved This pull request has been successfully code reviewed enhancement New feature or request QA approved This pull request has been successfully tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants