135 aplicar novo layout #168
Conversation
…stilo página de detalhes de ocorrências.
|
Finalmente o layout saiu. Galera, revisa pra mim, por favor? Para disponibilizar alguns dados na tela, eu precisei criar funções auxiliares em controllers, mas quero refatorar toda a bagunça. Enquanto isso, dá pra ir acompanhando o visual, rodar a aplicação localmente em busca de problemas visuais para reportar. Vamos conversando. |
| @@ -1,4 +1,4 @@ | |||
| describe('placeDetailsController', function() { | |||
| xdescribe('placeDetailsController', function() { | |||
There was a problem hiding this comment.
Esses testes serão consertados antes de mergear?!
There was a problem hiding this comment.
Consertados não, descomentados sim. Sugere que retiremos?
There was a problem hiding this comment.
Sugiro que refaçamos eles haha
There was a problem hiding this comment.
😏, é claro, jovem. hahahaha
tô perguntando o que faremos agora, nesse momentinho 😛
There was a problem hiding this comment.
eu preciso de outra issue (talvez a partir dessa branch) para consertar esses testes. quero cobrir todos os métodos que adicionei e também quero consertar mais um monte
scss/ionic.app.scss
Outdated
| border-color: #cc3399; | ||
| } | ||
| } | ||
|
|
||
| button { | ||
| with: 100%; |
There was a problem hiding this comment.
Não relacionado diretamente à PR, mas a regra with está inválida. Creio que seria width. Talvez já desse para corrigir junto?
There was a problem hiding this comment.
Muchas gracias, meu jovem. Acho que já corrigi algo assim em outro lugar. Corrijo aqui.
This reverts commit 1b3e455.
ebragaparah
force-pushed
the
135-aplicar-novo-layout
branch
from
f8ba786 to
fd93259
Compare
Tem a ver com essa tarefa, pessoal.
#130