<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>31667</bug_id>
          
          <creation_ts>2009-11-19 07:31:03 -0800</creation_ts>
          <short_desc>Web Inspector: Changes View / Drawer</short_desc>
          <delta_ts>2010-07-07 08:27:58 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Web Inspector (Deprecated)</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>20663</dup_id>
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Enhancement</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Joseph Pecoraro">joepeck</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>alessio.zito</cc>
    
    <cc>apavlov</cc>
    
    <cc>bweinstein</cc>
    
    <cc>joepeck</cc>
    
    <cc>pbakaus</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>timothy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>164969</commentid>
    <comment_count>0</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2009-11-19 07:31:03 -0800</bug_when>
    <thetext>This has been discussed numerous times on IRC. It would be similar to Firediff &lt;http://www.incaseofstairs.com/firediff/&gt; in that you would be able to see the number of changes you have made, what the changes were, potentially export them etc. I tried to identify what can be handled today:

Behaviors:
Add    = Ability to Create in Inspector
Edit   = Ability to Modify in Inspector
Remove = Ability to Delete from Inspector
Move   = Ability to Move in Inspector, think Drag+Drop

Fundamentals:
- DOM Node               [            Remove ]
- DOM Attribute          [ Add  Edit  Remove ]
- CSS Selector           [ Add  Edit  Remove ]
- CSS Property / Metrics [ Add  Edit  Remove ]
- Properties Sidebar     [ Add  Edit  Remove ]
- DOM Storage Pair       [ Add  Edit  Remove ]
- Cookies                [            Remove ]

Inspector Items: (might be worth knowing about)
- Inspected an Element   [ Add               ]
- Breakpoints            [ Add        Remove ]
- Watch Expressions      [ Add        Remove ]
- Profiles               [ Add               ]

Future Items to Consider:
- EventListeners (Remove/Move)
- Cookies (Add/Edit)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>164982</commentid>
    <comment_count>1</comment_count>
    <who name="Patrick Mueller">pmuellr</who>
    <bug_when>2009-11-19 08:16:52 -0800</bug_when>
    <thetext>The Items listed under &quot;Inspector Items&quot; would be useful to see from a &quot;history&quot; perspective, perhaps, especially if it would put the user in a position where they could easily &quot;replay&quot; the action

- Inspected an Element - could take you back to the same element in the Elements panel

- Breakpoints - could take you back to the same line in the file
 
- Watch Expressions - just display the expression; user could copy it, then paste back into the watch pane.

Might also want to see the list of things typed into the console, mainly for completeness.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>165371</commentid>
    <comment_count>2</comment_count>
    <who name="Paul Bakaus">pbakaus</who>
    <bug_when>2009-11-20 11:17:57 -0800</bug_when>
    <thetext>I love the idea of this functionality and I like the history idea as well. For inspiration, have a look at the branched history functionality of the &quot;e&quot; texteditor at http://www.e-texteditor.com.

I think really the most tricky part is the actual diffing. The problem is that running scripts have the ability to change the markup and CSS of the page. So in the most lazy implementation, the user hits save and gets all the script generated garbage into his export/file. So a couple thing we need to do:

1) Keep track of the initial, unchanged state
2) Optional: Log all subsequent changes into the new history (using DOMAttrChanged or something)
3) Optional: Offer an option to do a diff including script changes (easy!)
3) Normal export/save would ignore script changes, therefore tries to diff only if the user has changed DOM nodes and/or CSS that were already there at launch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>239026</commentid>
    <comment_count>3</comment_count>
    <who name="Alessio">alessio.zito</who>
    <bug_when>2010-06-16 12:46:53 -0700</bug_when>
    <thetext>An easy and fast solution before THE BIG solution may be the following:

from style.css I make changes in the inspector and I can save as modifiedStyle.css with the original style automatically commented and the modified style just below.

So that the modifiedStyle.css may look like:

/* original style overridden by Web Inspector
a:link {
	color: #CCC;
}
new style below */

a:link {
	color: #CCC;
	text-decoration: none;
}

/* end of new style by Web Inspector */</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>247561</commentid>
    <comment_count>4</comment_count>
    <who name="Alexei Masterov">masterov</who>
    <bug_when>2010-07-07 08:12:27 -0700</bug_when>
    <thetext>Depends on 20633 for exporting functionality.  I am not sure that once we have 
an abilty to export this feature (being able to track all history) will be 
useful.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>247567</commentid>
    <comment_count>5</comment_count>
    <who name="Alexei Masterov">masterov</who>
    <bug_when>2010-07-07 08:27:58 -0700</bug_when>
    <thetext>After discussing this with the team, I am marking this as a duplicate of 20663, 
because they both essentially talk about implementing functionality that is 
similar to Firediff.

*** This bug has been marked as a duplicate of bug 20663 ***</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>