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

Add possibility to hide "panel spy" #154

Closed
wants to merge 9 commits into from

Conversation

jsm-oxa
Copy link

@jsm-oxa jsm-oxa commented Jun 11, 2013

"Collapsable" enables the possibility to collapse a panel using "collapse" parameter in the dashboard configuration.

I just did the same : "spyable" enable the possibility to "spy" a panel. By setting spyable "true", the "panel spy" part of the editor is hide.

Because something you want to allow edition but not spy. With these changes, it will not be possible to enable spy through the editor.

work as collapse/collapsable
work as collapse/collapsable
work as collapse/collapsable
work as collapse/collapsable
work as collapse/collapsable
work as collapse/collapsable
work as collapse/collapsable
work as collapse/collapsable
work as collapse/collapsable
@rashidkpc
Copy link
Contributor

I think I'd rather not go down the path of having hidden options to disable specific options in the editor.

@rashidkpc rashidkpc closed this Jun 21, 2013
w33ble added a commit to w33ble/kibana that referenced this pull request Sep 13, 2018
* chore: make labeled select generic component

decouple it from the extended input template so it can be reused

* feat: add container style argType UI

* feat: add border radius to border form

* chore: separate background color and image

also, don't automatically append the 'px' on padding and border radius

* fix: make labeled select value optional

* feat: add container appearance form

* fix: correctly build border property

also make the value argument optional

* feat: set the container ast

also set the expected types on css values to string instead of number, since we're using the px values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants