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

Rounding in UIs #619

Closed
wants to merge 26 commits into from
Closed

Rounding in UIs #619

wants to merge 26 commits into from

Conversation

JFronny
Copy link
Contributor

@JFronny JFronny commented May 1, 2021

Of course this is optional and disabled by default

Changes after my message in discord:

  • The background of windows is drawn 50% into the title bar and not rounded at the top in order to look like it is connected to the window
  • Custom rendering for rounded background outlines to facilitate transparency
  • Topbar is rounded. If you want i can prevent the upper edges from being rounded, though i kinda like this
  • Rounding is a generalized theme property (for HUD support)
  • HUD backgrounds are rounded

Closes #544

@JFronny JFronny changed the title Allow rounding edges in UIs Rounding in UIs May 2, 2021
@AstroOrbis
Copy link

@seasnail8169 looks good :3

JFronny and others added 4 commits May 20, 2021 17:30
# Conflicts:
#	src/main/java/minegame159/meteorclient/gui/themes/meteor/MeteorGuiTheme.java
# Conflicts:
#	src/main/java/minegame159/meteorclient/systems/modules/render/hud/modules/InventoryViewerHud.java
@JFronny
Copy link
Contributor Author

JFronny commented Jun 8, 2021

Build failed because pipeline uses java 8

# Conflicts:
#	src/main/java/minegame159/meteorclient/systems/modules/render/hud/modules/CombatHud.java
#	src/main/java/minegame159/meteorclient/systems/modules/render/hud/modules/InventoryViewerHud.java
#	src/main/java/minegame159/meteorclient/systems/modules/render/hud/modules/PlayerModelHud.java
# Conflicts:
#	src/main/java/minegame159/meteorclient/systems/modules/render/hud/HudRenderer.java
# Conflicts:
#	src/main/java/meteordevelopment/meteorclient/gui/themes/meteor/widgets/WMeteorQuad.java
#	src/main/java/meteordevelopment/meteorclient/systems/modules/render/hud/HudRenderer.java
#	src/main/java/meteordevelopment/meteorclient/systems/modules/render/hud/modules/InventoryViewerHud.java
# Conflicts:
#	src/main/java/meteordevelopment/meteorclient/gui/GuiTheme.java
@AstroOrbis
Copy link

@seasnail8169

@arlomcwalter
Copy link
Member

what

@AstroOrbis
Copy link

merge it or you're a baka

@arlomcwalter
Copy link
Member

no

@Cypphi
Copy link

Cypphi commented Jun 19, 2021

mfw (my face when) still not accepted :trollface:

@ghost
Copy link

ghost commented Jul 13, 2021

or its just not very good
well not bad but not perfect at least

Maybe add as an option instead of replacing if it isn’t already

@JFronny
Copy link
Contributor Author

JFronny commented Jul 13, 2021

An option to select by how much everything is rounded is shown and set to 0 by default (= no rounding)

@Cypphi
Copy link

Cypphi commented Aug 21, 2021

It's possible to make the curves smooth with OpenGL stuff right

@RedCarlos26
Copy link
Contributor

It's possible to make the curves smooth with OpenGL stuff right

Yea I think so

@Cypphi
Copy link

Cypphi commented Aug 26, 2021

TYad8FrPud.mp4

This prob was a known issue already but the hover effect overrides the rounded edges

@JFronny
Copy link
Contributor Author

JFronny commented Aug 29, 2021

Already known, however these are things that I am too stupid to fix without large amounts of very bad code

@AstroOrbis
Copy link

TYad8FrPud.mp4

This prob was a known issue already but the hover effect overrides the rounded edges

Ain’t that sigma

@Cypphi
Copy link

Cypphi commented Aug 30, 2021

TYad8FrPud.mp4
This prob was a known issue already but the hover effect overrides the rounded edges

Ain’t that sigma

meteor (with sigma font)

@sbxte
Copy link
Collaborator

sbxte commented Oct 4, 2021

5 month old pr

@RedCarlos26
Copy link
Contributor

It's been so long 🤯

@AstroOrbis
Copy link

cmon lets get this to a year x3

@RedCarlos26
Copy link
Contributor

Lets get it

@Lerbiq
Copy link
Contributor

Lerbiq commented Nov 10, 2021

The pr makes the game crash with memory allocation error. "corrupted size vs. prev_size" and sometimes this "java: malloc.c:4259: _int_malloc: Assertion `(unsigned long) (size) >= (unsigned long) (nb)' failed."

This happens when you open the accounts screen with the round setting set to 7 and two cracked accounts in the manager after starting the game.

Opening the click gui before you open the accounts prevents the game from crashing, however it introduces a new issue when you then close the accounts screen, go to main menu and quit the game using the button in the game. The game will crash with this error: "double free or corruption (out)" and won't let meteor save.

Those issues seem to occur only when the round setting is set to 7 only.

@MineGame159
Copy link
Member

Closing this PR because the GUI is going go get rewritten in the near future and will most probably include rouded corners.

@Wide-Cat
Copy link
Collaborator

RIP 😔

@RickyTheRacc
Copy link
Contributor

RIP? Minegame just confirmed gui rewrite that's sexyyy 😳

@JFronny JFronny deleted the rounded-ui branch November 23, 2021 14:09
@RedCarlos26
Copy link
Contributor

RedCarlos26 commented Jul 9, 2022

8 months later 😭

@MineGame159
Copy link
Member

Uhhhh uhhh 👀

@C10udburst
Copy link
Contributor

🤨

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

Successfully merging this pull request may close these issues.

[Suggestion] Rounded edges in the HUD / ClickGUI