fix(icon): isActive="false" with ios mode

[manucorporat]

Short description of what this resolves:

#8435

Before: (red failing cases)

After:

[brandyscarney]

Code looks good as long as it passes snapshot. 👍

[manucorporat]

@brandyscarney @adamdbradley same thing here. I could not run snapshot. But the scope of this change is very limited, purely a logic change. No css.

I added additional e2e tests for the cases it used to fail, and previous ones continue working.

For me, it is ready to be merged.

I would not delay it until snapshot and build gets faster, we can always run snapshot before releasing RC1.