Skip to content

Commit c9d6658

Browse files
JeremyRudmancbaudouinjrpeterkos
authored
fix: School text-box dropdown in manage/questionnaires/edit missing background (#296)
* fix(questionnaire): fixed background in school autocomplete fixed the questionnaire edit page so that it was blank in the management edit page * fix(questionnaire): sperated autocomplete sass into its own file seperated the sass for the autocomplete into its own file so that manage could use it without the other sass in forms.sass * Use grays for manage backgrouind Co-authored-by: Chris Baudouin, Jr <cjb5326@rit.edu> Co-authored-by: Peter Kos <pkos91@icloud.com>
1 parent 113cd61 commit c9d6658

File tree

5 files changed

+41
-24
lines changed

5 files changed

+41
-24
lines changed

app/assets/stylesheets/application.sass

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
@import general/table
2626
@import forms/forms
2727
@import forms/confirmation
28+
@import forms/autocomplete
2829

2930
@import general/theming-editor
3031

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
2+
// jquery ui override
3+
.ui-autocomplete
4+
@include css4
5+
background: var(--input--background--focus)
6+
border: 1px solid var(--input--border-color--focus)
7+
color: var(--input--text--focus)
8+
list-style: none
9+
padding: 0
10+
.ui-menu-item div
11+
padding: 2px 5px
12+
display: block
13+
cursor: pointer
14+
overflow: hidden
15+
text-overflow: ellipsis
16+
white-space: nowrap
17+
&:hover
18+
@include css4
19+
background: var(--primary)
20+
@include css4
21+
color: var(--white)
22+
// needed for ellipsis in text to work and acts as a min-width
23+
.ui-menu
24+
width: 50px

app/assets/stylesheets/forms/_forms.sass

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -258,30 +258,6 @@ input[type=submit]
258258
margin-bottom: 3px
259259

260260

261-
// jquery ui override
262-
.ui-autocomplete
263-
@include css4
264-
background: var(--input--background--focus)
265-
border: 1px solid var(--input--border-color--focus)
266-
color: var(--input--text--focus)
267-
list-style: none
268-
padding: 0
269-
.ui-menu-item div
270-
padding: 2px 5px
271-
display: block
272-
cursor: pointer
273-
overflow: hidden
274-
text-overflow: ellipsis
275-
white-space: nowrap
276-
&:hover
277-
@include css4
278-
background: var(--primary)
279-
@include css4
280-
color: var(--white)
281-
// needed for ellipsis in text to work and acts as a min-width
282-
.ui-menu
283-
width: 50px
284-
285261
// form wizard
286262
.wizard-stage
287263
display: none

app/assets/stylesheets/manage.sass

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@
33
@import manage/selectize-fix
44
@import vendor/simplemde.min
55
@import vendor/codemirror
6+
@import general/variables
7+
@import general/css4
8+
@import manage/autocomplete
69

710
$grey-dark: #555
811
$grey-med: #999
@@ -172,6 +175,7 @@ $grey-med: #999
172175
.sidebar-mobile .sidebar-sticky
173176
height: auto
174177

178+
175179
/*
176180
* Datatable changes
177181
*/
Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
2+
@import ../forms/autocomplete
3+
4+
// jquery ui override
5+
.ui-autocomplete
6+
background: white
7+
border: 1px solid #333
8+
color: #222
9+
.ui-menu-item div
10+
&:hover
11+
background: #999
12+
color: white

0 commit comments

Comments
 (0)