popup.js: delMod('visible') broken

[aliosv]

https://github.com/bem/bem-components/blob/v2/common.blocks/popup/popup.js#L558 zIndexes is undefined by default PR #966 /cc @tadatuta, @veged

[dfilatov]

Could you provide sequence of your actions?

[aliosv]

https://github.com/aliosv/project-stub/tree/bem-components.965

[dfilatov]

?

[aliosv]

there is demonstration of bug, build it to see

[dfilatov]

I've cloned your repo, pointed browser to desktop.bundles/index/index.html and haven't seen anything about this bug.

[aliosv]

check browser console.

[dfilatov]

I see nothing interesting in browser console.

[dfilatov]

Could you provide your actions step by step?

[aliosv]

git clone git@github.com:aliosv/project-stub.git -b bem-components.965 npm i bem server .../index.html

win 7, checked in latest ff, chrome, yabrowser

[dfilatov]

I've understood. You are trying to set visible : true modifier from bemjson. That case isn't supported right now at all, because only js can properly position popup. We should add notes about that in documentation.

[aliosv]

ok, thanks

[innabelaya]

добавила информацию в блок popup(в рамках #944), спасибо!