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

[suggestion] WHM Cure 2/Cure 1 #198

Closed
wisdomsave opened this issue Feb 13, 2022 · 7 comments
Closed

[suggestion] WHM Cure 2/Cure 1 #198

wisdomsave opened this issue Feb 13, 2022 · 7 comments

Comments

@wisdomsave
Copy link

What is thy suggestion?
An option for WHM to have Cure 1 be replaced by Cure 2 (swapped back if the skill is unavailable or there is not enough MP to cast Cure 2)

  1. Are abilities in question must be truly mutually exclusive?
    Ask any WHM main and they'll tell you Cure 1 is pretty useless when compared to Cure 2. The proc it offers is useless and it is a heal and DPS loss.

  2. Does this actually saves you button space?
    It saves one whole button! Many WHM players unbind Cure 1 anyway but have to rebind it when doing lower level content.

  3. Is it wild and wacky?
    I don't think so!

  4. Is the overall level of intelligence required to execute is about as low as the floor?
    Simpler than many of the other features you've implemented I should think!

Thanks in advance!

@wisdomsave
Copy link
Author

P.S: Loving the plugin! Looking forward to further class support and features.

@rejahtavi
Copy link

"Many WHM players unbind Cure 1 anyway but have to rebind it when doing lower level content."
+1

@Ammako
Copy link

Ammako commented Aug 26, 2022

I would suggest the same for Medica/II tbh.

@attickdoor
Copy link
Collaborator

you clearly did not read the pinned issue completely, where I LITERALLY explain exactly why I won't implement this. I'm pinning this issue for my archives, and in hopes that others may read it and learn from it.

@attickdoor attickdoor reopened this Aug 31, 2022
@attickdoor attickdoor pinned this issue Aug 31, 2022
@wisdomsave
Copy link
Author

wisdomsave commented Aug 31, 2022

Ah sorry, I'm not very familiar with GitHub and can't figure out where to find pinned issues.

I readt he readme and that's it.

@wisdomsave
Copy link
Author

wisdomsave commented Aug 31, 2022

Ah I think I found what you were referring to; #119

It's a shame I disagree, as having the option would be nice as I and most WHM mains literally NEVER use Cure 1, it's a straight up dps loss, but I've already made that point and you've already made yours so yeah. Fair enoughski. 👍

@Ammako
Copy link

Ammako commented Aug 31, 2022

This is implemented on my end the other way around; Cure II downgrades back into Cure when synced, retaining the ability to use Cure alongside Cure II.

This will be split off into its own plugin by the end of this week, because the intention isn't to step on toes and compete against XIVCombo, but if you can't wait, a build is available for manual download, until I get around to working on it.

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

No branches or pull requests

4 participants