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

Add wind direction icon for forecasted figures #108

Closed
wants to merge 27 commits into from
Closed

Add wind direction icon for forecasted figures #108

wants to merge 27 commits into from

Conversation

vingerha
Copy link

Adds a (new) icon indicating the wind direction, if wind_bearing is available and not null (so in current reality only with hourly forecast)

@WarC0zes
Copy link

@vingerha
Bonjour, j'ai remarquer que quand wind_bearing: null dans les attributs, il n'y a pas d'icone. Normal tu me diras, mais serait t'il possible de mettre une icone ( genre une croix ), pour que ca soit plus homogène sur la carte.
C'est une idée, a toi de voir.

icone null

@vingerha
Copy link
Author

Added
image

@WarC0zes
Copy link

WarC0zes commented May 11, 2023

Added image

Mais quel rapidité :)
Au top !

@dx44
Copy link
Member

dx44 commented May 11, 2023

je verrai encore un petit ajout afin de mettre un switch dans le paramétrage de la carte pour afficher ou non ces icônes car ils prennent de la place et potentiellement certains utilisateurs ne souhaiteraient pas les afficher.

@dx44
Copy link
Member

dx44 commented May 11, 2023

Et l'idée de cette issue était surtout de mettre un icône dans le détails des infos et non dans les prévisions horaires.

@dx44 dx44 linked an issue May 11, 2023 that may be closed by this pull request
@dx44 dx44 added the enhancement New feature or request label May 11, 2023
@vingerha
Copy link
Author

Pour les détails: Car le wind_bearing n'est que fournis pour les prev. horaires, pas de chance sans changer l'intégration MF

@vingerha
Copy link
Author

je verrai encore un petit ajout afin de mettre un switch dans le paramétrage de la carte pour afficher ou non ces icônes car ils prennent de la place et potentiellement certains utilisateurs ne souhaiteraient pas les afficher.
...d'accord pour 'trops de place', je vais voir si facile ou non (neuf pour moi)

@WarC0zes
Copy link

Pour les détails: Car le wind_bearing n'est que fournis pour les prev. horaires, pas de chance sans changer l'intégration MF

Bonjour @vingerha,
il y a quand même un attribut wind_bearing , or des heures et jours.
wind bearing detail
Tu peu pas exploiter c'est attribut pour le détails ?

@vingerha
Copy link
Author

vingerha commented May 12, 2023

Curieux, rien chez moi, ni journalier ni horaire
EDIT, c'est bien integration MF d'ou tu parle?

image
Je ne le fait pas dans cette PR et pas sûre que ça sera assez facile car dans ce section on utilise des icones HA de base.
A côté stp tester le switch 'Show Wind Icons' (release 0.5)

@vingerha vingerha changed the title Add wind direction icon Add wind direction icon for forecasted figures May 12, 2023
@WarC0zes
Copy link

WarC0zes commented May 12, 2023

c'est bien integration MF d'ou tu parle?

Oui, c'est bien météo france.

une entité météo en daily:
wind bearing detail daily

une entité en hours:
wind bearing detail

J'ai bien wind_bearing sur les deux.

A côté stp tester le switch 'Show Wind Icons' (release 0.5)

C'est déja fait, c'est OK pour moi. Le switch et bien la et active ou désactive les icones du vent.
switch vent

@dx44
Copy link
Member

dx44 commented May 12, 2023

Je ne vois rien dans le code de cette PR ?

@vingerha
Copy link
Author

? ... 3 fichiers

image

@dx44
Copy link
Member

dx44 commented May 12, 2023

Les 3 fichiers sont les modifs pour l'affichage de la flèche dans les prévisions horaires.
Je ne vois pas de changement dans l'editor pour le switch on/off de l'icône comme le montre la capture de @WarC0zes !?

@vingerha
Copy link
Author

terminating as I made a mess of the code, will redo separately

@vingerha vingerha closed this May 12, 2023
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
None yet
Development

Successfully merging this pull request may close these issues.

Ajouter un icone de flèche indiquant l'orientation du vent
3 participants