Change to visibility:hidden instead of display none for inactive pages

[jaspermdegroot]

[arschmitz]

Something else to consider here is opacity:0 where as toggling display or visibility triggers a layout and repaint an opacity change only repaints that element. However elements with opacity:0 still receive events where as visibility:none do not. I cant think of a case where this would be an issue but something to keep in mind.

[jaspermdegroot]

PR: #8241

[arschmitz]

switching this to be for using visibility:hidden we cant use opacity because things in inactive pages will still be tabbable