What’s wrong on this picture?
Critical path. by definition, includes tasks with zero float, that is, those which can’t be delayed without delaying the whole project. Striped tasks on the picture have zero float. Non-striped have a few days float, so they are not on the critical path.
Float indicates how much flexibility you have with respect to task end date. Zero float => no flexibility, task must be completed, otherwise the end date of the whole project moves forwards. Non-zero float => you may delay this task if needed.
Anyway, if you really want to have zero float in all tasks, make any of those which are not yet on the critical path. e.g. 1.3.1 Marketing, one week longer.