improve navbar collapse for mobile views
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
This commit is contained in:
parent
a74dfa6e74
commit
01f6ae2351
3 changed files with 10 additions and 13 deletions
|
@ -177,6 +177,10 @@ $ ->
|
|||
$(@).next('table').show()
|
||||
$(@).remove()
|
||||
|
||||
$('.navbar-toggle').on 'click', ->
|
||||
$('.header-content .title').toggle()
|
||||
$('.header-content .navbar-collapse').toggle()
|
||||
|
||||
# Show/hide comments on diff
|
||||
$("body").on "click", ".js-toggle-diff-comments", (e) ->
|
||||
$(@).toggleClass('active')
|
||||
|
|
|
@ -42,15 +42,12 @@ header {
|
|||
}
|
||||
}
|
||||
|
||||
.nav {
|
||||
float: right;
|
||||
margin-right: 0;
|
||||
}
|
||||
|
||||
.navbar-toggle {
|
||||
color: #666;
|
||||
margin: 0;
|
||||
border-radius: 0;
|
||||
position: absolute;
|
||||
right: 2px;
|
||||
|
||||
&:hover {
|
||||
background-color: #EEE;
|
||||
|
@ -195,7 +192,8 @@ header {
|
|||
}
|
||||
|
||||
.navbar-nav {
|
||||
margin: 5px 0;
|
||||
margin: 0px;
|
||||
float: none !important;
|
||||
|
||||
.visible-xs, .visable-sm {
|
||||
display: table-cell !important;
|
||||
|
@ -205,11 +203,6 @@ header {
|
|||
li {
|
||||
display: table-cell;
|
||||
width: 1%;
|
||||
|
||||
a {
|
||||
text-align: center;
|
||||
font-size: 18px !important;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -8,12 +8,12 @@
|
|||
%h1.title
|
||||
= title
|
||||
|
||||
%button.navbar-toggle{type: 'button', data: {target: '.navbar-collapse', toggle: 'collapse'}}
|
||||
%button.navbar-toggle
|
||||
%span.sr-only Toggle navigation
|
||||
= icon('bars')
|
||||
|
||||
.navbar-collapse.collapse
|
||||
%ul.nav.navbar-nav
|
||||
%ul.nav.navbar-nav.pull-right
|
||||
%li.hidden-sm.hidden-xs
|
||||
= render 'layouts/search'
|
||||
%li.visible-sm.visible-xs
|
||||
|
|
Loading…
Reference in a new issue