[CLOSED] Issue #5517 - css quick edit blank line number

[core-ai-bot]

Issue by thehogfather
Saturday Oct 19, 2013 at 18:41 GMT
Originally opened as adobe/brackets#5582

---

This relates to the blank line number issue described by@njx in #5517

---

thehogfather included the following code: https://github.com/adobe/brackets/pull/5582/commits

[core-ai-bot]

Comment by njx
Monday Oct 21, 2013 at 23:43 GMT

---

This is almost certainly correct (I'm assuming the original code was a minor premature optimization :)), though I think we can no longer easily get into the case where this was broken before because of the recent change to make it so that a given rule is only shown once. I bet you could still get into it if you happen to have two different CSS files that have matching rules on the same line number, though.

In any case, this seems fine to merge, except that it needs to be merged with master due to conflicts. Would you mind doing that?

[core-ai-bot]

Comment by thehogfather
Tuesday Oct 22, 2013 at 08:10 GMT

---

Done.

[core-ai-bot]

Comment by njx
Tuesday Oct 22, 2013 at 16:54 GMT

---

Thanks! Merging.