Heather Arthur [:harth] Reporter
	Description • 13 years ago

If you edit an overridden rule in the ruleview the line-through / strikeout indicator disappears. The line-through should be re-applied after editing the rule.

	Dave Camp (:dcamp)
	Comment 1 • 13 years ago

So it does, weird.

	Dave Camp (:dcamp)
	Comment 2 • 13 years ago

Fairly straightforward:

* During refresh, TextPropertyEditor hides the strikethrough while editing (to avoid looking strange) by checking if there's an active editor on its spans.
* inplace editor was calling done() callback before tearing itself down.
* done() indirectly causes a refresh (good) which saw a still-active editor (bad).

Attached patch tears down the editor before calling the done() callback.

	Joe Walker [:jwalker]
	Comment 3 • 13 years ago

Comment on attachment 579906 [details] [diff] [review]
fix

Review of attachment 579906 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/devtools/styleinspector/CssRuleView.jsm
@@ +1284,1 @@
>      }

This is a blatant 'to prove i read it' comment:

this._clear();
if (typeof this.done == "function") {
  let value = this.cancelled ? this.initial : this.input.value.trim();
  this.done(value, !this.cancelled);
}

It's 0.0002ms faster in the case when this.done isn't set, and it doesn't crash if this.done is set incorrectly, and it read easier to me, and I really don't care if you ignore me.

	Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Assignee
	Comment 4 • 13 years ago

Bug triage, filter on PEGASUS.

	Rob Campbell [:rc] (:robcee)
	Comment 5 • 13 years ago

Comment on attachment 579906 [details] [diff] [review]
fix

yeah, sure thing.

	Rob Campbell [:rc] (:robcee)
	Comment 6 • 13 years ago

https://hg.mozilla.org/integration/fx-team/rev/b8281fe1b926

	Tim Taubert [:ttaubert] (inactive)
	Comment 7 • 13 years ago

https://hg.mozilla.org/mozilla-central/rev/b8281fe1b926

	Heather Arthur [:harth] Reporter
	Comment 8 • 12 years ago

Unfortunately, this is happening again.

	Dave Camp (:dcamp)
	Comment 9 • 12 years ago

Not sure when I can get to this, so assigning to default in case someone else wants to pick it up.

	Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Assignee
	Comment 10 • 12 years ago

Simple fix. Use linethrough if we should when the value is overridden.

We were displaying as overridden when the name was changed, but not the value.

	Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Assignee
	Comment 11 • 12 years ago

https://hg.mozilla.org/integration/fx-team/rev/a5c189768303

	Tim Taubert [:ttaubert] (inactive)
	Comment 12 • 12 years ago

https://hg.mozilla.org/mozilla-central/rev/a5c189768303