Problem Solving & Design B

The app covers the full user journey from adding items to detecting overweight, with sensible defaults, per-cube progress bars, and unit conversion that keeps weights consistent.

• +Switching units converts both the limit field and every stored item weight so totals remain numerically consistent after a toggle.
• +Per-cube progress bars fill relative to the overall limit, giving an at-a-glance sense of how each cube contributes to the total.
• +Empty cubes show a contextual placeholder that hints at the click-to-move interaction rather than leaving a blank space.

Completeness C

Core features are all present — add, move, delete, limit warning, and clear all work end-to-end — but drag-and-drop is absent and the app is a single monolithic file with no module separation.

• +Items can be moved between cubes with a single click, and keyboard users can also trigger the move via Enter or Space, making the interaction accessible without drag support.
• +Both an inline alert banner and a card-level red highlight activate together when the grand total exceeds the user-set limit, giving clear redundant feedback.
• -Drag-and-drop between cubes is not implemented; the brief explicitly asks for it as an assignment method alongside click.
• -The entire app lives in one HTML file with no module boundaries, making the codebase harder to extend or test in isolation.

Technical Craft F

The code uses JSDoc typedefs and named functions for most logic, but lives entirely in one file, relies on inline event handlers, and the built-in test runner has a bug that marks failures as passing.

• +Core logic is split into clearly named functions — calculateTotals, updateUI, renderList, validateInputs — keeping each concern separate within the single file.
• +CSS custom properties define the colour palette once and are reused consistently for backgrounds, borders, and accent colours.
• -The test runner marks failures with class 'test-ok' instead of 'test-fail', so a failing assertion is visually indistinguishable from a pass.
• -Item rows are built with innerHTML including the item id directly interpolated, which could allow a crafted item name to inject HTML into the delete button's onclick attribute.