There is a Safari bug affecting our color picker popover for the tool shelf's working colors input. It causes Safari to render the popover by about a 130px upward offset. However, the click targets and devtools hover highlight visualizations for the elements in this popover do show up in the correct location, so this only affects how Safari paints the content.
It only occurs if there exists a scrollbar in the Properties panel
But strangely, it doesn't happen if there exists a scrollbar in the Layers panel— something must somehow be different between those two panels
It only occurs if the tool shelf has at least two columns
However, it being two or three columns doesn't affect how tall the ~130px upward offset is
Deleting the .tools container will visibly shrink the shelf back to a width of 1 column, but doesn't stop the issue from occurring until the web page gets resized
Disabling container-type: size; on the .shelf-and-table element prevents the issue from occurring
We need to investigate this further, ideally create a minimal reproduction test case, and report this. Then hope it's fixed. Or, find a particular way that we can change things to mitigate this on our end.
This is tested on Safari 16.5. container-type was added in Safari 16 (released 2022-09-12).
There is a Safari bug affecting our color picker popover for the tool shelf's working colors input. It causes Safari to render the popover by about a 130px upward offset. However, the click targets and devtools hover highlight visualizations for the elements in this popover do show up in the correct location, so this only affects how Safari paints the content.
https://github.com/GraphiteEditor/Graphite/assets/4388688/575d079d-803d-4fc5-8194-8ec977142a32
.tools
container will visibly shrink the shelf back to a width of 1 column, but doesn't stop the issue from occurring until the web page gets resizedcontainer-type: size;
on the.shelf-and-table
element prevents the issue from occurringWe need to investigate this further, ideally create a minimal reproduction test case, and report this. Then hope it's fixed. Or, find a particular way that we can change things to mitigate this on our end.
This is tested on Safari 16.5.
container-type
was added in Safari 16 (released 2022-09-12).