Molinari

Molinari

1M Downloads

Have to click more than once to disenchant

Neipas09 opened this issue ยท 27 comments

commented

Before patch 10.1, i was able to hold ALT and click on an item, and it would disenchant it.
Now, if i hold ALT and click on an item, it turns the icon Blue and i have to let go of ALT and click on the item again to make it disenchant.

commented

Sounds like you have another addon that uses the alt key, try adjusting the modifiers used in /molinari. If you find that that is the case I'd be interested to see which addon it is you have that is interfering.

commented
  1. , It's the same no matter what modifiers i use.
  2. Disabled all addons except Molinari, and the same thing is happening.
commented

+1

This is only happening with 10.1 items for me.

commented

I'll need a screenshot or something of this.

commented

You said the item turned blue, there's nothing in Molinari that would do that.

commented

What is a screenshot going to do?

commented

Sorry, i explained it wrong.

  1. Mouse over the item = Mouse icon (hand) is gray
    2023-05-10 13_08_50-World of Warcraft

  2. Hold down ALT while mouse over item = Item starts to glow
    2023-05-10 13_10_17-World of Warcraft

  3. Click on item = Mouse highlights in blue (normal disenchant action), but does NOT start disenchanting
    2023-05-10 13_12_22-World of Warcraft

  4. Have to LET GO of ALT and then click on the item to get disenchant to work.

commented

Also, what bag addon is that?

commented

Ah, thanks for the thorough explanation. I am however unable to reproduce this myself.

commented

What do you get from this? /dump C_CVar.GetCVarBool('ActionButtonUseKeyDown')

commented

The bag addon is AdiBags.
But it happens even when i have all my addons disabled (minus Molinari of course).

commented

i also have this issue. thanks for the workaround tip, btw.

commented

Does it work for you in version 100005.97-Release?

commented

Does it work for you in version 100005.97-Release?

Yes, this release version works as expected.

commented

/dump C_CVar.GetCVarBool('ActionButtonUseKeyDown')

2023-05-11 08_29_18-World of Warcraft

commented

Alright, I'll look for the regression and let you know when I have something for you to test, as I can't replicate this issue myself.

commented

Same issue here, but using Arkinventory.
I reverted back to version 100005.97-Release and it works as intended.

commented

Thanks for testing, new release is being pushed out now.

commented

Please use this version and try to disenchant an item as you normally would with Molinari, once. It will output some stuff to the chat, please provide me with a screenshot of this.

https://github.com/p3lim-wow/Molinari/suites/12874499573/artifacts/694483322

commented

The mouse 'hand icon' turned blue, but did NOT disenchant the item on the first click.

image

commented

Thanks, that helps.

commented

Give this build a shot: https://github.com/p3lim-wow/Molinari/suites/12877062784/artifacts/694643808

Lockpicking is probably broken, I need to test that, but disenchanting should work for you now.

commented

Give this build a shot: https://github.com/p3lim-wow/Molinari/suites/12877062784/artifacts/694643808

Lockpicking is probably broken, I need to test that, but disenchanting should work for you now.

Can confirm, disenchanting is working as expected now.

commented

Great, I'll have to verify with classic and wrath, and probably fix lockpicking, then I'll have a new release out.

commented

Here's a screenshot from trying to do lockpicking on my rogue.

image

And this is using the latest build you posted up above.

commented

Sorry, I should have clarified - lockpicking in trade is probably broken, not lockpicking from bags :)

Also, doing anything in the bank has never been supported, so the fact that it activates there is a regression I'll fix.

commented

Ah ok, i just tested again and it works when the lockbox is in my bag and not in the bank.
image