is toggle safe? should this not always be show()?