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

Added automatic screenshots #657

Merged
merged 4 commits into from
Sep 21, 2024
Merged

Conversation

ipkpjersi
Copy link
Contributor

@ipkpjersi ipkpjersi commented Sep 21, 2024

This adds an option for automatic screenshots when clicking on the stats tab and opening the bank. This is basically a poor man's player exports. It's very handy, I love this feature and I think other players will love it too.

Also, there are official clients out there like RL that take automated screenshots on levelup, on kill and on death, etc so IMO this isn't too unheard of as a feature.

This branch is based off of the screenshots branch, but it will also require the develop branch to be merged first to not cause a git conflict with adding more QoL options to ClientSettings.java, so we'll want to pull the develop branch onto this branch eventually (or, I suppose, the master branch after it's merged).

In other words, we can add 2 new QoL settings to ClientSettings at a later time (after my develop branch fixes/improvements PR is merged).
@ipkpjersi
Copy link
Contributor Author

Weird, why can't I mark this as draft? I'll need to add a var for the auto screenshots feature.

@ipkpjersi
Copy link
Contributor Author

Oh, there's the convert to draft.

@ipkpjersi ipkpjersi marked this pull request as draft September 21, 2024 01:39
@ipkpjersi
Copy link
Contributor Author

Okay it's ready again.

@ipkpjersi ipkpjersi marked this pull request as ready for review September 21, 2024 01:52
@Dark98 Dark98 merged commit 0120892 into 2006-Scape:master Sep 21, 2024
1 check passed
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.

2 participants