can we just change this to `is_landscape`?