jaheppler
jaheppler

Howdy @manton. I have a bug report for you. On the latest macOS using the latest Safari, what I assume is related to the new image light box is bugging for me. It takes over the whole screen and I cannot close it. This happens when the page loads.

|
Embed
Progress spinner
jaheppler
jaheppler

@jaheppler Just noticed it looks like @wcaleb has the same bug report not long ago.

|
Embed
Progress spinner
wcaleb
wcaleb

@jaheppler @manton Yep, same problem here.

|
Embed
Progress spinner
manton
manton

@wcaleb @jaheppler Oops, thanks! Not happening for me so I wonder if it's a caching problem. Can you try View → "Reload Page from Origin" (cmd-option-R) in Safari and see if that fixes it?

|
Embed
Progress spinner
jaheppler
jaheppler

@manton No luck here with the reload, unfortunately.

|
Embed
Progress spinner
wcaleb
wcaleb

@manton I’ll check again later when back at computer but it seemed to continue happening even after I cleared caches.

|
Embed
Progress spinner
vincent
vincent

@jaheppler @wcaleb Looking at the screenshots, and what I can see is, that no styles have been loaded either. I can see an un-styled button in the middle of that page though. What is happening is that the lightbox should be hidden when it loaded, however because the styles don't seem to have made it over, it's causing this to show. Can you confirm the version of Safari you are on please? cc: @manton

|
Embed
Progress spinner
manton
manton

@vincent @jaheppler @wcaleb Another idea: are you using an ad-blocker that might be interfering?

|
Embed
Progress spinner
vincent
vincent

@manton @jaheppler @wcaleb — that was my thought too, however I now think this is related to CSS nesting which is supported by all modern browsers — with Safari being an outlier because it was Safari 17 onwards (so I believe).

|
Embed
Progress spinner
wcaleb
wcaleb

@vincent @manton @jaheppler I’ll preface this with apology that my computer sorely needs updating. For me the problem is on both Safari 14.1.2 (😬) and Chrome 116 (😬😬) on Mojave (😬😬😬). Any chance of a patch for us slow adopters?

|
Embed
Progress spinner
jaheppler
jaheppler

@wcaleb @vincent @manton I am running 1Blocker but disabling it and hard refreshing the page didn’t resolve the issue. I’m on Safari 17.1.2.

|
Embed
Progress spinner
vincent
vincent

@wcaleb @jaheppler — I have a PR to be merged and hopefully have @manton get this deployed today/tomorrow. I think that would explain what is happening. Sorry for that!

|
Embed
Progress spinner
manton
manton

@jaheppler @wcaleb Try now! Thanks @vincent.

|
Embed
Progress spinner
wcaleb
wcaleb

@manton @vincent @jaheppler Fixed for me! Thanks!!

|
Embed
Progress spinner
vincent
vincent

@wcaleb awesome!

|
Embed
Progress spinner
In reply to
jaheppler
jaheppler

@manton @wcaleb @vincent Fixed here too, thanks!

|
Embed
Progress spinner
vincent
vincent

@jaheppler Nice! Thanks for letting me know. Sorry for the trouble here.

|
Embed
Progress spinner
jaheppler
jaheppler

@vincent No worries! Glad we were able to isolate a fix.

|
Embed
Progress spinner