--It has terrible formatting and typos.
--It puffs up something more important than it is.
--It is more about pedagogy than a mathematical idea.
--It suggests the idea is original, when it almost certainly is not.
--It doesn't link to the original (and better source).
Okay, here is the good things about the approach:
--It is good to shift your thinking about mathematical derivations and proofs and think about them as code that runs on people's brains. You input a derivation into someone's brain and they return a boolean value (this is true, it makes sense, etc.). Pedagogy is trying to optimize the code for less powerful architectures. Just like when you are optimizing code tiny little details of instruction orders matter, the same with mathematical derivations.
--Fundamentally, algebraic manipulations are uncomfortable and nonintuitive for students. They feel like tricks. Going forwards from (x+a)^2=x^2+2ax+a^2 makes sense but going backwards as in the case of completing the square is hard. It's not the same case for x^2=a vs sqrt(x)=a. This is kind of a similar case to math students feeling confused by adding and subtracting the same quantity when doing calculus limits. For any trained mathematician, this is obvious, but it really feels like a trick at first. The nice thing about this approach is that it avoids this issue and gives you a good reason WHY the -b/2 term shows up. Additionally, it avoids the problem of substituting, which tends to bog students down (try teaching the chain rule someday).
Students should still understand completing the square but I don't think this is a bad way to introduce them to the quadratic formula. It highlights the symmetric of the roots (at least for real values), which makes sense if you plot a quadratic.