exoplatform / platform-ui

Platform Skin
GNU Lesser General Public License v3.0
2 stars 35 forks source link

Updated lesscss-maven-plugin version and @navbarHeight #52

Closed Robinyo closed 7 years ago

Robinyo commented 8 years ago

Updated @navbarHeight: 41px; // 40px; to match uiToolbarContainer/Style.less Updated height: uiToolbarContainer/Style.less to use @navbarHeight;

thomasdelhomenie commented 8 years ago

Thanks a lot for this contribution @Robinyo ! I added some comments, please consider them.

Robinyo commented 8 years ago

Updated as per your feedback.

thomasdelhomenie commented 8 years ago

The PR looks good to me. Thanks @Robinyo ! In order to allow us to accept definitely your PR, you have to fill our Contributor Agreement please : http://community.exoplatform.org/rest/contents/download/collaboration/Users/t___/th___/tho___/thomas_delhomenie/Public/eXo_Platform_ContributorAgreement_EN_v1.2.pdf

thomasdelhomenie commented 8 years ago

Hi @Robinyo would you mind signing the CLA please, so we can accept your fix ?