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

[Input Controls] Hide ability to create legacy input controls #156455

Conversation

Heenawter
Copy link
Contributor

@Heenawter Heenawter commented May 2, 2023

Closes #150639

Note
The attached issue mentions possibly preventing legacy input control panels from being cloned - while I understand the reasoning for this (if we allow cloning, aren't we essentially still giving users a roundabout way to create new legacy input controls?), I think that the primary goal of this should be to make legacy input controls more difficult to access, not necessarily to prevent users from creating legacy input controls entirely.

After all, if we went to prevent any new legacy input controls from being created, wouldn't we also have to prevent users from cloning existing dashboards that have at least one legacy input control? I think that simply hiding them from the creation menus is a good enough beginning step until we can finally remove them for good.

Summary

This PR takes another step forward in the deprecation process of the legacy input controls by hiding them from the visualization wizard + the visualization creation menu by marking the input_control_vis type as hidden.

Visualize Wizard

Before After
image image

Note
As a follow up, the visualize team will be removing the visualize:enableLabs setting, since it is used exclusively to hide the controls from this wizard; however, because this PR hides them from this wizard via the hidden property, this setting is now obsolete.

Creation Menu in Dashboard

Before After
image image

Checklist

For maintainers

@Heenawter Heenawter added release_note:deprecation Feature:Input Control Input controls visualization Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas loe:medium Medium Level of Effort impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. v8.9.0 labels May 2, 2023
@Heenawter Heenawter self-assigned this May 2, 2023
@Heenawter Heenawter force-pushed the remove-creation-deprecated-controls_2023-05-02 branch from 320a8d8 to e659882 Compare May 5, 2023 17:35
@Heenawter Heenawter added loe:small Small Level of Effort loe:medium Medium Level of Effort and removed loe:medium Medium Level of Effort loe:small Small Level of Effort labels May 5, 2023
@Heenawter Heenawter force-pushed the remove-creation-deprecated-controls_2023-05-02 branch from 034b139 to f08c2a7 Compare May 5, 2023 20:18
@Heenawter Heenawter force-pushed the remove-creation-deprecated-controls_2023-05-02 branch from f08c2a7 to 218cf23 Compare May 5, 2023 20:35
@Heenawter Heenawter added the backport:skip This commit does not require backporting label May 5, 2023
@Heenawter Heenawter marked this pull request as ready for review May 5, 2023 20:38
@Heenawter Heenawter requested review from a team as code owners May 5, 2023 20:38
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

Copy link
Contributor

@stratoula stratoula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Visualizations changes LGTM!

Copy link
Contributor

@stratoula stratoula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Visualizations changes LGTM!

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
visualizations 775 776 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
visualizations 260.2KB 260.2KB +11.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
inputControlVis 5.5KB 5.5KB +10.0B
Unknown metric groups

API count

id before after diff
visualizations 805 806 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
securitySolution 399 403 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
securitySolution 479 483 +4
total +6

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @Heenawter

Copy link
Contributor

@ThomThomson ThomThomson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looked through the code and tested this locally and everything LGTM!

It's important to note that you can still access the Legacy Input Controls by navigating directly to the URL {baseURL}/app/visualize#/create?type=input_control_vis, which I'd consider a good thing seeing as we haven't yet covered the entirety of the feature set from the legacy controls.

@@ -111,6 +111,63 @@
"version": "WzE4LDFd"
}


{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Smart to add the Input Control to kbn archiver so that the test doesn't have to create one. Nice!

@Heenawter Heenawter merged commit ac84798 into elastic:main May 10, 2023
@Heenawter Heenawter deleted the remove-creation-deprecated-controls_2023-05-02 branch May 10, 2023 14:55
@gaby
Copy link

gaby commented Jul 28, 2023

@Heenawter What is replacing the legacy input controls? All our dashboards use this visualization to allow users to filter data based on multiple predefined fields.

@Heenawter
Copy link
Contributor Author

Heenawter commented Jul 28, 2023

@gaby They are being replaced by the new input controls, which are much more powerful - you can find the documentation for these here :)

@gaby
Copy link

gaby commented Jul 29, 2023

@gaby They are being replaced by the new input controls, which are much more powerful - you can find the documentation for these here :)

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting Feature:Input Control Input controls visualization impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:small Small Level of Effort release_note:deprecation Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v8.9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Input Controls] Disallow Creation of Legacy Input Controls
6 participants