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

fix: divide minecraftpage into tabs #950

Merged
merged 2 commits into from May 6, 2023

Conversation

DioEgizio
Copy link
Member

@DioEgizio DioEgizio commented Mar 25, 2023

this way small screen users can use the launcher setting without having window a bigger than their actual screens
closes partially #220

some screenshots
i tried to resize the new instance dialog but failed at it :/. Not too much of an issue as it's not as bad as what settings was
general general tab
System-related tweaks page System-related tweaks tab (here there'd be the perfomance part on linux)

this way small screen users can use the launcher settings without having window a bigger than their actual screens
Signed-off-by: DioEgizio <83089242+DioEgizio@users.noreply.github.com>
@DioEgizio DioEgizio added bug Something isn't working simple change labels Mar 25, 2023
@TheKodeToad
Copy link
Member

TheKodeToad commented Mar 25, 2023

Maybe it should be named "Workarounds" like the instance window? Ideally the group box should also be labelled "Native libraries" too.

@DioEgizio
Copy link
Member Author

Maybe it should be named "Workarounds" like the instance window? Ideally the group box should also be labelled "Native libraries" too.

Mangohud/gamemode aren't workarounds though. I didn't find any better name :/

@TheKodeToad
Copy link
Member

Maybe have it in a performance tab then.

@DioEgizio
Copy link
Member Author

Native lib workarounds aren't necessarly performance, and making a tab for every group seems a bit weird so idk

@TheKodeToad
Copy link
Member

That is the situation in the instance window.
The settings probably need reworking though :P. Maybe with a scrollbar... (i probably already said this)

@flowln flowln added this to the 7.0 milestone Apr 7, 2023
@DioEgizio DioEgizio requested a review from Scrumplex May 2, 2023 13:39
Co-authored-by: Sefa Eyeoglu <contact@scrumplex.net>
Signed-off-by: DioEgizio <83089242+DioEgizio@users.noreply.github.com>
@Scrumplex Scrumplex merged commit d535f73 into PrismLauncher:develop May 6, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working simple change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants