Existem algumas partes que podemos precisar de alguma limpeza.
ng-lazyloaded
, ng-lazyloading
, ng-failed-lazyloaded
)setLoadedStyle
. No momento, estamos definindo os nomes das classes de três maneiras diferentes. Normalize-o (consulte https://github.com/tjoskar/ng-lazyload-image/pull/274#discussion_r160511311). https://github.com/tjoskar/ng-lazyload-image/blob/7b2261798a13066f23c10f353a60e636d95bad2c/src/lazyload-image.ts#L109 https://github.com/tjoskar/ng-lazyload-image/blob/7f353a60e636d95bad2c/src/lazyload-image.ts#L109 https://github.com/tjoskar/ng-lazyload-image/blob/7b2261760c98a62366c106623e62366c10623e630e62366c10623e630c10 /lazyload-image.ts#L116 https://github.com/tjoskar/ng-lazyload-image/blob/7b2261798a13066f23c10f353a60e636d95bad2c/src/lazyload-image.ts#L128window
é indefinido. Estamos verificando window
em três lugares: https://github.com/tjoskar/ng-lazyload-image/blob/7b2261798a13066f23c10f353a60e636d95bad2c/src/lazyload-image.directive.ts#L21 https: // github. com / tjoskar / ng-lazyload-image / blob / 7b2261798a13066f23c10f353a60e636d95bad2c / src / lazyload-image.directive.ts # L75 https://github.com/tjoskar/ng-lazycroll-image/blob/7f353a60e636d95bad2c / src / lazyload-image.directive.ts # L75 https://github.com/tjoskar/ng-lazycroll-image/blob/7f2261798ebad63066dlist233cbcb .ts # L20platformId
de LazyLoadImageDirective
. Isso foi adicionado para detectar se foi renderizado pelo servidor (https://github.com/tjoskar/ng-lazyload-image/commit/7140a0f8f6dea1968c32bd84b7b90ede83803ed9), mas foi implementado de uma maneira diferente posteriormente.npx tslint -c tslint.json 'src/**/*.ts'
para obter os erros"strict": true
em tsconfig.json
e corrija os erros"noImplicitAny": true
em tsconfig.json
e corrija os errosSolicitações pull são mais que bem-vindas :)
Frio! Vou dar uma olhada nas primeiras 5 tarefas desta semana.
Comentários muito úteis
Frio! Vou dar uma olhada nas primeiras 5 tarefas desta semana.