Bug 7578

Summary: TinyMCE: Implement execCommand CreateLink
Product: WebKit Reporter: Justin Garcia <justin.garcia>
Component: HTML EditingAssignee: Justin Garcia <justin.garcia>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Bug Depends on:    
Bug Blocks: 6627    
Attachments:
Description Flags
patch mjs: review+

Description Justin Garcia 2006-03-03 14:55:14 PST
Implement execCommand link and unlink
Comment 1 Justin Garcia 2006-03-09 12:04:07 PST
I'm going to split UnLink off into it's own bug
Comment 2 Justin Garcia 2006-03-14 16:09:47 PST
Created attachment 7066 [details]
patch

Another way to do this would be to pull out the parts of ApplyStyleCommand that apply style to text runs from the parts that do the setup, traversal and cleanup, and make two new classes, PushInlineChangeCommand and ApplyInlineStyleCommand, with CreateLink and ApplyinlineStyleCommand as subclasses of PushInlineChangeCommand.  That's more risky and probably not worth the effort unless we think we'll end up needing PushInlineChange in order to implement something else.
Comment 3 Maciej Stachowiak 2006-03-14 21:27:20 PST
Comment on attachment 7066 [details]
patch

Slighly odd way to do it. r=me