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

Magazine Repack - Add setting & hint for repacking loaded magazines #8475

Merged
merged 6 commits into from
Oct 12, 2021

Conversation

LinkIsGrim
Copy link
Contributor

@Drofseh
Copy link
Contributor

Drofseh commented Oct 3, 2021

I'm happy with this solution.
I don't think there is a need for 0,1,2 though, just bool should be fine and always show the hint if it's turned on.
I think repackLoadedMagazines would be clearer than repackEquippedMagazines, but whatever.

@jonpas jonpas added the kind/bug-fix Release Notes: **FIXED:** label Oct 3, 2021
@jonpas jonpas added this to the 3.14.0 milestone Oct 3, 2021
@jonpas jonpas changed the title MagazineRepack - Add setting & hint for repacking equippped magazines Magazine Repack - Add setting & hint for repacking equippped magazines Oct 3, 2021
@LinkIsGrim
Copy link
Contributor Author

Went with equipped because technically any magazine with ammo in it can be called 'loaded'. Equipped can only mean it's currently in use.

@Drofseh
Copy link
Contributor

Drofseh commented Oct 4, 2021

equipped to me sounds like any magazine in my equipent/inventory
eg. "I am equipped with 10 magazines and a rifle."

Since arma doesn't have unloaded magazines, I still think repackLoadedMagazines sounds clearer

@Dabako
Copy link
Contributor

Dabako commented Oct 6, 2021

I'm happy with this solution. I don't think there is a need for 0,1,2 though, just bool should be fine and always show the hint if it's turned on. I think repackLoadedMagazines would be clearer than repackEquippedMagazines, but whatever.

Do I understand that right. If there are no three options that would mean there is only the option to either don't repack the magazin that is loaded or do it with an hint? I am personally liking that touch that when you forget to reload your gun after repacking you are just without bullets. So it would be nice to have the option to repack that mag without a hint.

@Drofseh
Copy link
Contributor

Drofseh commented Oct 6, 2021

Right now there are three settings, "no unload", "no hint", "hint".

Because the player never sees their weapon get unloaded (the bullets just magically disappear), I think the "no hint" option is a bad idea. This is the current situation that I want to avoid.

If you really really want it though, I guess it's ok if it's included, but personally I think it's a bug not a feature.

@Dabako
Copy link
Contributor

Dabako commented Oct 6, 2021

Yeah it would really be nicer if there would be an animation playing unloading the weapon so people could tell that you gun got unloaded. (I think to add something like an unloading animation would be a bit to complicated for this problem?)

But I was kinda happy that I unloaded it and that I got punished for not reloading my weapon. I understand that this behaviour can be seen as a bug. (What it kinda seems to be without an animation).
I think it would be nice to give players that type of option to make it with our without hints because I personally just want as few hints as possible in my gameplay experience.

But if it would make problems to leave the 3 options in, it would also be okay for me. Just wanted to show that there could be people who like this feature (bug).

@jonpas
Copy link
Member

jonpas commented Oct 10, 2021

Since arma doesn't have unloaded magazines, I still think repackLoadedMagazines sounds clearer

Agreed.

If you really really want it though, I guess it's ok if it's included, but personally I think it's a bug not a feature.

Not a bug, just a missing animation as @Dabako said. I can see why it can be seen as a bug though, and that could be another solution in addition to this PR. I am ok with removing the "no hint" option for now for the sake of clarity.

@jonpas jonpas modified the milestones: 3.14.0, 3.14.1 Oct 10, 2021
@LinkIsGrim LinkIsGrim changed the title Magazine Repack - Add setting & hint for repacking equippped magazines Magazine Repack - Add setting & hint for repacking loaded magazines Oct 10, 2021
@jonpas jonpas modified the milestones: 3.15.0, 3.14.0 Oct 12, 2021
@PabstMirror PabstMirror merged commit bbe4b48 into acemod:master Oct 12, 2021
AndreasBrostrom pushed a commit to AndreasBrostrom/ACE3 that referenced this pull request Dec 3, 2021
…cemod#8475)

* add setting to repack equipped magazines

* derp

* derpderp

* always show hint, remove setting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug-fix Release Notes: **FIXED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Magazine Repack - When repacking a weapon can be unloaded without the player knowing
6 participants