-
Notifications
You must be signed in to change notification settings - Fork 215
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
Enable Curved Windows (and explore whether to also support flat as option) #216
Comments
If curvature control makes sense, instead of a toggle we could make a slider from 0 (no curve) to the maximum curve allowed. |
Trello card created for this issue. We'll explore designs for this as part of the v1.1 work. |
Updating - based on discussion around font rendering quality and curved window edges (not being done), more explicitly asking to implement curved window support. Question is if we need design from @thenadj to:
|
|
@avrignaud What's the verdict on this? Seems like window positioning and cylindrical/curved windows don't play well together. |
I believe that we will not be doing this for v1.1. We are working on purely flat layers for 1.1, which will enable us to experiment with / investigate for post-1.1. |
a lot of folks complain about the curvature on Reddit. personally, I don't find it helps readability, but I'm also not reading comic books or Wikipedia articles for hours, so I'm probably not the best person to weigh in on this. if we do allow the option, I'd recommend it be buried as a non-default preference in the Settings. |
cc @joshmarinacci: Might be nice to promote here on this Reddit thread this upcoming feature in the next release of FxR: https://www.reddit.com/r/OculusGo/comments/asegfn/oculus_browser_with_a_flat_screen_rather_than/ |
Hardware: All
Steps to Reproduce: N/A
Actual Behavior: flat window
Expected Behavior: option to enable curved window, and presumably control curvature
Recommendation: enable option to toggle between flat and curved windows, and explore whether curvature window control makes sense
From YouTube video review: https://youtu.be/YfF0R2v0kEk?t=250
The text was updated successfully, but these errors were encountered: