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

You can stop carrying/dragging from both interaction menu and mousewheel menu #2284

Closed
gienkov opened this issue Aug 26, 2015 · 15 comments
Closed

Comments

@gienkov
Copy link
Contributor

gienkov commented Aug 26, 2015

ACE3 Version: 3db981a

Mods:

Placed ACE3 Modules:

  • -

Description:
Like in title, you get action to drop crates under scrollwheel menu. I don't think this piece of code is longer needed since you can stop dragging under interaction menu.

// add scrollwheel action to release object
_actionID = _unit getVariable [QGVAR(ReleaseActionID), -1];

if (_actionID != -1) then {
    _unit removeAction _actionID;
};

_actionID = _unit addAction [
    format ["<t color='#FF0000'>%1</t>", localize LSTRING(Drop)],
    QUOTE([ARR_2(_this select 0, (_this select 0) getVariable [ARR_2(QUOTE(QGVAR(draggedObject)),objNull)])] call FUNC(dropObject)),
    nil,
    20,
    false,
    true,
    "",
    QUOTE(!isNull (_this getVariable [ARR_2(QUOTE(QGVAR(draggedObject)),objNull)]))
];

Steps to reproduce:

  • Start carrying something.
  • You get two options to drop it, in both mousewheel and interaction menu

Where did the issue occur?
"Multiplayer", "Singleplayer"

RPT log file:
-

@jonpas
Copy link
Member

jonpas commented Aug 26, 2015

That was by design in AGM where you couldn't open interaction menu in some cases (object not directly in front of you). I believe this can now be removed since you can look at the object and use the menu normally.

@alganthe
Copy link
Contributor

Indeed it became redundant since you can just press the interaction key / scroll down to release the object.

@PabstMirror
Copy link
Contributor

When carrying a patient on your back there is no way to access his interaction point from first person.
When dragging a patient, I can't aim low enough to access the interaction point to release.

With track IR (or holding ALT/free look), or using different interaction menu options it is possible to overcome these problems, but with default options it is difficult/impossible.

Now I hate the addAction menu as much as anyone, so maybe we can come up with a cleaner solution, but the user needs a clear, easily accessible way to drop what they are carrying.

@gienkov
Copy link
Contributor Author

gienkov commented Aug 26, 2015

Keybind?

@jonpas
Copy link
Member

jonpas commented Aug 26, 2015

That would work, I suggest R. And next to that display the same icon as is used in attach?

@gienkov
Copy link
Contributor Author

gienkov commented Aug 26, 2015

R will reload your weapon. Maybe LCTRL+R ?

@SzwedzikPL
Copy link
Contributor

but the user needs a clear, easily accessible way to drop what they are carrying.

So why vanilla action menu is so bad here? I dont like vanilla menu too but in this case it works just fine.

@jonpas
Copy link
Member

jonpas commented Aug 26, 2015

R will reload your weapon. Maybe LCTRL+R ?

You can reload while dragging? That sounds weird, one-handed reloads! 😁

@rebelvg
Copy link
Contributor

rebelvg commented Aug 26, 2015

Maybe move that option to self-interaction. That way you will always able to drop the person you are carrying.

@gienkov
Copy link
Contributor Author

gienkov commented Aug 27, 2015

You can even throw grenades while dragging :D.

@sargken
Copy link
Contributor

sargken commented Aug 28, 2015

I agree add it to self interaction and you could add a setting in your ACE controls that way if people want to use a keybind they can.

@commy2
Copy link
Contributor

commy2 commented Aug 30, 2015

I like the scroll wheel option for this. It's faster.

I'm against adding a hotkey for something you probably do only once or twice a mission.

@Khaosmatic
Copy link

You may as well leave it in the action menu Commy is right its not that often you need to use it anyway. The maximum amount of options on the action menu will only be like 6 at a time so its no hassle.

@MikeMatrix
Copy link
Contributor

Why not just make it an optional keybind, that can be assigned by yourself, if you really want a keybind? There is no need to default a key to it. Just add in the option, since it's not gonna be a lot of work anyways.
Allows everybody to use their preferred method.

@PabstMirror
Copy link
Contributor

#4034 Added optional keybind

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

No branches or pull requests

10 participants