Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
"rotate(0.1turn)");
I need to understand why this transform will disable the PEPC. This will rotate the container (+ PEPC inside). Following the visibility logic, the PEPC is still visible and should be clickable.
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
"rotate(0.1turn)");
I need to understand why this transform will disable the PEPC. This will rotate the container (+ PEPC inside). Following the visibility logic, the PEPC is still visible and should be clickable.
The IntersectionObserver only allows non-distorting visual effects such as translations and up-scaling [1]. Rotations are not allowed, probably due to how complicated it would be to compute this. I don't think this should be a problem for us.
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Code-Review | +1 |
"rotate(0.1turn)");
Andy PaicuI need to understand why this transform will disable the PEPC. This will rotate the container (+ PEPC inside). Following the visibility logic, the PEPC is still visible and should be clickable.
The IntersectionObserver only allows non-distorting visual effects such as translations and up-scaling [1]. Rotations are not allowed, probably due to how complicated it would be to compute this. I don't think this should be a problem for us.
It seems to be disabled permanently. In this case, users and devs will have no idea why it fails and not be directed to the root cause and how to fix it.
I am not opposing, but a task for us is show the useful/correct information in this case (eg: a log like " the element is not visible" is not relevant)
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
"rotate(0.1turn)");
Andy PaicuI need to understand why this transform will disable the PEPC. This will rotate the container (+ PEPC inside). Following the visibility logic, the PEPC is still visible and should be clickable.
Thomas NguyenThe IntersectionObserver only allows non-distorting visual effects such as translations and up-scaling [1]. Rotations are not allowed, probably due to how complicated it would be to compute this. I don't think this should be a problem for us.
It seems to be disabled permanently. In this case, users and devs will have no idea why it fails and not be directed to the root cause and how to fix it.
I am not opposing, but a task for us is show the useful/correct information in this case (eg: a log like " the element is not visible" is not relevant)
Should we expand the console message then to include also transforms?
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Code-Review | +1 |
"rotate(0.1turn)");
Andy PaicuI need to understand why this transform will disable the PEPC. This will rotate the container (+ PEPC inside). Following the visibility logic, the PEPC is still visible and should be clickable.
Thomas NguyenThe IntersectionObserver only allows non-distorting visual effects such as translations and up-scaling [1]. Rotations are not allowed, probably due to how complicated it would be to compute this. I don't think this should be a problem for us.
Andy PaicuIt seems to be disabled permanently. In this case, users and devs will have no idea why it fails and not be directed to the root cause and how to fix it.
I am not opposing, but a task for us is show the useful/correct information in this case (eg: a log like " the element is not visible" is not relevant)
Should we expand the console message then to include also transforms?
Yes we should, the idea is, we should show a log when the PEPC is disabled *permanently*. We only had logs on several static checks like security context: permission policy and fenced frame, currently the visibility check is from IntersectionObserver -> we are missing it (or might be misaligned)
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
"rotate(0.1turn)");
Andy PaicuI need to understand why this transform will disable the PEPC. This will rotate the container (+ PEPC inside). Following the visibility logic, the PEPC is still visible and should be clickable.
Thomas NguyenThe IntersectionObserver only allows non-distorting visual effects such as translations and up-scaling [1]. Rotations are not allowed, probably due to how complicated it would be to compute this. I don't think this should be a problem for us.
Andy PaicuIt seems to be disabled permanently. In this case, users and devs will have no idea why it fails and not be directed to the root cause and how to fix it.
I am not opposing, but a task for us is show the useful/correct information in this case (eg: a log like " the element is not visible" is not relevant)
Thomas NguyenShould we expand the console message then to include also transforms?
Yes we should, the idea is, we should show a log when the PEPC is disabled *permanently*. We only had logs on several static checks like security context: permission policy and fenced frame, currently the visibility check is from IntersectionObserver -> we are missing it (or might be misaligned)
Since we have a task to add logs for developers that we might have missed. Let's leave it as part of that.
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Commit-Queue | +2 |
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
[PEPC] Add tests that check container CSS affects validation
There are various properties that can be applied to parent elements
that would affect the validity of the PEPC element. These are covered
by the IntersectionObserver visibility logic. This CL only adds tests
to verify the behavior.
Also a small bug fix to the "disable" logic where an indefinite disable
would not override a temporary disable. The scenario is verified in
*.IntersectionChangedDisableEnableDisable
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |