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

Context menu appearance when option is disabled #3865

Closed
AMBudnik opened this Issue Nov 7, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@AMBudnik
Copy link
Contributor

AMBudnik commented Nov 7, 2016

Description

If an option in menu is temporary disabled menu is shown as a wide thin box.

Steps to reproduce

  1. Click any cells in the grid with RMB

Demo

http://jsfiddle.net/7q93jxhf/

img

Your environment

  • Version: 0.28.4
  • Browser Name and version: Chrome 54
  • Operating System: Win 10
@AMBudnik

This comment has been minimized.

Copy link
Contributor Author

AMBudnik commented Feb 27, 2017

Still replicable in v0.31.0

@AMBudnik

This comment has been minimized.

Copy link
Contributor Author

AMBudnik commented May 19, 2017

It happens for the dropdownMenu as well.

@AMBudnik

This comment has been minimized.

Copy link
Contributor Author

AMBudnik commented Jan 18, 2018

Same issue in 0.35.0

swistach added a commit that referenced this issue Feb 13, 2019

swistach added a commit that referenced this issue Feb 13, 2019

@swistach swistach self-assigned this Feb 14, 2019

@swistach swistach added this to the February 2019 milestone Feb 14, 2019

swistach added a commit that referenced this issue Feb 15, 2019

swistach added a commit that referenced this issue Feb 19, 2019

swistach added a commit that referenced this issue Feb 19, 2019

swistach added a commit that referenced this issue Feb 19, 2019

swistach added a commit that referenced this issue Feb 19, 2019

ContextMenu should notify if there is no available options. (#5788)
* WIP. Notification about no available items in contextMenu. #3865

* WIP: changed tests + removed console.log. #3865

* Added missing file. #3865

* Added test spec + translation for noItems for pl-PL. #3865

* Missing test file. #3865
@AMBudnik

This comment has been minimized.

Copy link
Contributor Author

AMBudnik commented Mar 6, 2019

Fixed in 7.0.0

@AMBudnik AMBudnik closed this Mar 6, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.