Д/з №6 — Жевак Антон #3
Conversation
| return num < 10 ? '0' + num : num; | ||
| }, | ||
|
|
||
| formatDate : function(date) { |
There was a problem hiding this comment.
Не смотрел на http://momentjs.com/?
Для текущего функционала, конечно, оверкилл, но если брать навырост — самое то.
There was a problem hiding this comment.
Ну, библиотека может и крутая, но это какой вырост должен быть :)
There was a problem hiding this comment.
«Библиотека отличная» – я бы сказал
|
Выглядит очень круто. Но у меня изображения появляются только через три секунды после открытия страницы. Можно это как-то победить? |
|
Потом приделаю какой-нибудь анимационный лоадер. |
| { | ||
| margin: 0; | ||
|
|
||
| font: 14px/normal sans-serif; |
There was a problem hiding this comment.
По рекомендациями W3C 1.0 < normal < 1.2 (http://www.w3.org/TR/CSS21/visudet.html#propdef-line-height)
There was a problem hiding this comment.
С точки зрения «однозначности», пожалуй, лучше все-таки задавать конкретно. Но в данном случае да, можно и так. Ок
| width: 16px; | ||
| height: 16px; | ||
|
|
||
| vertical-align: text-top; |
There was a problem hiding this comment.
Эм, нет. Это чтоб иконка визуально по центру селекта была.
|
|
||
| initialize: function(options) { | ||
| _.bindAll(this, 'keyPress'); | ||
| App.vent.on('picture:select', this.close, this); |
There was a problem hiding this comment.
vent._events будет разбухать при каждом создании инстанса, было бы неплохо делать .off по закрытию
|
@solomein, спасибо, поправил. |
http://rawgithub.com/mayton/6-api-and-markup/master/index.html