search

simov / express-admin

MySQL, MariaDB, PostgreSQL, SQLite admin for Node.js
MIT License
1.17k stars 223 forks source link

Restructure README #72

Closed knownasilya closed 9 years ago

knownasilya commented 9 years ago

Merge this if you think this is a better flow, otherwise feel free to close.

See changes here: https://github.com/knownasilya/express-admin/blob/patch-1/README.md

simov commented 9 years ago

Well, I'll have to think about this one :grinning: What's the problem with the current readme structure? I intentionally put that big fat screenshot on top. Also I really do like my big fat links :grinning: so I don't know.

knownasilya commented 9 years ago

The screenshot is great, but only on first visit. This change still keeps the screenshot relatively above the fold. Also the screenshots tend to get outdated faster, so making that the focus is probably not the best idea. But yeah, up to you. On Sep 9, 2014 4:58 PM, "simo" notifications@github.com wrote:

Well, I'll have to think about this one [image: :grinning:] What's the problem with the current readme structure? I intentionally put that big fat screenshot on top. Also I really do like my big fat links [image: :grinning:] so I don't know.

— Reply to this email directly or view it on GitHub https://github.com/simov/express-admin/pull/72#issuecomment-55034405.

simov commented 9 years ago

Hey, @knownasilya

I just refactored the markdown used in the readme (also added a few more badges). Apart from that I'm still not convinced that the screenshot shouldn't be on top, but I agree that it's outdated (I didn't even noticed it)

https://github.com/simov/express-admin/commit/812b392490f838b570f54f2fa0d4b7a0142c2844