PI-1056 fixed active zone selector not clickable bug #114
Conversation
| @@ -57,7 +57,8 @@ class ActiveZoneSelector extends Component { | |||
| top: '50%', | |||
| transform: 'translateY(-50%)', | |||
| width: '200px', | |||
| marginLeft: '170px' | |||
| marginLeft: '170px', | |||
| zIndex: '1' | |||
There was a problem hiding this comment.
Ugh, I hate this but I don't have a better solution. Which integration is this for?
There was a problem hiding this comment.
This only happens in multi zoned integrations but I added to general because why not. The solution is already a mess. Tested on WordPress and works fine
|
I figured the exact reason of the bug. Under attack button is too long. Fixing the whole CSS would also fix this issue |
|
Can we do that instead? |
|
Yes, I'll try to fix the CSS of the whole bar. |
|
Tested on CPanel and WordPress. Will add WordPress commit hacks.css as well. The problem was earlier I had put After fixing that WordPress was fine but CPanel messed up. Changing CSS is weird, Integrations are weirder... |
|
Currently the only issue is long domain names are kind of hidden
The average domain name length is 14 characters. The average looks OK
|
|
thats okay |
|
LGTM |
|
You can approve an approved PR. No need for LGTM anymore 😄 |
After the first zone switch, the div was being rendered below other divs so it wasn't recognizing the click. I did a quick
z-index. The whole active zone selector CSS is a mess. We may need to fix it at some point