<?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>16349</bug_id>
          
          <creation_ts>2007-12-07 16:37:15 -0800</creation_ts>
          <short_desc>Double-clicking styles in the Computed Style or User Agent sections to edit them silently fails</short_desc>
          <delta_ts>2011-07-01 06:18:53 -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>WONTFIX</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Roben (:aroben)">aroben</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>apavlov</cc>
    
    <cc>cameron</cc>
    
    <cc>dmkash</cc>
    
    <cc>eric</cc>
    
    <cc>idan</cc>
    
    <cc>joepeck</cc>
    
    <cc>mmurrett</cc>
    
    <cc>rik</cc>
    
    <cc>Wout.Mertens</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>63638</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2007-12-07 16:37:15 -0800</bug_when>
    <thetext>Double-clicking styles in the Computed Style section to edit them silently fails. I&apos;m guessing that this is the source of a lot of the &quot;inline css editing doesn&apos;t work&quot; comments on &lt;http://webkit.org/blog/148/web-inspector-update/&gt;.

One possible solution is to make double-clicking a property in the Computed Style section copy that property to the Inline Style Attribute section and start editing it there. I think it&apos;s important that double-clicking in the Computed Style section let you edit *somewhere*, rather than popping up an error dialog or something.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>63651</commentid>
    <comment_count>1</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2007-12-07 22:24:26 -0800</bug_when>
    <thetext>The same happens for user agent rules and mapped attributes. We could allow editing those i think, but it seems weird.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>63653</commentid>
    <comment_count>2</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2007-12-07 22:25:00 -0800</bug_when>
    <thetext>*** Bug 16350 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64607</commentid>
    <comment_count>3</comment_count>
    <who name="fALk Gärtner">falk</who>
    <bug_when>2007-12-18 00:06:20 -0800</bug_when>
    <thetext>For me editing also does not work on the inline element. no matter where you click I can only select values (for copying) but I can not change any of them. maybe there is something more fundamental missing on my part, but it sure as hell is no &quot;click and play&quot; at the moment ;) i double clicked everywhere tried every ctrl-click and there is just nowhere an edit value box as shown on the blog. 
Also I would love to click in the metrix box to change box models if that is possible. Just let me change whatever value I defined in the first place wherever it appears - in the code/stylesheet, in the stlye, metrics and property boxes - everywhere. would make life as a part time webdeveloper so much easier. 

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>66021</commentid>
    <comment_count>4</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2008-01-02 06:42:10 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; Also I would love to click in the metrix box to change box models if that is
&gt; possible. Just let me change whatever value I defined in the first place
&gt; wherever it appears - in the code/stylesheet, in the stlye, metrics and
&gt; property boxes - everywhere. would make life as a part time webdeveloper so
&gt; much easier. 

Could you file a new bug about this enhancement request?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68648</commentid>
    <comment_count>5</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2008-01-29 11:07:32 -0800</bug_when>
    <thetext>&lt;rdar://problem/5712879&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>72258</commentid>
    <comment_count>6</comment_count>
    <who name="Wout Mertens">Wout.Mertens</who>
    <bug_when>2008-02-28 08:31:38 -0800</bug_when>
    <thetext>the Firebug editor for Mozilla does allow adding ad-hoc css rules for an element.

Maybe double-clicking a computed style should create a new style box specific to the element and copy the attribute name and value into it, allowing the user to edit that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>93862</commentid>
    <comment_count>7</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2008-10-03 00:20:34 -0700</bug_when>
    <thetext>*** Bug 21325 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>94874</commentid>
    <comment_count>8</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2008-10-10 04:28:50 -0700</bug_when>
    <thetext>*** Bug 21488 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>156397</commentid>
    <comment_count>9</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2009-10-20 21:48:27 -0700</bug_when>
    <thetext>Is this something that we want to add or should Computed / User Agent / Mapped Attribute styles be off limits to editing?  There are already plenty of other ways to add styles.

If we keep them uneditable then there should be some indication to the user (as evident by all the duplicates of this bug).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>156450</commentid>
    <comment_count>10</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2009-10-21 03:27:41 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; Is this something that we want to add or should Computed / User Agent / Mapped
&gt; Attribute styles be off limits to editing?  There are already plenty of other
&gt; ways to add styles.
&gt; 
&gt; If we keep them uneditable then there should be some indication to the user (as
&gt; evident by all the duplicates of this bug).

I think double click could copy the property to the Style Attribute section and edit.

We should also show non-editable sections differently. Maybe a &quot;not pencel&quot; icon in the section header like Finder uses for folders?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>307282</commentid>
    <comment_count>11</comment_count>
    <who name="Alexander Pavlov (apavlov)">apavlov</who>
    <bug_when>2010-11-10 03:21:42 -0800</bug_when>
    <thetext>(In reply to comment #10)
&gt; (In reply to comment #9)
&gt; &gt; Is this something that we want to add or should Computed / User Agent / Mapped
&gt; &gt; Attribute styles be off limits to editing?  There are already plenty of other
&gt; &gt; ways to add styles.
&gt; &gt; 
&gt; &gt; If we keep them uneditable then there should be some indication to the user (as
&gt; &gt; evident by all the duplicates of this bug).
&gt; 
&gt; I think double click could copy the property to the Style Attribute section and edit.
&gt; 
&gt; We should also show non-editable sections differently. Maybe a &quot;not pencel&quot; icon in the section header like Finder uses for folders?

Currently, we mark non-editable sections with a gray background which makes it quite clear that they are read-only. Does this sound good enough wrt the rw/ro distinction? Are we still bothered with letting the user edit at least something when they double-click inside the read-only section?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>431119</commentid>
    <comment_count>12</comment_count>
    <who name="Alexander Pavlov (apavlov)">apavlov</who>
    <bug_when>2011-07-01 06:18:53 -0700</bug_when>
    <thetext>The currently implemented graying-out seems to be a good and intuitively clear indicator of read-only sections.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>