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.

Sign up for GitHub

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

Jump to bottom

Adapt the breadcrumb to the new menu and remove old menu #5814

Closed
3 tasks done
Desvelao opened this issue Aug 25, 2023 · 3 comments · Fixed by #5817
Closed
3 tasks done

Adapt the breadcrumb to the new menu and remove old menu #5814

Desvelao opened this issue Aug 25, 2023 · 3 comments · Fixed by #5817
Assignees
@yenienserrano
Labels
level/task Task issue type/enhancement Enhancement issue

Comments

@Desvelao
Copy link
Member

Desvelao commented Aug 25, 2023 •
edited by yenienserrano
Loading

Description

The section of the applications will be moved to the side menu of the platform. We need to adapt the breadcrumb to the new applications.

The root item of the breadcrumb should be the name of the application.

Screenshots

image

Tasks

  • Adapt the breadcrumb
  • Change the redirections
  • Remove legacy menu
@Desvelao Desvelao added type/enhancement Enhancement issue level/task Task issue labels Aug 25, 2023
@Desvelao Desvelao mentioned this issue Aug 25, 2023
Closed
3 tasks
@wazuhci wazuhci moved this to In progress in Release 4.8.0 Aug 25, 2023
@yenienserrano yenienserrano mentioned this issue Aug 25, 2023
Merged
6 tasks
@yenienserrano yenienserrano linked a pull request Aug 25, 2023 that will close this issue
5814 adapt the breadcrum to the new menu #5817
Merged
6 tasks
@yenienserrano yenienserrano changed the title Adapt the breadcrum to the new menu Adapt the breadcrum to the new menu and remove old menu Aug 28, 2023
@yenienserrano
Copy link
Member

Two types of breadcrumb renderings were found, one that has an icon with a tooltip with a description and the other has no such icon. We would have to decide to leave one for all modules, if all modules have the icon or not.

@Desvelao Desvelao changed the title Adapt the breadcrum to the new menu and remove old menu Adapt the breadcrumb to the new menu and remove old menu Aug 29, 2023
@AlexRuiz7 AlexRuiz7 mentioned this issue Aug 30, 2023
Closed
19 tasks
@wazuhci wazuhci moved this from In progress to Pending review in Release 4.8.0 Aug 31, 2023
@yenienserrano
Copy link
Member

We have decided to remove the breadcrumbs icon since other applications do not have it, if we want to add it in the future we would have to add it in all the views.

@Desvelao Desvelao closed this as completed Sep 7, 2023
@wazuhci wazuhci moved this from Pending review to Done in Release 4.8.0 Sep 7, 2023
@Desvelao
Copy link
Member Author

I reopened the issue due to changes in the app names, so we will change the application ID too.

@Desvelao Desvelao reopened this Sep 18, 2023
@wazuhci wazuhci moved this from Done to Triage in Release 4.8.0 Sep 18, 2023
@wazuhci wazuhci moved this from Triage to Done in Release 4.8.0 Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yenienserrano yenienserrano

Labels
level/task Task issue type/enhancement Enhancement issue
Projects
No open projects
Release 4.8.0
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

5814 adapt the breadcrum to the new menu wazuh/wazuh-dashboard-plugins
2 participants
@Desvelao @yenienserrano