[CLOSED] Fix for Issue 3891: Extension name and description should be selectable

[core-ai-bot]

Issue by WebsiteDeveloper
Wednesday Jun 19, 2013 at 17:16 GMT
Originally opened as adobe/brackets#4284

---

#3891

changes are on line 590 and 596

---

WebsiteDeveloper included the following code: https://github.com/adobe/brackets/pull/4284/commits

[core-ai-bot]

Comment by njx
Wednesday Jun 19, 2013 at 18:32 GMT

---

Looks good, but we should also make the cursor over the selectable text be an I-beam so people know they can select it. Could you add that?

BTW, re you running some kind of whitespace cleanup plugin? I don't think it's a big deal, but it would be nice to avoid all the diffs for empty lines. (We do like getting rid of trailing whitespace on non-empty lines, but for empty lines it's not really important.)

[core-ai-bot]

Comment by WebsiteDeveloper
Wednesday Jun 19, 2013 at 20:05 GMT

---

@njx Added the cursor.
And i am indeed running a witespace cleanup plugin, is it okay for now?
or should i readd the whitespace?

[core-ai-bot]

Comment by njx
Thursday Jun 20, 2013 at 23:03 GMT

---

Looks good, thanks. No need to fixup the whitespace...in the future, though, it would be good to turn off the whitespace cleanup for blank lines. Merging.