Ben Allfree :: Painless Programming

Just like the pre-checks, I also have post-checks to ensure quality. Here they are:

Layout

• Margins/padding match comp
• Liquid layout does not introduce gaps on resize
• Capitalization matches comp
• Text styling matches comp
• Coloring matches comp
• Text leading matches comp
• Proportions and relative sizes match comp
• Indentical or downlevel rendering in FF3, Safari, IE7, IE6
• Navigation does not wrap in small liquid layout
• Status bar features do not wrap in small liquid layout
• Links do not wrap in small liquid layout
• Headings/headers do not wrap in small liquid layout
• No hard-coded duplicate CSS in views
• CSS definition blocks are in external files
• JS functions are included via external links

Wireup

• All db operations incorporate user id or role in WHERE
• No input params are trusted to directly control privileged operation
• Front end is not trusted for security checks
• JavaScript/Ajax and downlevel compatibility in FF3, Safari, IE7, IE6
• FireBug console statements are not present or console() function is overridden in case it is not present
• Database tables, pks, and fks follow mvc-style conventions
• File uploads stored on disk with UUID referencing database table holding MIME type, original name, posting date, disk file name, and user ID if needed
• Thumbnails generated dynamically
• View helpers used where possible
• Database FK’s indexed
• No off-screen validation failures are possible
• Transactions are used only if user’s current work can be restored to the screen upon transaction rollback
• RESTful routing used
• Route helpers used for all RESTful paths
• No hard-coded <a> tags for site paths
• All calculations in models – none in view or controller
• Form helpers are used instead of <form> tag
  document.getElementById() is $()

• Paths to root and core are constants and not hard coded
• JS functions are included via external links
• Raw $_REQUEST not used ($params) instead
• No raw <input> fields
• No magic values in code
• File paths refactored and DRY’d into constants
• TIMESTAMP type not used
• DATETIME defaults to null if you want it to auto-populate