[CLOSED] Extension Info Column Tweak

[core-ai-bot]

Issue by larz0
Tuesday Sep 10, 2013 at 20:11 GMT
Originally opened as adobe/brackets#5153

---

See #5147

---

larz0 included the following code: https://github.com/adobe/brackets/pull/5153/commits

[core-ai-bot]

Comment by njx
Monday Sep 23, 2013 at 23:23 GMT

---

This looks a lot nicer. Looking at it now, though, I wonder if it would make more sense to group the version number with the date, since those are logically related. So how about:

Extension Title
Author
Version x.x.x -- yyyy-mm-dd

(I thought it might be nice to add the label "Version", and there's probably room anyway, but don't feel strongly either way.)

[core-ai-bot]

Comment by larz0
Tuesday Sep 24, 2013 at 05:44 GMT

---

@njx updated and it definitely looks much better now. The author name link competes with the extension name visually but once adobe/brackets#4930 is merged then hopefully we can restore balance to this universe.

[core-ai-bot]

Comment by njx
Tuesday Sep 24, 2013 at 17:53 GMT

---

Yup, I like this layout. Just one code review comment.

[core-ai-bot]

Comment by njx
Tuesday Sep 24, 2013 at 18:02 GMT

---

Also, this breaks one of the unit tests due to the format change (ExtensionManager-test, "should populate itself with registry entries and display their fields when created"). I'll push up a fix for that after you've made your fix.

[core-ai-bot]

Comment by larz0
Tuesday Sep 24, 2013 at 18:43 GMT

---

@njx fixed.

[core-ai-bot]

Comment by njx
Tuesday Sep 24, 2013 at 20:02 GMT

---

Fixed unit test. Merging.