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

enh(ui) : better delimitation of menu #7253

Closed
wants to merge 1 commit into from

Conversation

EdytaSki
Copy link
Contributor

@EdytaSki EdytaSki commented Mar 6, 2019

Pull Request Template

Description

Please include a short resume of the changes and what is the purpose of PR. Any relevant information should be added to help:

You should see a grey border on the right side of the left menu.

Type of change

Please delete options that are not relevant.

  • New functionality (non-breaking change)

Target serie

You should see a grey border on the right side of the left menu.

  • 18.10.x

How this pull request can be tested ?

Please describe the procedure to verify that the goal of the PR is matched. Provide clear instructions so that it can be correctly tested.

Any relevant details of the configuration to perform the test should be added.

Checklist

Community contributors & Centreon team
  • I followed the coding style guidelines provided by Centreon

* updating the release note results in adding the pull request id and description at the end of the file. Product Managers will rework it for the release. Release notes can be found here.

Centreon team only
  • I have made sure that acceptance tests related to the story are successful (local and CI)

@EdytaSki EdytaSki requested review from NikolaL87 and kduret March 6, 2019 09:37
sc979
sc979 previously approved these changes Mar 6, 2019
@sc979 sc979 dismissed their stale review March 6, 2019 10:27

this PR isn't properly rebased. As it merges the code from another PR when it shouldn't : https://github.com/centreon/centreon/pull/7251

@@ -395,7 +396,7 @@ a {
font-size: 0.8rem;
background-color: $white-color;
display: block;
padding: 3px 10px 3px 47px;
padding: 3px 10px 3px 56px;
Copy link
Contributor

Choose a reason for hiding this comment

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

this modification is from another PR not yet approved by the QA : https://github.com/centreon/centreon/pull/7251

// color: $white-color;
// background-color: #0a78a5;
// }
padding: 8px 10px 8px 56px;
Copy link
Contributor

Choose a reason for hiding this comment

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

this modification is from another PR not yet approved by the QA : https://github.com/centreon/centreon/pull/7251

@EdytaSki EdytaSki closed this Mar 6, 2019
@EdytaSki EdytaSki deleted the MON-3535-menu-delimitation-18-10 branch March 6, 2019 13:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants