Bug 11300 - [Drosera] Inline breakpoint editor
Summary: [Drosera] Inline breakpoint editor
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Enhancement
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-15 02:59 PDT by David Smith
Modified: 2008-05-17 09:55 PDT (History)
1 user (show)

See Also:


Attachments
Draft patch to add an Xcode 3 style inline breakpoint editor (58.65 KB, patch)
2006-10-15 03:03 PDT, David Smith
no flags Details | Formatted Diff | Diff
Updated patch (26.41 KB, patch)
2006-10-18 01:46 PDT, David Smith
timothy: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Smith 2006-10-15 02:59:36 PDT
Xcode 3 has an inline breakpoint editor that, while less than ideal, provides some interesting ideas for interaction. Attached is a draft patch to add this functionality to Drosera.
Comment 1 David Smith 2006-10-15 03:03:46 PDT
Created attachment 11094 [details]
Draft patch to add an Xcode 3 style inline breakpoint editor

Just an added comment to the bug... I believe this Xcode 3 thing is safe to talk about, since it's on the Apple leopard page. If I'm breaking my NDA here, please tell me to shut up rather than cease-and-desisting. ;)
Comment 2 David Smith 2006-10-15 03:05:25 PDT
One further comment: the lack of transparent background on the images is a known issue with the current patch; I'll be re-photoshopping them before I ask for review.
Comment 3 David Smith 2006-10-18 01:46:14 PDT
Created attachment 11131 [details]
Updated patch

Fixes the transparency issues, and some state consistency issues.
Comment 4 Timothy Hatcher 2006-10-18 09:37:58 PDT
Comment on attachment 11131 [details]
Updated patch

Looks good.
Comment 5 Timothy Hatcher 2006-10-18 09:38:48 PDT
Three issues I fixed before landing.

I tweaked the margins of the editor to have less space.

Also the onlick in the editor html needs to reference window.parent to work. I made that change.
onclick="window.parent.toggleBreakpointForEditor(this.parentNode.parentNode.parentNode)

I made the save button remove the editor.

Landed in r17107.
Comment 6 Timothy Hatcher 2006-10-18 09:40:30 PDT
Also when creating the editor div you need to create it in the source document, not the parent document. The source is in an iframe. Since that is the document you will apend to you need to create it there also. I made that change.
Comment 7 Timothy Hatcher 2008-05-17 09:55:59 PDT
Closing since Drosera has been replaced by the new Web Inspector debugger. Moving to the New Bugs component so the Drosera component can be closed and removed.