Skip to content

Commit 3c385de

Browse files
committed
Merge pull request #244 from panicbit/mobile-ui
Fix menu dropdown and add searchbar to mobile layout
2 parents a729d36 + 8da6840 commit 3c385de

File tree

2 files changed

+38
-35
lines changed

2 files changed

+38
-35
lines changed

app/styles/app.scss

Lines changed: 25 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -72,27 +72,6 @@ body {
7272
right: 0;
7373
left: auto;
7474
min-width: 200px;
75-
76-
li.search {
77-
padding: 5px;
78-
input.search {
79-
margin: 0;
80-
border: 1px solid black;
81-
}
82-
}
83-
}
84-
85-
input.search {
86-
font-size: 90%;
87-
border: none;
88-
color: black;
89-
outline: 0;
90-
margin-left: 30px;
91-
padding: 5px 5px 5px 25px;
92-
background-image: url('/assets/search.png');
93-
background-repeat: no-repeat;
94-
background-position: 6px 6px;
95-
@include border-radius(15px);
9675
}
9776

9877
button {
@@ -126,6 +105,31 @@ body {
126105
}
127106
}
128107

108+
#header input.search, #mobile-search input.search {
109+
font-size: 90%;
110+
border: none;
111+
color: black;
112+
outline: 0;
113+
margin-left: 30px;
114+
padding: 5px 5px 5px 25px;
115+
background-image: url('/assets/search.png');
116+
background-repeat: no-repeat;
117+
background-position: 6px 6px;
118+
@include border-radius(15px);
119+
}
120+
121+
#mobile-search {
122+
display: none;
123+
margin-bottom: 10px;
124+
input.search {
125+
width: 100%;
126+
margin: 0;
127+
}
128+
@media only screen and (max-width: 820px) {
129+
display: block;
130+
}
131+
}
132+
129133
span.small {
130134
color: $main-color-light;
131135
font-size: 80%;

app/templates/application.hbs

Lines changed: 13 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -48,14 +48,12 @@
4848
</div>
4949

5050
<div class='menu'>
51-
<div class='dropdown-container'>
52-
<button {{action 'toggleUserOptions'}}
53-
class='{{if showUserOptions "active"}} dropdown'>
51+
{{#rl-dropdown-container class='dropdown-container'}}
52+
{{#rl-dropdown-toggle class='dropdown'}}
5453
Menu
5554
<span class='arrow'></span>
56-
</button>
57-
<ul id='current-user-links'
58-
class='{{if showUserOptions "open"}} dropdown'>
55+
{{/rl-dropdown-toggle}}
56+
{{#rl-dropdown tagName='ul' id='current-user-links' class='dropdown' closeOnChildClick='a:link'}}
5957
<li>{{#link-to "crates"}}Browse All Crates{{/link-to}}</li>
6058
{{#if session.currentUser}}
6159
<li>{{#link-to 'dashboard'}}Dashboard{{/link-to}}</li>
@@ -64,20 +62,21 @@
6462
{{else}}
6563
<li>{{#link-to "login"}}Log in with GitHub{{/link-to}}</li>
6664
{{/if}}
67-
<li class='search'>
68-
{{input type="text" class="search"
69-
placeholder="Search"
70-
value=search
71-
enter="search"}}
72-
</li>
73-
</ul>
74-
</div>
65+
{{/rl-dropdown}}
66+
{{/rl-dropdown-container}}
7567
</div>
7668

7769
<div class='links'>
7870
</div>
7971
</div>
8072

73+
<form id='mobile-search' class='search' action='/search' {{ action "search" on="submit" }} >
74+
{{input type="text" class="search" name="q"
75+
placeholder="Search"
76+
value=search
77+
tabindex="1"}}
78+
</form>
79+
8180
<div id="main" class='inner-content'>
8281
<p id='flash' class={{if flashError "shown"}}>{{flashError}}</p>
8382

0 commit comments

Comments
 (0)