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

Is there a better term than "TransientInput" for method names? #75

Open
thetuvix opened this issue Jan 22, 2020 · 3 comments
Open

Is there a better term than "TransientInput" for method names? #75

thetuvix opened this issue Jan 22, 2020 · 3 comments

Comments

@thetuvix
Copy link

For the recently merged transient hit test PR, the design seems like it holds together at first glance. The primary unfortunate aspect to me there is that we'd be introducing methods like requestHitTestSourceForTransientInput that have ...TransientInput in the name. Transient input is a concept defined in the core WebXR spec, but the term transient never appears in the actual API surface anywhere yet. It feels odd as someone approaching the hit-test API to only encounter the term TransientInput here.

It seems worth some bikeshedding to see if we can align those "transient" member names closer to the core API somehow. Another option would be to tweak the design to more directly reference core API members. A random idea that comes to mind: switching the method to requestHitTestSourceForInputProfile to build on the profiles list that already manifests in the core API surface, returning hits whether the specified profile is transient or not.

Originally posted by @thetuvix in #74 (comment)

@thetuvix
Copy link
Author

@Manishearth's comment from this discussion in #74:

A random idea that comes to mind: switching the method to requestHitTestSourceForInputProfile to build on the profiles list that already manifests in the core API surface, returning hits whether the specified profile is transient or not.

I'd kinda prefer if the transient API was only used for transient inputs; it doesn't seem great to have two ways of doing the same thing for non-transient inputs, and furthermore it only works with the target ray space, not the grip space. I don't hold this opinion strongly.

That said I agree that the naming is kinda weird.

@bialpio
Copy link
Contributor

bialpio commented Jan 24, 2020

I agree - the naming is awkward now, but I did not find a better alternative that would send a sufficiently strong signal that this flavor of hit test is supposed to be used only in a fairly specific case. Keeping this constraint in mind, is there a better-suited name?

(...) it doesn't seem great to have two ways of doing the same thing for non-transient inputs (...)

Yeah, I'd like to keep the responsibilities of those 2 methods separate, especially given that the main reason behind creating this variant was to close the gaps that the non-transient variant has w.r.t. transient input.

@toji
Copy link
Member

toji commented Jan 27, 2020

/agenda to discuss overall comfort level with the API shape of the transient hit test functionality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants