Skip to content
This repository has been archived by the owner on Apr 8, 2023. It is now read-only.

client: Improve menumanager #2026

Merged
merged 5 commits into from
Nov 13, 2019
Merged

client: Improve menumanager #2026

merged 5 commits into from
Nov 13, 2019

Conversation

Lucwousin
Copy link
Contributor

Makes swaps work all the time, even with really low fps (lol) or when using mousekeys/anything

@Lucwousin Lucwousin requested a review from Ganom November 13, 2019 03:05
@Lucwousin Lucwousin added the bug Something isn't working label Nov 13, 2019
@tomcylke
Copy link
Contributor

Doesn't it work the way it does rn for performance reasons? Iirc we had the capability to do this a while ago but we took it out.

@Lucwousin
Copy link
Contributor Author

No it was the way it was because menumanager is a pile of yikes and nobody knew why it was really happening or how to fix it. Come to think of it, this can be improved (easily (as easy as changing 3 lines)) so let me do that real quick

@tomcylke
Copy link
Contributor

I stand corrected

@Lucwousin
Copy link
Contributor Author

Seems to have broken for me..? smh

@Lucwousin Lucwousin closed this Nov 13, 2019
@Lucwousin Lucwousin reopened this Nov 13, 2019
Ganom
Ganom previously approved these changes Nov 13, 2019
Copy link
Contributor

@Ganom Ganom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working well for me when "isDraggingWidget" check was removed.

Ganom
Ganom previously approved these changes Nov 13, 2019
@Lucwousin Lucwousin merged commit bfa3b22 into open-osrs:master Nov 13, 2019
@Lucwousin Lucwousin deleted the menumane branch November 13, 2019 16:13
@Switchssb
Copy link

noob question, but how do i reverse this commit in my live build

@Crystalknoct
Copy link
Contributor

Search in Intellij, Version control, Log tab.
https://i.imgur.com/S2PjO42.jpg
Commit: bfa3b22

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

Successfully merging this pull request may close these issues.

5 participants