Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: fillable SVG map dot #33

Merged

Conversation

DanielAgostinho
Copy link
Contributor

Versão svg dos pontos das estações. O ficheiro index.html mostra uma forma simples de implementação do conceito

@afonsosousah
Copy link
Owner

Simplifiquei e reorganizei um pouco do que ainda percebia. Ficou bastante fixe! Obrigado pela excelente contribuição!

@afonsosousah afonsosousah changed the title Mapdot svg version feat: fillable SVG map dot Mar 15, 2024
Copy link
Contributor Author

@DanielAgostinho DanielAgostinho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Com a formula ((1 - ratio) * 460).toFixed(0) e pela largura do traço, proporção de 0% a 7% e de 96% a 100% são indistinguíveis

@afonsosousah
Copy link
Owner

Com a formula ((1 - ratio) * 460).toFixed(0) e pela largura do traço, proporção de 0% a 7% e de 96% a 100% são indistinguíveis

então seria melhor manter uma casa decimal?

@DanielAgostinho
Copy link
Contributor Author

então seria melhor manter uma casa decimal?

Pode se arredondar às unidades ou às decimas, não é critico. O problema é que desta forma, o y varia entre 0 e 460, mas tal variação só é visível entre 18 e 430. Utilizando a expressão anterior 430 – ratio(430 – 18) ou uma função com os mesmos extremos eliminaria esta “saturação”

@afonsosousah
Copy link
Owner

então seria melhor manter uma casa decimal?

Pode se arredondar às unidades ou às decimas, não é critico. O problema é que desta forma, o y varia entre 0 e 460, mas tal variação só é visível entre 18 e 430. Utilizando a expressão anterior 430 – ratio(430 – 18) ou uma função com os mesmos extremos eliminaria esta “saturação”

então algo assim funcionaria? não tinha percebido a expressão inicial muito bem

// Calculate the Y coordinate at which the top of the "fill" rectangle will be
// (the filled area is only visible from y=18 to y=430)
const y = (18 + (1 - ratio) * (430 - 18)).toFixed(0);

@ImRodry
Copy link
Collaborator

ImRodry commented Mar 16, 2024

Com a formula ((1 - ratio) * 460).toFixed(0) e pela largura do traço, proporção de 0% a 7% e de 96% a 100% são indistinguíveis

Bem visto mas honestamente eu acho que isso é um “happy little accident” porque a essas percentagens pode perfeitamente ser um erro da Gira. Muitas vezes se está 1 doca livre provavelmente não está e é uma doca estragada ou uma bike não detetada a verde
Mesma coisa para 1 bike numa estação com muitas bikes livres, é muito provável que ela esteja estragada e por isso ninguém a tira. Ou isso ou alguém tira antes de lá chegares mas de qualquer forma aí terias o 1 para saber que ela lá está.

@afonsosousah afonsosousah added the enhancement New feature or request label Mar 16, 2024
@afonsosousah afonsosousah added this to the v0.0.5 milestone Mar 16, 2024
@afonsosousah afonsosousah merged commit 7f4043d into afonsosousah:main Mar 19, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants