[Webkit-unassigned] [Bug 57941] New: Web Inspector: it should be possible to use TextViewer to display and edit both javascript and css.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 6 05:43:14 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=57941

           Summary: Web Inspector: it should be possible to use TextViewer
                    to display and edit both javascript and css.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Web Inspector
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: podivilov at chromium.org
                CC: rik at webkit.org, keishi at webkit.org, pmuellr at yahoo.com,
                    joepeck at webkit.org, pfeldman at chromium.org,
                    yurys at chromium.org, bweinstein at apple.com,
                    apavlov at chromium.org, loislo at chromium.org,
                    aandrey at google.com


Web Inspector: it should be possible to use TextViewer to display and edit both javascript and css.

Currently, TextViewer can only be used when wrapped with SourceFrame. SourceFrame is mostly javascript-specific and should not be used for css editing and displaying resource contents in resources panel.
We should decouple TextViewer from SourceFrame (SourceFrame knows too much about TextViewer internal details) and introduce CSSEditor class based on TextViewer.

TextViewer gets a TextViewerDelegate with the following interface:
 - populateContextMenu, mouseDown, mouseHover, etc
 - editable, startEditing, finishEditing

SourceFrame becomes a TextViewerDelegate implementation for viewing and editing javascript in scripts panel.
Resources panel uses its own TextViewerDelegate implementations to show and edit (when applicable) css, html, xhrs etc.

Any thoughts?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list