also the condition block should be seeking a price level property, not a color.
You could also just refer to it by color as you did.
if you want to keep the configuration you have, you could also just search for objects by name "that contains" and then just put FIBO in both spaces in that by 'name object loop' block. From there, no matter how many fibs are drawn, they will all be named FIBO#, with an increaseing # of each successive drawn fib. Then you;d have to delete each prior one, once a new one is drawn.
plenty of ways to do it!