iamDistractech / jungle-gym

JungleGym is a simple, user-friendly app for group teachers who have little or no expertise in physical education and have little material at their disposal.

https://jungle-gym.netlify.app/

MIT License

5 stars 4 forks source link

Feature/docent kan snel mening geven #246

Closed jochemvogel closed 3 years ago

jochemvogel commented 3 years ago

Description

Closes #217

Dit is de front-end van het feedback geven. Onderaan de spelpagina kan er feedback gegeven worden. Dit wordt nog niet verstuurd naar de database. Dit komt in een later PR.

To do

[ ] Verberg het als iemand offline is
[ ] Check of JavaScript aan staat en hoe handel je dit goed af

Type of change

Please select the option that is relevant

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist

[x] I have commented my code, particularly in hard-to-understand areas.
[x] All the required documentation is added.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.

Additional information

[Provided extra information if necessary]

netlify[bot] commented 3 years ago

:heavy_check_mark: Deploy Preview for jungle-gym ready!

:hammer: Explore the source changes: 86c20d2017cb94e5e09d40025e13357cc2490595

:mag: Inspect the deploy log: https://app.netlify.com/sites/jungle-gym/deploys/60c1ebcc520c6c000757d652

:sunglasses: Browse the preview: https://deploy-preview-246--jungle-gym.netlify.app/spellen/offline

netlify[bot] commented 3 years ago

:heavy_check_mark: Deploy Preview for jungle-gym ready!

:hammer: Explore the source changes: a10bd2d852f1b4915aad552f2c93ed03c1d946ed

:mag: Inspect the deploy log: https://app.netlify.com/sites/jungle-gym/deploys/60c5f7a6242c0600076dce5c

:sunglasses: Browse the preview: https://deploy-preview-246--jungle-gym.netlify.app/spellen/offline

RooyyDoe commented 3 years ago

De Material Icon library heeft ook smilies die gebruikt kunnen worden. Misschien kan je die in plaats van gedownloade SVG's gebruiken. Op deze manier houden we de consistentie erin

jochemvogel commented 3 years ago

De Material Icon library heeft ook smilies die gebruikt kunnen worden. Misschien kan je die in plaats van gedownloade SVG's gebruiken. Op deze manier houden we de consistentie erin

Ik vind deze emojis wel een stuk mooier dan die material icon emojis, tbh.

RooyyDoe commented 3 years ago

Dan houden we ze zo! :)

jochemvogel commented 3 years ago

Is het misschien een idee om later een aparte route voor reviews te maken?

Voor de ALO studenten bedoel je dan, lijkt mij? Dat moet inderdaad ergens onder account komen staan.

jochemvogel commented 3 years ago

Deze doet nog niks en daarom is het ook niet handig om dit nu te mergen denk ik. Beetje onnodig om het er dan in te zetten.

Ik wil er misschien ook nog wel wat aan toevoegen bedenk ik me net: misschien kijken of iemand online is of dat javascript aan staat. Anders doet dit niet zo veel.