Skip to content

Commit 6b4c977

Browse files
committed
update less files based on feedback
1 parent 0a1f798 commit 6b4c977

2 files changed

Lines changed: 24 additions & 33 deletions

File tree

plugins/CoreHome/stylesheets/layout.less

Lines changed: 23 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ nav {
163163
.layoutWithSidebar--hasSidebar {
164164
display: flex;
165165
flex-direction: row;
166-
align-items: flex-start;
166+
align-items: stretch;
167167

168168
#secondNavBar {
169169
flex: 0 0 224px;
@@ -231,7 +231,8 @@ nav {
231231

232232
#secondNavBar {
233233
width: 224px;
234-
height: fit-content;
234+
min-height: 100vh;
235+
height: auto;
235236
flex: none;
236237
background-color: @theme-color-menu-contrast-background;
237238
box-shadow: none !important;
@@ -270,12 +271,16 @@ nav {
270271
align-items: center;
271272
.font-default(14px, 20px);
272273
color: @left-menu-item-color;
273-
274-
&:focus, &:hover {
275-
border-radius: 8px;
274+
border-radius: 8px;
275+
cursor: pointer;
276+
text-decoration: none;
277+
&:hover, &:focus {
278+
outline: 0;
276279
}
277-
&:focus {
278-
outline: 2px solid;
280+
&:focus-visible {
281+
color: @left-menu-item-color;
282+
background-color: @theme-color-background-base;
283+
outline: 1px solid @left-menu-item-hover-text-color;
279284
}
280285
&:hover {
281286
background-color: @theme-color-background-base;
@@ -318,38 +323,28 @@ nav {
318323
}
319324

320325
> .item {
321-
cursor: default;
322326
font-weight: 600;
323-
&:hover, &:focus {
324-
text-decoration: none;
325-
}
326327
}
327328

328-
//Selects the subcategory of the menu item
329+
//Selects the submenu of the menu item
329330
> ul {
330331
position: relative;
331-
332+
// Submenu element
332333
li {
333334
position: relative;
334335
width: 100%;
335336
margin-left: auto;
336337
margin-right: auto;
337338
.item {
338339
.font-default(13px, 16px);
339-
text-decoration: none;
340-
transition: background-color 200ms linear;
341340
padding: 10px 22px 10px 36px;
342-
&:focus {
343-
text-decoration: none;
344-
color: @theme-color-menu-contrast-textActive;
345-
background-color: @theme-color-background-base;
346-
border-radius: 8px;
347-
}
341+
height: 36px;
342+
width: 98%;
348343
}
349344
&.active {
350345
> .menuDropdown,
351346
> .item {
352-
color: @left-menu-item-hover;
347+
color: @left-menu-item-hover-text-color;
353348
background-color: @theme-color-background-base;
354349
text-decoration: none;
355350
border-radius: 8px;
@@ -367,9 +362,9 @@ nav {
367362
}
368363
li.active, li:hover {
369364
>.item {
370-
color: @left-menu-item-hover;
365+
color: @left-menu-item-hover-text-color;
371366
.menu-icon {
372-
color: @left-menu-item-hover;
367+
color: @left-menu-item-hover-text-color;
373368
}
374369
}
375370
}
@@ -386,6 +381,9 @@ nav {
386381

387382
.search {
388383
margin: 15px 16px 10px 17px;
384+
input {
385+
font-size: 14px;
386+
}
389387
}
390388

391389
.item {
@@ -407,7 +405,7 @@ nav {
407405
justify-content: space-between;
408406

409407
&:hover, &:focus {
410-
color: @left-menu-item-hover;
408+
color: @left-menu-item-hover-text-color;
411409
}
412410

413411
&:after {
@@ -440,16 +438,9 @@ nav {
440438
}
441439

442440
&.Menu--admin .navbar > li {
443-
> .item {
444-
padding: 14px 21px 6px 19px;
445-
}
446441
.item .icon-arrow-right:before {
447442
content: "\e63b";
448443
}
449-
.menu-icon {
450-
padding-right: 13px;
451-
display: inline;
452-
}
453444
}
454445

455446
.collapsible {

plugins/Morpheus/stylesheets/base/colors.less

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@
8989

9090
@left-menu-item-color: #646464; // This should be a theme colour
9191
@left-menu-item-icon-color: #37474F;
92-
@left-menu-item-hover: #1976D2;
92+
@left-menu-item-hover-text-color: #1976D2;
9393
/*
9494
Qualitative data color series inspired from colorbrewer2.org/
9595
next ones could be: #cab2d6 #ffff99 # #b2df8a

0 commit comments

Comments
 (0)