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

Ajout d'un bandeau #5

Open
wants to merge 33 commits into
base: dev
Choose a base branch
from
Open

Ajout d'un bandeau #5

wants to merge 33 commits into from

Conversation

DridriLaBastos
Copy link

@DridriLaBastos DridriLaBastos commented Nov 17, 2020

Sauvegarde du travail pour pouvoir le récupérer sur mon mac et ajouter les icones.
J'essaie de faire le meme design que celui que m'a envoye @aemi-dev

AF288EBF-7E25-40B6-9653-A3556B04E4EC

J'ouvre la pull request comme ça vous pouvez voir en direct mes modifications

@DridriLaBastos DridriLaBastos requested a review from g-cqd November 17, 2020 20:39
@DridriLaBastos DridriLaBastos self-assigned this Nov 17, 2020
@DridriLaBastos DridriLaBastos added the enhancement New feature or request label Nov 17, 2020
@DridriLaBastos
Copy link
Author

DridriLaBastos commented Nov 18, 2020

Capture d’écran 2020-11-18 à 01 09 28

@DridriLaBastos
Copy link
Author

@aemi-dev il faudra que tu m'expliques comment tu changes d'image en fonction du thème, j'ai cherché mais je n'arrive pas encore à comprendre

@sonarcloud
Copy link

sonarcloud bot commented Feb 20, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot E 2 Security Hotspots
Code Smell A 31 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

samuelortion and others added 22 commits July 19, 2022 23:01
Removing tabs before the Markdown code marker to make the code for settings.json properly formatted
Sauvegarde du travail pour pouvoir le récupérer sur mon mac et ajouter les icones.
J'essaie de faire le meme design que celui que m'a envoye Guillaume
Uniquement visuel, j'ai fais en sorte d'avoir un responsive design
Il faut encore implémenter les fonctionnalitées
Implémentation de la fenetre modale pour l'emploi du temps.
La couleur de fond de la fenetre modale s'adapte en fonction du thème
(j'ai du modifie le fichier standard.css pour ca car pour le thème 'light'
le fond était transparent). Je n'ai pas pu tester que l'affichage de l'EDT
se faisait bien car firefox ne me laisse pas recevoir l'image
Sauf pour les icônes, je ne comprends pas du tout comment le changement d'images en fonction
du thème opère sur 'infoEvry' donc je ne peux pas copier son fonctionnement
- Suppression de Code JS inutile
- Correction de JS
- ajout de classes de ColorScheme pour les icones
- les icônes restnt à la même place sans utiliser de 'diplay: none'
- les fenêtres modales ne se ferment que en cliquant sur 'x' ou en dehors de la fenêtre
Deux serveurs HTTP et HTTPS sont démarrés avec node. Cela permet d'afficher l'emploi du temps
lorsque l'on se connecte au serveur http dans firefox (la version https ne fonctionne pas encore
donc pas encore d'affichage dans safari
some limitations are still here yet:

* Unable to click on the edt image to scale it up
* Not tested on different device screen
* Evenments are not blocked, clicking on any form in the edt close the modal window
Preparation of the edt scale up when the edt img is clicked, see 1eba273 (backing up my work before I accidentally break everything)
still few limitations:
* buttons not working
For now only a banner is pemanently displayed instead of a real API fetch
* The banner is not yet reponsible
* The goal is to display when an API request fails
@sonarcloud
Copy link

sonarcloud bot commented Aug 24, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 2 Security Hotspots
Code Smell A 11 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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.

4 participants