Enable test on Safari

[tinovyatkin]

Description

Adds missing test on Safari for our suite of visual tests.

Types of changes

• Chore (tooling change or documentation change)
• Refactor (non-breaking change which maintains existing functionality)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.

[tinovyatkin]

@jshjohnson Please review the new Safari snapshot (GitHub allows to see side-by-side, so, it's handy) - it has some visual changes regarding placeholders, not sure is as supposed to be.

[jshjohnson]

I think the comparison is fine but I have noticed since the 'ch' unit PR that placeholder widths are a lot wider than they need to be:

[tinovyatkin]

@jshjohnson Yeah, I know and wrote in original PR (including a link to Chris Mayer writing about it), that ch is the width of 0, so, on average it's about 20% wider than normal text. We can tune that in separate PR.
But here I've mostly referred these cases where placeholder disappears at all - I've seen your recent changes to placeholder handling and not sure whether such behavior is desired or it's a regression?

Anyway, I think it's better to merge like this (as it reflects current master state), so then any new changes will come with adjusted snapshots and easier to understand.

[jshjohnson]

The placeholder change removed the ability to pass a placeholder via config so that's expected 👍

[jshjohnson]

Hey - i'm not sure if it's related to this PR but visual checks seem to behaving strangely in this PR: #739 which was opened after merging this one

[tinovyatkin]

I'm not sure neither, changes here are pretty minimal. I would prefer to wait a day to see if that's a Github Actions problem...