https://codereview.appspot.com/7060066/diff/1/app/templates/login.handlebars#newcode8
app/templates/login.handlebars:8: <input type="text" disabled="disabled"
value="admin"></input>
On 2013/01/10 00:32:12, matthew.scott wrote:
> In general, <a> and <script> are the only empty tags. input (along
with hr, b,
> etc) is usually a void tag, meaning that it self-closes like <input
/>. I
> belieeeeve that's what's done elsewhere? I'll need to check, but I
suppose
> matching the current standard we use is best.
https://codereview.appspot.com/7060066/diff/1/lib/views/stylesheet.less#newcode1406
lib/views/stylesheet.less:1406:
On 2013/01/10 00:32:12, matthew.scott wrote:
> Indenting is a mix of 2 and 4 here (which, to be fair, is the case for
the whole
> file. Maybe this is something to look into in the future for the rest
of the
> file, but could be solidified for this bit below.
https://codereview.appspot.com/7060066/diff/1/test/test_login.js#newcode119
test/test_login.js:119: test('the view login method calls the
environment login one', function() {
On 2013/01/10 00:32:12, matthew.scott wrote:
> The 'one' at the end of the test description is ambiguous; thought it
was
> referring to view, which may be due to our dual use of 'environment'
for the
> view and the WS layer. 'the view login method logs in through the
environment'
> maybe?
Thanks for the review Matthew. /codereview. appspot. com/7060068
This branch has been fixed and landed.
See https:/
https:/ /codereview. appspot. com/7060066/ diff/1/ app/templates/ login.handlebar s login.handlebar s (right):
File app/templates/
https:/ /codereview. appspot. com/7060066/ diff/1/ app/templates/ login.handlebar s#newcode8 login.handlebar s:8: <input type="text" disabled="disabled" admin"> </input>
app/templates/
value="
On 2013/01/10 00:32:12, matthew.scott wrote:
> In general, <a> and <script> are the only empty tags. input (along
with hr, b,
> etc) is usually a void tag, meaning that it self-closes like <input
/>. I
> belieeeeve that's what's done elsewhere? I'll need to check, but I
suppose
> matching the current standard we use is best.
Done.
https:/ /codereview. appspot. com/7060066/ diff/1/ lib/views/ stylesheet. less stylesheet. less (right):
File lib/views/
https:/ /codereview. appspot. com/7060066/ diff/1/ lib/views/ stylesheet. less#newcode140 6 stylesheet. less:1406:
lib/views/
On 2013/01/10 00:32:12, matthew.scott wrote:
> Indenting is a mix of 2 and 4 here (which, to be fair, is the case for
the whole
> file. Maybe this is something to look into in the future for the rest
of the
> file, but could be solidified for this bit below.
Done. Indented 4 spaces.
https:/ /codereview. appspot. com/7060066/ diff/1/ test/test_ login.js
File test/test_login.js (right):
https:/ /codereview. appspot. com/7060066/ diff/1/ test/test_ login.js# newcode119 login.js: 119: test('the view login method calls the
test/test_
environment login one', function() {
On 2013/01/10 00:32:12, matthew.scott wrote:
> The 'one' at the end of the test description is ambiguous; thought it
was
> referring to view, which may be due to our dual use of 'environment'
for the
> view and the WS layer. 'the view login method logs in through the
environment'
> maybe?
Done.
https:/ /codereview. appspot. com/7060066/