Tom Tromey :tromey Assignee
	Description • 8 years ago

I noticed this while working on bug 1301078.
STR:

Open rule view.
Add a new commented-out property, e.g. by pasting in "/* background: yellow; */".
Now select the property name, clear it, type enter -- this deletes the property.
Switch to style editor.

I see:

  #testid {
    font-size: 4em;
    /*!  */
  }

The editing has left the mostly-empty comment behind.

I think this is just a small oversight in RuleRewriter.removeProperty.

	Patrick Brosset <:pbro>
	Comment 1 • 8 years ago

Inspector bug triage (filter on CLIMBING SHOES).

	Patrick Brosset <:pbro>
	Comment 3 • 8 years ago
mozreview-review

Comment on attachment 8793390 [details]
Bug 1301692 - remove comment when deleting disabled property;

https://reviewboard.mozilla.org/r/80142/#review79200

Works well, and looks good.

	Pulsebot
	Comment 4 • 8 years ago

Pushed by ttromey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3f41d2e492db
remove comment when deleting disabled property; r=pbro

	Iris Hsiao [:ihsiao]
	Comment 5 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/3f41d2e492db