Skip to content

Commit ef1aaf9

Browse files
committed
Form is almost done i think
1 parent 30ff53e commit ef1aaf9

File tree

5 files changed

+47
-26
lines changed

5 files changed

+47
-26
lines changed

conditional/blueprints/major_project_submission.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import boto3
55

66
from conditional.models.models import MajorProject, MajorProjectSkill
7+
from conditional.util.user_dict import user_dict_is_eval_director
78
from config import AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY
89
from flask import Blueprint
910
from flask import request
@@ -65,7 +66,7 @@ def display_major_project(user_dict=None):
6566
"time_spent": p.timeSpent,
6667
"skills": p.skills,
6768
"desc": p.description,
68-
"links": p.links,
69+
"links": list(filter(None, p.links.split("\n"))),
6970
"status": p.status,
7071
"is_owner": bool(user_dict["username"] == p.uid)
7172
}
@@ -208,7 +209,7 @@ def submit_major_project(user_dict=None):
208209
def major_project_review(user_dict=None):
209210
log = logger.new(request=request, auth_dict=user_dict)
210211

211-
if not ldap_is_eval_director(user_dict["account"]):
212+
if not user_dict_is_eval_director(user_dict["account"]):
212213
return redirect("/dashboard", code=302)
213214

214215
post_data = request.get_json()
@@ -236,7 +237,7 @@ def major_project_delete(pid, user_dict=None):
236237
major_project = MajorProject.query.filter(MajorProject.id == pid).first()
237238
creator = major_project.uid
238239

239-
if creator == user_dict["username"] or ldap_is_eval_director(user_dict["account"]):
240+
if creator == user_dict["username"] or user_dict_is_eval_director(user_dict["account"]):
240241
MajorProject.query.filter(MajorProject.id == pid).delete()
241242

242243
db.session.flush()

conditional/templates/major_project_submission.html

Lines changed: 22 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ <h3 class="mb-4">Major Project Form</h3>
125125
id="links"
126126
name="links"
127127
rows="2"
128-
class="form-control"
128+
class="form-control form-textarea"
129129
placeholder="If you have links (GitHub, YouTube, etc.), put them here!"
130130
></textarea>
131131
</div>
@@ -184,15 +184,15 @@ <h3 class="mb-4">All Major Projects</h3>
184184

185185
{% for p in major_projects %}
186186

187-
<div class="card shadow-lg bg-white mb-4">
187+
<div class="card shadow-lg proj-listing bg-white mb-4">
188188
<div class="card-body mb-extra px-5">
189189

190190
<div class="container-fluid px-0 px-extra">
191191

192192
<!-- HEADER ROW -->
193193
<div class="row d-flex align-items-center">
194194
<!-- Title -->
195-
<div class="col-md-11" style="margin-bottom: 1em;">
195+
<div class="col-md-10" style="margin-bottom: 1em;">
196196
<h4 class="font-weight-bold mb-1">
197197
<strong>{{p['proj_name']}}</strong>
198198
</h4>
@@ -209,10 +209,10 @@ <h4 class="font-weight-bold mb-1">
209209
</div>
210210

211211
<!-- Status -->
212-
<div class="col-md-1">
212+
<div class="col-md-2">
213213
{% if is_eval_director %}
214214
<div
215-
class="btn-group btn-group-sm"
215+
class="btn-group btn-group-md"
216216
data-module="majorProjectStatus"
217217
data-id="{{p['id']}}"
218218
>
@@ -281,23 +281,31 @@ <h5><strong>Time Commitment</strong></h5>
281281
<div class="row">
282282
<div class="col-md-6 mb-3">
283283
<h5><strong>Skills</strong></h5>
284-
{% for s in p['skills'] %}
285-
<span class="h6 skill-display p-2 mr-1 mb-1">
286-
{{s}}
287-
</span>
288-
{% endfor %}
284+
<div class="skill-group">
285+
{% for s in p['skills'] %}
286+
<span class="h6 skill-display p-2 mr-1 mb-1">
287+
{{s}}
288+
</span>
289+
{% endfor %}
290+
</div>
289291
</div>
290292

291293
<div class="col-md-6 mb-3">
292-
<strong>Links</strong>
293-
<p class="mb-0">{{p['links']}}</p>
294+
<h5><strong>Links</strong></h5>
295+
{% for l in p['links'] %}
296+
{% set href = l %}
297+
{% if not href.startswith('http://') and not href.startswith('https://') %}
298+
{% set href = 'https://' + href %}
299+
{% endif %}
300+
<p class="line-short h6"><a href="{{ href }}" class="mb-0">{{ href }}</a></p>
301+
{% endfor %}
294302
</div>
295303
</div>
296304

297305
<div class="row">
298306
<div class="col-md-9 ">
299-
<strong>Description</strong>
300-
<p class="mb-0">{{p['desc']}}</p>
307+
<h5><strong>Description</strong></h5>
308+
<p class="mb-0 h6 line-med">{{p['desc']}}</p>
301309
</div>
302310
</div>
303311
</div>

frontend/javascript/modules/majorProjectForm.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,7 @@ export default class MajorProjectForm {
5151
let skills = [];
5252

5353
Array.from(this.form.getElementsByClassName('skill-tag')).forEach(tag => skills.push(tag.firstChild.data))
54-
55-
console.log("Links: " + this.form.querySelector('textarea[name=links]').value);
56-
54+
5755
let payload = {
5856
projectName: this.form.querySelector('input[name=name]').value,
5957
projectTldr: this.form.querySelector('input[name=tldr]').value,

frontend/javascript/modules/majorProjectStatus.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ export default class MajorProjectStatus {
4444
$(dashboardContainer).hide();
4545
} else {
4646
// Major projects page button
47-
$(this.control.closest(".panel")).fadeOut();
47+
$(this.control.closest(".card")).fadeOut();
4848
}
4949
});
5050
} else {

frontend/stylesheets/pages/_major-project.scss

Lines changed: 19 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@
5252

5353
.proj-listing {
5454
background-color: #fff;
55-
padding: 0.5em;
55+
box-shadow: 1px 1px 2px grey;
5656
}
5757

5858
.rounded-circle {
@@ -67,10 +67,10 @@
6767
.skill-tag {
6868
display: block;
6969
float: left;
70-
background: #d979e3;
70+
background: #b0197e;
7171
padding: 4px 30px 4px 8px;
7272
margin: 2px 3px;
73-
color: #444;
73+
color: #fff;
7474
border-radius: 5px;
7575
transition: .5s all;
7676
}
@@ -88,14 +88,28 @@
8888
.skill-display {
8989
display: block;
9090
float: left;
91-
background: #d979e3;
91+
background: #b0197e;
9292
padding: 4px 8px 4px 8px;
9393
margin: 2px 3px;
94-
color: #444;
94+
color: #fff;
95+
font-weight: 600;
9596
border-radius: 5px;
9697
transition: .5s all;
9798
}
9899

99100
.skill-tag:hover {
100101
box-shadow: 0 4px 8px 0 rgba(0, 0, 0, 0.2), 0 6px 20px 0 rgba(0, 0, 0, 0.19);
102+
}
103+
104+
.skill-group {
105+
display: flex;
106+
flex-wrap: wrap;
107+
}
108+
109+
.line-short {
110+
line-height: 0.5em;
111+
}
112+
113+
.line-med {
114+
line-height: 1.5em;
101115
}

0 commit comments

Comments
 (0)