Skip to content

Commit 134e3a0

Browse files
committed
Reuse dropdown SCSS
1 parent d6a941c commit 134e3a0

File tree

3 files changed

+46
-21
lines changed

3 files changed

+46
-21
lines changed

warehouse/static/sass/blocks/_dropdown.scss

Lines changed: 42 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -28,14 +28,6 @@
2828
position: relative;
2929
display: inline-block;
3030

31-
// Remove form styling
32-
form,
33-
button {
34-
border: 0;
35-
background-color: transparent;
36-
padding: 0;
37-
}
38-
3931
&__trigger {
4032
cursor: pointer;
4133
}
@@ -49,11 +41,10 @@
4941
&__content {
5042
position: absolute;
5143
right: 0;
52-
border: 1px solid $header-border-color;
44+
margin-bottom: -4px;
5345
box-shadow: 1px 1px 2px 1px rgba(0, 0, 0, 0.05);
5446
z-index: index($z-index-scale, "dropdown");
55-
border: 1px solid rgba(255, 255, 255, 0.25);
56-
border-top: 1px solid $header-border-color;
47+
border: 1px solid $border-color;
5748
display: none;
5849
}
5950

@@ -68,11 +59,11 @@
6859
&__link,
6960
button.dropdown__link {
7061
display: block;
71-
padding: 15px 15px 15px 45px;
72-
border-bottom: 1px solid $header-border-color;
73-
background-color: $header-background-color;
62+
padding: 15px 15px 15px 15px;
63+
border-bottom: 1px solid $border-color;
64+
background-color: $white;
7465
min-width: 175px;
75-
color: $white;
66+
color: $text-color;
7667
cursor: pointer;
7768
text-align: left;
7869
position: relative;
@@ -82,8 +73,8 @@
8273
}
8374

8475
&:hover {
85-
background-color: darken($header-background-color, 1.5);
86-
color: $white;
76+
background-color: mix($border-color, #fff, 50);;
77+
color: $text-color;
8778
text-decoration: underline;
8879
}
8980

@@ -94,4 +85,38 @@
9485
top: 20px;
9586
}
9687
}
88+
89+
&--on-menu {
90+
// Remove form styling
91+
form,
92+
button {
93+
border: 0;
94+
background-color: transparent;
95+
padding: 0;
96+
}
97+
98+
.dropdown__content {
99+
border-color: $header-border-color;
100+
margin-bottom: 0;
101+
}
102+
103+
.dropdown__link,
104+
button.dropdown__link {
105+
border-bottom-color: $header-border-color;
106+
background-color: $header-background-color;
107+
color: $white;
108+
109+
&:hover {
110+
background-color: darken($header-background-color, 1.5);
111+
color: $white;
112+
}
113+
}
114+
}
115+
116+
&--with-icons {
117+
.dropdown__link,
118+
button.dropdown__link {
119+
padding: 15px 15px 15px 45px;
120+
}
121+
}
97122
}

warehouse/templates/includes/current-user-indicator.html

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

1515
{% if request.user %}
1616
<div id="user-indicator" class="horizontal-menu horizontal-menu--light horizontal-menu--tall">
17-
<div class="dropdown">
17+
<div class="dropdown dropdown--on-menu dropdown--with-icons">
1818
<button class="horizontal-menu__link dropdown__trigger">
1919
Welcome back, {{ request.user.username }}
2020
<span class="dropdown__trigger-caret">

warehouse/templates/manage/project.html

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ <h3 class="sidebar-section__title"><a href="{{ request.route_path('manage.projec
4646
<div class="package-snippet">
4747
<h1>{{ project.name }}</h1>
4848
<div class="dropdown">
49-
<button class="dropdown__trigger">
49+
<button class="dropdown__trigger button">
5050
<i class="fa fa-cog" aria-hidden="true"></i>
5151
Project Options
5252
<span class="dropdown__trigger-caret">
@@ -56,7 +56,7 @@ <h1>{{ project.name }}</h1>
5656
</button>
5757
<div class="dropdown__content">
5858
<a href="{{ request.route_path('packaging.project', name=project.normalized_name) }}" class="dropdown__link">Preview Project</a>
59-
<a href="#" class="dropdown__link">View Latest Package Info</a>
59+
<a href="#" class="dropdown__link">View Package Info</a>
6060
<a href="" class="dropdown__link">Delete Project</a>
6161
<a href="{{ request.route_path('help')}}#updating-projects" class="dropdown__link">Update Project (help)</a><!-- TODO: add help content -->
6262
</div>
@@ -86,7 +86,7 @@ <h2>Releases ({{ project.releases|length }})</h2>
8686
</td>
8787
<td>
8888
<div class="dropdown">
89-
<button class="dropdown__trigger">
89+
<button class="dropdown__trigger button">
9090
Options
9191
<span class="dropdown__trigger-caret">
9292
<i class="fa fa-caret-down" aria-hidden="true"></i>

0 commit comments

Comments
 (0)