-
Notifications
You must be signed in to change notification settings - Fork 0
/
feedback.txt
73 lines (46 loc) · 2.5 KB
/
feedback.txt
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
Capstone Project Review Checklist
Did the student pass this evaluation?
yes
Is this project ready for official scoring?
yes
Mobile-first Design
yes
Landing page
yes
Quality Content
yes
Login
no
Concept score
Functionality score
Other feedback
Hi Fu Lin,
Thanks for submitting your app for review. You've done great work and I commend your efforts!
Overall, I have recommended that your app be approved.
I've included detailed feedback and suggestions below, along with screenshots, which I hope will be helpful.
Your screenshots and a downloadable version of this feedback are available at https://www.dropbox.com/sh/v32shq7nkf7hbp0/AACrUsaH4m-B82DkBZe-GGiCa?dl=0. Please download them within one week, as I may remove them after that time.
If you'd like to discuss anything in more detail, you're welcome to get in touch with me via email at [email protected], or on Slack at @joshua_harris.
Kind regards, Joshua Harris, Capstone Grader
Concept
Concept is clear from the feature descriptions. Recommend adding a basic one-sentence tagline below the application title which succinctly describes the benefit a user will get from using the application.
Functionality
I use a traffic light system for functionality review. G stands for green, meaning the feature is working well. Y stands for yellow, meaning the feature is mostly working but needs some attention. R stands for red, meaning the feature appears to be broken.
[G] Search for Jobs
Recommend implementing loading indicator for any functionality like this that can take some time to complete (especially if your Heroku application has gone to sleep), and keeping the form field populated with the user's search until the search results come up so it's clear the application is working.
[G] View Listings
[G] Add and Remove Skills
Mobile-First Design
Mobile-First Design: Landing Page
Recommend decreasing whitespace on mobile to bring content closer to the top of the screen. (1)
Mobile-First Design: Application
All screens respond well on all tested screen sizes and devices. Great job!
Design/UX
Design/UX: Landing Page
Landing page has good information, but styles are sparse. Recommend adding some variation in color and layout to add visual interest. See existing landing pages for examples.
Simple
Duolingo
InVision
Design/UX: Application
Recommend increasing spacing and font size of listings to make them more readable. See http://uxarchive.com, http://mobile-patterns.com, and http://dribbble.com for design inspiration.
Did the student pass this evaluation?
yes