fix border and opacity lightshadow #10
Conversation
|
Teria como adicionar imagens do antes/depois, para documentar a alteração? Obrigado pela contribuição! |
| DrawOuterShadow(context); | ||
|
|
||
| SetPaintColor(context); | ||
| //SetPaintColor(context); |
There was a problem hiding this comment.
Porque precisou remover a busca pelo paint color?
Eu criei essa parte para garantir que a cor de Background seria aplicada no controle, pois como o paint é passado por referencia, muitas vezes alteramos alguns parametros.
Um exemplo disso é a NeoProgressView
There was a problem hiding this comment.
Vou refazer alguns testes, achei que ele estava redundante, reaplicando o efeito duas vezes
| { | ||
| var fShadowDistance = Convert.ToSingle(ShadowDistance); | ||
| var darkShadow = Color.FromRgba(DarkShadowColor.R, DarkShadowColor.G, DarkShadowColor.B, Elevation); | ||
| var lightShadow = Color.FromRgba(LightShadowColor.R, LightShadowColor.G, LightShadowColor.B, Elevation); |
There was a problem hiding this comment.
Interessante, imaginei que o light Shadow precisaria ser sempre solido, pois no neomorfismo temos esse efeito de "continuidade" entre o fundo e o controle. Porém, faz sentido alterar a "força" dessa sombra com relação à elevação.
Seria interessante uma imagem para visualizar o efeito com alteração dessa opacidade.
There was a problem hiding this comment.
A verdade é que precisamos diferencia-la, porque caso o fundo seja estilo dark, precisamos de mais escuro e menos claro, e caso o fundo seja light, precisamos justamente do contrário, menos intensidade no preto, e mais intensidade no branco
There was a problem hiding this comment.
Sim, mas para isso está aberta a possibilidade de alterar a cor, podendo utilizar um hexa decimal por exemplo.
O Elevation é a aplicação da opacidade em cima dessa cor ja definida, para dar ao programador uma ferramenta que possa utilizar nesse sentido. Criando efeitos de diferentes relevos ou animações.
Acredito que não seja o caso aplicado ao LightShadow
Co-authored-by: Felipe Baltazar <[email protected]>
Co-authored-by: Felipe Baltazar <[email protected]>
2 participants
Overview
Issues Resolved
API Changes
None
Platforms Affected
Behavioral Changes
None
Testing Procedure
PR Checklist