-
-
Notifications
You must be signed in to change notification settings - Fork 18
Update settingsSection.tsx #91
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
base: master
Are you sure you want to change the base?
Conversation
added more res option with aspect ratio and GFN description
|
Can we please NOT remove resolution options? |
Didn't see 720p option was missing
Didn't see 720p option was removed my bad. |
AstralVixen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add back HD option:
1366 x 768 (16:9)
It also requires at least a perrformance subscribtion
bringing back HD res
|
readded HD res, sry, my fault |
| { label: "2560x1440(16:9) - QHD - Performance", value: "2560x1440" }, | ||
| { label: "2560x1080(21:9) - UW FHD - Performance", value: "2560x1080" }, | ||
| { label: "1920x1080(16:9) - FHD - Basic", value: "1920x1080" }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps trim down the whitespace, if it is there for a reason, can we make sure it applies to all of them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i have reworked this section to hit the previous layout better. Further all resolutions mentioned in gfn article are now inlcuded.
I added a link to this Article in sidebar, that the user can take a look if his settings are supported in the subscription. please take a look an provide feedback
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure if 4K+5K even works?
If I remember correctly when @Elliesaur made the initial PR for the resolutions & FPS settings she mentioned that 4K and 240FPS doesn't work .(Nvidia itself states that 240 FPS doesn't even work on the web version which GFI is based on, only on the windows native app so maybe the same is true for 4K/5K resolutions?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i have no ultimate sub, therefore i cannot be sure, therefore i have deleted 4/5k option
added firther resolutions, added link to gfn knowldge base
deleted 4k/5k option
AstralVixen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I'll go ahead and approve it. Please wait while @t0msk also gives it an approval before we merge it.
|
I'm sorry, is it supposed to work or is it just the dropdown ? I pulled your repo, looked at the modified file by hand, reloaded, yeah, I see the options, but they just don't do anything ? I tested all 3 21:9 resolutions (on Performance, with a 21:9 screen), kept the Infinity sidebar opened, it just does not do what it says on the tin. EDIT : It does work but only If I use Chrome as User Agent, maybe this should be written somewhere in the Readme ! |
|
I had the same issue with the former Version, that changing the resolution is only honored when set to Chrome. |
|
Any chance this is going to be released soon? |
|
@maz0r maybe you take a look at following link github formerly i used this solution with mitmproxy, chrome, interceptor script. maybe try this as tempory solution. it would be nice to get feedback, if the proxy version works for you and higher res (i have just performance subscription... i was never able to get higher than 2560x1080 -> https://nvidia.custhelp.com/app/answers/detail/a_id/5223/~/how-can-i-set-up-my-windows-pc-with-my-geforce-now-ultimate-or-performance |
Well I got that working at 2560x1440x120, I cannot get 3440x1440x120 working on either method, and I suspect its a browser limitation? |


added more res option with aspect ratio and GFN description