Bug 192570 – [iOS] Support dropping contact card data (public.vcard) in editable content

• Component: HTML Editing
• Status: RESOLVED FIXED

[Wenson Hsieh]

<rdar://problem/35626913>

[Wenson Hsieh]

For added context, our current behavior when dropping a vCard in both rich and plain editable elements is that we dump the entire contents of the vCard as plain text :(

The desired behavior is probably:

• If there is a URL accompanying the vCard, dropping into a rich contenteditable with attachments enabled should produce a link and an attachment element, and dropping into a rich contenteditable without attachments should insert just a link.

• If there is no URL accompanying the vCard, dropping into a rich contenteditable area should insert an attachment element, and dropping into a rich contenteditable without attachments enabled should do nothing.

• If there is a URL accompanying the vCard, dropping into a plain contenteditable should insert the URL.

• If there is no URL accompanying the vCard, dropping into a plain contenteditable should do nothing.

[Wenson Hsieh]

Created attachment 357080 [details]
First pass

[Wenson Hsieh]

Created attachment 357084 [details]
Try to fix macOS builds

[Wenson Hsieh]

Created attachment 357085 [details]
Fix 32-bit macOS build

[Tim Horton]

Comment on attachment 357085 [details]
Fix 32-bit macOS build

View in context: https://bugs.webkit.org/attachment.cgi?id=357085&action=review

> Tools/TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:28
> +#if WK_API_ENABLED && !PLATFORM(WATCHOS) && !PLATFORM(TVOS)

isn't this just PLATFORM(IOS) now?

WK_API_ENABLED is always true there, and that excludes watch and tv

> Tools/TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:1851
> +    [contact setFamilyName:@"Bitdiddle"];

🤨

[Tim Horton]

(In reply to Tim Horton from comment #5)
> Comment on attachment 357085 [details]
> Fix 32-bit macOS build
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=357085&action=review
> 
> > Tools/TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:28
> > +#if WK_API_ENABLED && !PLATFORM(WATCHOS) && !PLATFORM(TVOS)
> 
> isn't this just PLATFORM(IOS) now?
> 
> WK_API_ENABLED is always true there, and that excludes watch and tv

OH you want mac too. OK so (PLATFORM(MAC) || PLATFORM(IOS)) && WK_API_ENABLED

> > Tools/TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:1851
> > +    [contact setFamilyName:@"Bitdiddle"];
> 
> 🤨

[Wenson Hsieh]

Comment on attachment 357085 [details]
Fix 32-bit macOS build

View in context: https://bugs.webkit.org/attachment.cgi?id=357085&action=review

>>> Tools/TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:28
>>> +#if WK_API_ENABLED && !PLATFORM(WATCHOS) && !PLATFORM(TVOS)
>> 
>> isn't this just PLATFORM(IOS) now?
>> 
>> WK_API_ENABLED is always true there, and that excludes watch and tv
> 
> OH you want mac too. OK so (PLATFORM(MAC) || PLATFORM(IOS)) && WK_API_ENABLED

Sounds good! Changed to (PLATFORM(MAC) || PLATFORM(IOS)) && WK_API_ENABLED

>>> Tools/TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:1851
>>> +    [contact setFamilyName:@"Bitdiddle"];
>> 
>> 🤨
> 
> 

Heh, I do like my SICP names.

Changed to a more generic "Foo Bar"

[Wenson Hsieh]

Created attachment 357258 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 357258 [details]
Patch for landing

Clearing flags on attachment: 357258

Committed r239184: <https://trac.webkit.org/changeset/239184>