[WIP] Share modal updates

[asolove]

• “Cancel”/“close” distinction #492: fix wording of text
• Hide the import code in the publish menu #493: import snippet should be less visible than share url.
• Show currently published-version v current version

[schanzer]

@asolove any chance we can close this out in the near future? It's super low-hanging fruit..

[asolove]

Happy to. Can you review the open questions and venture some opinions? I’ll have to load it back into my head but there isn’t much left here as I recall.

…

On Thu, Jan 11, 2024 at 12:13 PM Emmanuel Schanzer ***@***.***> wrote: @asolove <https://github.com/asolove> any chance we can close this out in the near future? It's super low-hanging fruit.. — Reply to this email directly, view it on GitHub <#494 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAACCL6EJY7SXBRXQHSJPJLYOAMSLAVCNFSM6AAAAAA7BDAT2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBXGYYDINBSGQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[schanzer]

Excuse my ignorance here - where are these questions?

[asolove]

Oh, sorry, there is a long discussion in another issue around further updates to the share sheet. But yeah, this small round of tweaks can just get merged shortly.

[schanzer]

Does this PR address all three checkboxes, above? Or just the one? If it's just one I think it's best to wait until we snag the others.

[jpolitz]

This makes things better and it looks like the comments say “just get this much in”, so merging. Nice work.