Aanpassen form 27 - #91
Conversation
Kitkatisvibing
assigned
Kitkatisvibing,
ThomasSeisveld,
roxyfokker,
Kayleighhhhh and
JulianDavelaar and unassigned
Kitkatisvibing
|
top! ziet er goed uit! ik heb wel een paar dingetjes met de styling gevonden 
de kleur van naam is anders dan die 2 eronder, ook staat er 'naam aanpasser' het is minder verwarrend als je dit houdt op 'naam' |
JulianDavelaar
left a comment
JulianDavelaar
left a comment
There was a problem hiding this comment.
voor de rest alles top! vooral die try/catch&finally vind ik heel nice!
| {% block content %} | ||
| <main> | ||
| <form class="form aanpassen" action="/instrumenten/{{ instrument.key }}/aanpassen" method="post"> | ||
| <legend>instrument aanpassen</legend> |
There was a problem hiding this comment.
legend is alleen geldig binnen
zonder dat is het geen geldige HTML en werkt het niet in alle browsers.je kan de velden wrappen in een fieldset of H2 gebruiken.
| </label> | ||
|
|
||
| <label>wijzig merk | ||
| <input type="text" name="merk" placeholder="Nieuw merk"> |
There was a problem hiding this comment.
je input names kloppen niet met de API, merk is bijv: 'brand'
Check https://fdnd-agency.directus.app/items/preludefonds_instruments voor de juiste field names. tenzij je in je server route gebruik maakt van: { brand: request.body.merk } maar dat is verwarrend.
|
|
||
| // Succes state | ||
| aanpassenBtn.classList.remove("loading") | ||
| aanpassenBtn.textContent = "✅ Aanpassing is doorgevoerd!" |
There was a problem hiding this comment.
je succes state heeft nog geen reset, als de JS wel zou werken en iemand zou de succes state krijgen dan blijft ie staan op "✅ Aanpassing is doorgevoerd!" je kan een simpele 'setTimeout' erbij zetten van 2 seconden zodat hij altijd terug gaat.
5 participants
Wat is er veranderd?
Het schade formulier, alles hiervan behalve de server.js
Bij welke issue hoort deze pull request?
Link de issue(s), bijvoorbeeld: #27
Is deze issue hiermee ook opgelost? Nee, er is nog geen server.js code om het functioneel te maken
Hoe wil je dat dit gereviewd wordt?
Geef aan waar reviewers op moeten letten, bijvoorbeeld:
Eventuele live link, screenshot of bronnen:
RAPPE Principles
Hoe heb je deze site getest?
Beschrijf hoe je hebt getest, bijvoorbeeld:
ik heb getest op accessibility