Skip to content

Latest commit

 

History

History
105 lines (99 loc) · 4.87 KB

Project.org

File metadata and controls

105 lines (99 loc) · 4.87 KB

The Indie Quest (Final project WEBWEU01)

🗓️ Plan

30 hours have been allocated to this project.

WeekTime (h)ActivityDelivery
179Create working site
1Customer feedbackFirst
184Adjust layout from feedback
6Find and fix bugs
3Test accessibility and fix issues
192Customer feedbackSecond
5Review checklist and deliverFinal

📋 Checklist

Layout & Design

[X] Montserrat font (With fallback to system sans-serif)

Pages

[X] Landing page

[X] One YouTube video

[X] Curriculum page

[X] Find us page

[X] Map

[X] Application

[X] Form which validates obligatory fields

[X] Navbar

[X] Logo

[X] Responsive design

Accessibility

[X] All images must have alt text

[X] Declare language

HTML

[X] Semantic HTML

[X] Validated with W3 Validator

[X] Well documented HTML & CSS

Git

[X] At least 10 commits

[X] Two weeks

🐛 Bugs

  • [X] Apply page always shows citizenship input field on reload
  • [X] Navbar hamburger menu doesn’t open on ‘apply’ page
    • Fixed by adding missing script end tag

🧪 Test results

Test 1 (index.html)

Lots of errors due to the fact that <html> was misspelled as <hmtl>

  • Solution: Fix spelling of the tag

No alt text on logos

  • Solution: Add alt text to logos

frameborder attribute on the iframe element is obsolete

This attribute came with the source code generated by YouTube. Since it doesn’t seem to do anything, it can likely be safely removed.

Test 2 (index.html)

Attribute `alt` not allowed on element link at this point

I forgot to remove alt text from the favicon after adding alt text to all logos with sed.

  • Solution: Remove alt from favicon

Test 3

Index.html : Passed

About.html : Passed

Curriculum.html

Bad value assets/Indie Quest Map.png for attribute src on element img: Illegal character in path segment: space is not allowed.

Solution: Rename assets/Indie Quest Map.png to assets/indie-quest-map.png.

Test 4:

Findus.html : Passed

Apply.html

Many instances of ‘Text not allowed in element ul in this context.’

  • Solution: Align text and inputs with <table>.

Test 5:

All documents passed!

⌛ Time tracking

DateTimeComment
04-272hCreate basic HTML and CSS
05-052hFix issues with navbar scaling and put text content in markdown
05-190.5hPolish margins and add video
05-2120minFix navbar on mobile
10minFix logo and text scaling
10minFix video scaling on main page
2hCreate About, Curriculum, Find us and Apply pages
05-220.5hValidate forms on ‘apply’ page
0.5hTweak index layout to show more content
0.5hFormat ‘Find us’ page properly
1hMake site pass W3 test
1hOrganize project file and commit bootstrap files