Hello,
For this specific purpose (blocking/allowing comms between two VLANs and only for IPv4) I would go with single "family inet" IRB filter.
However, if You also have IPv6 (which is not matched in Your current filter apart from icmpv6) then You have to write another "family inet6" filter, as opposed to combining IPv4 and IPv6 match terms in a single "family ethernet-switching" filter.
HTH
Thx
Alex