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

[New Plugin] - Max-Hit Sounds - v1.0.0 - Sololegends #6559

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sololegends
Copy link
Contributor

Max-Hit Sounds

Add a special sound for when you hit max hit

  • Set sound effects per range, mage, and or melee hits.
  • Whitelist only certain items to trigger the sounds (Looking at p2 wardens dds deafening you)

# Max-Hit Sounds
Add a special sound for when you hit max hit

- Set sound effects per range, mage, and or melee hits. 
- Whitelist only certain items to trigger the sounds (Looking at p2 wardens dds deafening you)
@runelite-github-app
Copy link

@DominickCobb-rs
Copy link
Contributor

This may sound silly, but I have a similar plugin that is half baked. Have you looked at Attack Sound Notifications? A poorly named plugin that also can achieve this

@sololegends
Copy link
Contributor Author

This may sound silly, but I have a similar plugin that is half baked. Have you looked at Attack Sound Notifications? A poorly named plugin that also can achieve this

I didn't see that one in my searching for something that could do this. The main thing I wanted to have that I did find, and yours also doesn't seem to do this either, is to use in-game sounds and not sounds the user would have to load in themselves.

There are a bunch of plugins that allow people to load in sounds from files, but not use in-game sounds which is where this ones comes in.

@DominickCobb-rs
Copy link
Contributor

I could definitely make a small change to the panel to accommodate this. A toggle between an external sound and a sound ID is not a significant change

@sololegends
Copy link
Contributor Author

sololegends commented Sep 8, 2024

That sounds fine to me.

Would you also make it handle when you manual cast magic with the weapon and the hitsplat isn't directly tied to the weapon being held? That is something I'm working on today as as addition to this proposed plugin as I ran into that issue this morning.

Additionally a categorical setting for all mage weapons / all melee / all mage would be nice.

Edit: Though I'll also say I don't like it being a side panel. too many of those things already lol Given the option I'd still use the, admittedly more limited, config only version to keep the side panels cleaner.

@DominickCobb-rs
Copy link
Contributor

I'll be honest, I don't have the time or will to expand the scope beyond the initially mentioned panel changes on mine. Apologies for bringing it up. :)

@sololegends
Copy link
Contributor Author

I'll be honest, I don't have the time or will to expand the scope beyond the initially mentioned panel changes on mine. Apologies for bringing it up. :)

No worries, it is always good to make sure there isn't a bunch of plugins doing the exact same thing on there

@sololegends
Copy link
Contributor Author

This getting looked at maybe?

@iProdigy
Copy link
Member

To avoid plugin proliferation:
@sololegends would you be willing to contribute your changes to the existing plugin?
and @DominickCobb-rs would you be willing to add them as a collaborator on the repo?

@DominickCobb-rs
Copy link
Contributor

I'm willing to give full access/ownership of my existing plugin, but the way it sounded before is that this would supercede mine and cover the use case.

@LlemonDuck
Copy link
Contributor

Even still, it would be better to add it to the existing plugin so that current users are maintained instead of needing to find a new plugin, and new users aren't required to determine which one supercedes the other.

@sololegends Can you move your code into @DominickCobb-rs's repo?

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin added waiting for author waiting for the pr author to make changes or respond to questions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants