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

Navbar hide flick #1558

Merged
merged 109 commits into from
Aug 16, 2019
Merged

Navbar hide flick #1558

merged 109 commits into from
Aug 16, 2019

Conversation

adri9valle
Copy link
Contributor

Hi team,

This PR makes that the wz-menu directive does not hide while changing between views. Also, it adds the Wazuh logo into the Kibana navbar.

Solves: #1502

Regards,

@adri9valle adri9valle marked this pull request as ready for review July 30, 2019 07:33
@adri9valle adri9valle changed the base branch from 3.10-7.1 to 3.10-7.2 July 30, 2019 08:39
@jesusgn90
Copy link
Contributor

Here are some points that must be corrected:

@juankaromo
Copy link
Contributor

juankaromo commented Aug 14, 2019

Hi @jesusgn90 I'm going to clarify the points above:

  • Indentation, spaces, and the IDE linter It seems that there is a problem with my prettier version. It's updated, but he keeps doing weird things.

  • Style The main change is to adapt the menu to be outside of the main container, and, on the other hand, keep it fixed, to improve the UX. Maybe this needs another review on more browsers.

  • Logos The Wazuh logos have been updated to adapt them to the new corporative blue color.

  • Removed image This image was added to show it when 'No SCA results' message was shown. But it has been deprecated.

  • Empty value for the very first time That was fixed here 0f4cbeb

  • Using EUI classes It was removed finally.

  • 500ms is too big The interval time has been reduced to 100ms 0f4cbeb

  • Commented code block and removing wazuh-version.json Fixed here 0f4cbeb

@juankaromo juankaromo changed the base branch from 3.10-7.2 to 3.10-7.3 August 16, 2019 09:23
@jesusgn90 jesusgn90 merged commit 84905b3 into 3.10-7.3 Aug 16, 2019
@jesusgn90 jesusgn90 deleted the navbar-hide-flick branch August 16, 2019 11:55
@jesusgn90 jesusgn90 restored the navbar-hide-flick branch August 16, 2019 12:29
jesusgn90 pushed a commit that referenced this pull request Aug 16, 2019
jesusgn90 pushed a commit that referenced this pull request Aug 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Enhancement issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants