Latest release breaks select when tagging and tagging-label="false" are set

[karptonite]

I don't have a plunker for it yet, but the latest release has broken select for me. Here is a quick video example:
http://quick.as/r2zvin2lb

[karptonite]

I suspect that the bug is related to this commit:
f6b6d81
activeIndex is no longer being set on mouseover, but appears to be being used.

[dimirc]

I cannot reproduce it, I tried the multi select demo but when pressing a choice it works correctly, can you setup a plunker or confirm how to reproduce it?

[karptonite]

I'll see if I can get it on a plunkr tomorrow, if I have time.

[karptonite]

For now, here is the code for the select that doesn't work:
https://gist.github.com/karptonite/3f72ec9e0f8c0d912d57

[dimirc]

Can you confirm if force-options-down is a private directive you did or where this comes from?

[karptonite]

force-options-down is something I did before the position attribute was added (it was equivalent to position="down"). I'm pretty certain it is unrelated to this bug, but I don't have the code in front of me.

[realityking]

This seeminly also broke the styling for the bootstrap theme, previously the hover effect was a blue background, now it's gray.

[karptonite]

@dimirc Actually, as @realityking says, you can even see in you multiselect demo that the bootstrap styles are broken in the current release. I suspect if you track down that bug, the bug I see in functionality (but have yet reproduced in a plunker) may be fixed.

[dimirc]

@realityking @karptonite maybe the confusion here is that with #1211 we removed the ability to set activeIndex on hover and actually only change the background (with css) on hover to give some visual feedback. To change the activeIndex was causing a useless performance hit on hover since the activeIndex was change (and a full $digest). I see more sense on having the current activeIndex change only by the UP/DOWN keys and only a hover effect when mouse over.

[karptonite]

@dimirc I'm sure that caused the problem, but I'd argue that until it works as before, the performance hit wasn't useless. :) unfortunately, I still haven't had time to reproduce the functional bug I showed in the video, but the problem with the bootstrap theme is out there on the current release, and reproducible in the plunker.

[dimirc]

@karptonite but what is the problems that is reproducible? Please explain me clearly what is expected and what is now showing so I can fix it if it's actually broken

[karptonite]

@dimirc OK, while I wasn't able to reproduce the exact problem, what I found is this:
http://plnkr.co/edit/6rLzCvmyNKorQo3CuyhA?p=preview
The new version breaks specifically when tagging is being used, and tagging-label="false".
Selecting any of the existing options in this code with a mouse is impossible (it adds an empty user instead). This code works fine on version 0.12.1.

[ghaiat]

Hello, any update on this? this is still not working
Anyone has a workaround for now?

[karptonite]

Honestly, my workaround was just not to use tagging-label="false" in my code. That was possible for me because tagging-label="false" was actually a workaround for a previous bug (I don't remember what) that was no longer an issue, as far I could tell. Sorry, not too helpful, but it may work for you, depending on your use case.