This looks OK, as a physical/layer 1 redundancy.
you would need to add a similar diagram for layer 2 (VLANs)
and layer 3 (ip interfaces, subnets and what routing you are going to use, or potentially set the FWs up as an HA pair).
Please remember to rate useful posts, by clicking on the stars below.