Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

change side bar position for anomly detection plugin #335

Merged
merged 2 commits into from
Nov 25, 2020

Conversation

zhyuanqi
Copy link
Contributor

Issue #, if available:

Description of changes:
Change position of Anomaly detection plugin in side bar.
The order is designed by UX designer and in step of 1000(https://github.com/elastic/kibana/blob/master/docs/development/core/public/kibana-plugin-core-public.appcategory.md).

image (4)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@@ -40,8 +40,13 @@ export class AnomalyDetectionKibanaPlugin
core.application.register({
id: 'opendistro-anomaly-detection-kibana',
title: 'Anomaly Detection',
category: DEFAULT_APP_CATEGORIES.kibana,
Copy link
Contributor

Choose a reason for hiding this comment

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

thanks for the change, can you also remove the unused import DEFAULT_APP_CATEGORIES at line 22 after the only usage is removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure. Thanks for reminding.

Copy link
Contributor

@ohltyler ohltyler left a comment

Choose a reason for hiding this comment

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

Thanks for the change!

@seraphjiang seraphjiang merged commit 6d022b0 into opendistro-for-elasticsearch:master Nov 25, 2020
@ohltyler ohltyler added the enhancement Enhance current feature for better performance, user experience, etc label Nov 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Enhance current feature for better performance, user experience, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants