Posted to: http://drupal.org/node/366964
I have prepared a case study based on user comments on this thread: http://drupal.org/node/342390
I have thumbnails for the images as well but they were stripped out by the html filter, would love to be able to put those back in before it was promoted.
Please contact me if you have any questions/feedback,
Andrew Mellenger
Comments
Comment #1
mellenger commentedShould I change the release date for this article? I'd like to add in my images and get it a bit more ready to go before tomorrow.
Regards,
Andrew
Comment #2
mellenger commentedI added the code with images to drupalbin: http://drupalbin.com/7883
Comment #3
gregglesI've updated the post with the new images. Ideally the images should be attached to this issue and the HTML updated so they are hosted on drupal.org's infrastructure.
I also don't see this on http://drupal.org/node/280488 - could you schedule it in there?
The writeup seems great to me. I'm +1 on promoting it.
Comment #4
Amazon commentedThis case study shows a lot of promise. It's great to see case studies of this quality develop organically. However, it does needs some review. In particular, it needs to be reviewed for use of capitalization Drupal, HTML and CSS are some examples. All sentences should begin with caps. See "the primary menu", "the secondary menu" bullets.
The flow of the case study could be improved visually if the images are positioned beside the text, particularly the examples of custom modules. Images needs to be local to keep the front page loading quickly.
Normally we mention the development shop at the end of the case study. This keeps down criticism about front page posts being too promotional.
Some sentences need review, such as "This helped tons because we had real content coming back rather than greeking or our ideas of what content should or would be in an article, restaurant listing or popup list.".
The list of of Key contributed modules should be organized in some obvious fashion and have completed descriptions ideally. So I would organize them alphabetically, or numbered if they are in order of importance.
Comment #5
rickvug commentedI agree with Kieren's points. The only one I wonder about is how to naturally speak about the project background without mentioning Grey or ImageX. I'm fine with moving the shop names lower so that the article appears less commercial but I wouldn't want to do so if it impacts the flow.
Before this gets promoted to the front page I'd like to jump in with some last minute edits and suggestions. In general I think that the write-up is informative and unfolds in a logical fashion. However there are some inconsistencies with the tone and a tendency for long sentences (watch out for anything than runs to 3+ lines). To help with some final polish I've created a list of potential edits below. Please do not consider any of the changes "blockers" to this being promoted. As I said, I feel that the writeup is already of a high-quality and informative. On the technical side, both myself and Ben (the lead ImageX developer on the project) have reviewed for accuracy and everything checks out.
Potential Edits:
Great work Andrew!
Comment #6
mellenger commentedOkay I have incorporated most of Rick's comments above and added a "Credits" section with links and our job titles to the bottom. Stick a fork in it?
Comment #7
rickvug commentedThe post is up on the front page so I'm closing the issue. Thanks for the perseverance with all of the edits!