elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.59k stars 8.21k forks source link

[Timelion] Add deprecation badge #197176

Open markov00 opened 1 week ago

markov00 commented 1 week ago

The Timelion App was deprecated in 7.10 and the standalone app was removed in 7.16. The app was removed and a Timelion visualization type was created instead, providing just a simplified textual experience.

We are now looking to deprecate completely Timelion and remove it from Kibana within the next ~2-year timespan.

The reasons for deprecating timelion are:

The task for now is only to add a deprecation badge to the Timelion icon when creating an Agg Based Visualization

elasticmachine commented 1 week ago

Pinging @elastic/kibana-visualizations (Team:Visualizations)

mbondyra commented 6 days ago

The task for now is only to add a deprecation badge to the Timelion icon when creating an Agg Based Visualization

I don't think we have the deprecation badge as something already designed, do we? Do we need a little mockup here? CC @MichaelMarcialis

MichaelMarcialis commented 3 days ago

We already have Timeline (as well as all other "aggregation-based" visualizations) housed under the "Legacy" tab in the visualization creation modal. This tab is accompanied by a tooltip that explains those visualizations are scheduled for deprecation. Do we need an additional deprecation badge specifically on the Timelion option? If so, why only for Timelion? Alternatively, should we change the "Legacy" tab to "Deprecated" to avoid fragmenting terminology for our users? Or is Timelion really the only one that that is actually officially deprecated?

Also, this request sounds like we only wish to add a badge during the creation flow. Should we also be informing users within the actual editor (whether new or existing)? If so, what terminology should we use (legacy or deprecated)?

CCing @teresaalvarezsoler.

teresaalvarezsoler commented 2 days ago

I never shared the idea that Timelion needs a different treatment than the rest of Agg based visualizations. I know it's a separate animal in our code base but it's confusing for users that may see it as part of Agg based visualizations. I never understood the benefits of this deprecation badge in isolation and why we cannot wait to do it as part of the Agg based visualizations @markov00