ding2tal/ddbasic — pull request #13 by vibjerg (2014-10-20 12:22:04+00:00)
Title: Review comments

Overordnede Kommentarer:
Der er en masse designfiler i temaet + nogle mockup html filer. Bør disse ikke fjernes

Er der en grund til der er en htc fil med?

Skal codekit filen være inkluderet i repositoriet. Hører den ikke til i det lokale udviklings miljø

Ville det give mening af gruppere externe biblioteker (css/sass/js) i særskilte mapper, så det er let at se hvad der er custom kode og hvad der er importeret kode.

Templates: Nogle templates er grupperet i mapper nogle er ikke. Er der nogen logik i det? F.eks. har opening-hours en mappe men profile2, place2book osv. har ikke?

Sass strukturen er overskuelig og virker fornuftig. Findes der evt. noget dokumentation af hvilke principper det er udviklet udfra og kan disse principper evt. overføres til TING kodestandard for sass/css