Jeeqs/Jeeqs — pull request #153 by egor83 (2013-02-24 16:06:57+00:00)
Title: Showing existing reviews (#143)

Hi Ali,

Here's how an existing review would look:
http://i.imgur.com/I1MYB1E.png

One possible improvement I see right away:
Now I'm displaying disabled form and buttons for an existing review. I'm doing this by manually constructing such form in an {% if ... %} branch in a template, that's some duplicated code.

Alternatively, I could only have one form in HTML and disable elements in Javascript, but then I guess I'd have to put some JS code in a template to trigger that function depending on a template variable value - not sure if directly injecting JS code is a better solution.
What do you think?

----------------------------------------------------------------------
Ran 20 tests in 3.961s

OK