geen idee knop toegevoegd bij invoervragen#175
geen idee knop toegevoegd bij invoervragen#175JesseHoekema wants to merge 2 commits intopolarnl:stablefrom
Conversation
|
LGTM, maak het ook zo dat het hetzelfde doet als je niks invult maar wel op enter of "Controleer" drukt |
Ik zal later even kijken |
|
wrm niet gewoon een defalt value van |
??? |
|
geloof het of niet het niks invulen doet niks is een functie geloof ik |
|
leerinstellingen |
keyword: geloof ik |
|
in het geval dat het niet werkt kan je beter bij de functies zelf het probleem oplossen ipv een extra functie hebben die dingen anders doet |
|
@copilot zou de coden kapot gaan als we de check voor dat de string een waarde heeft kapot gaan als we die zouden verwijderen in learntool |
|
@andr3i1010 waneer word dit eens gemerged |
There was a problem hiding this comment.
Pull request overview
Deze PR voegt een “Geen idee” actie toe bij invoervragen in de leer-tool, zodat gebruikers kunnen aangeven dat ze het antwoord niet weten en alsnog door kunnen gaan.
Changes:
- “Geen idee” knop/link toegevoegd in test en hints mode.
- Nieuwe
handleDontKnowhandler die de vraag als fout markeert en de result overlay toont. - Brede formatting/quote-style aanpassingen in
LearnTool.tsx.
| import { motion, AnimatePresence } from "motion/react"; | ||
| import { saveLearnSession } from "@/utils/saveLearnSession"; | ||
| import type { ListStoreState } from "./listStore"; | ||
| import { useRouter } from "next/navigation"; | ||
| import { Button } from "../ui/button"; |
There was a problem hiding this comment.
Button is imported from ../ui/button but never used. Remove the unused import (or replace the plain <button> below with the shared Button component) to avoid lint/build failures in stricter TS/ESLint setups.
There was a problem hiding this comment.
@copilot apply changes based on this feedback
There was a problem hiding this comment.
apply changes based on this feedback
dit kan niet voor cross repo pull requests @JesseHoekema doe het zelf
There was a problem hiding this comment.
of doe dit in je eigen repo en dan update de pr vanzelf
| const handleSubmit = () => { | ||
| if (!currentWord || !userInput.trim()) return; | ||
| const answer = currentWord["2"] || ''; | ||
| if (!currentWord) return; | ||
| if (!userInput.trim()) { | ||
| handleDontKnow(); | ||
| return; | ||
| } |
There was a problem hiding this comment.
handleSubmit now treats an empty/whitespace input as “Geen idee” by calling handleDontKnow(). This changes prior behavior (empty Enter/"Controleer" becomes a logged wrong answer) and can be triggered accidentally. Consider restoring the early return for empty input and only invoking handleDontKnow from the explicit "Geen idee" action.
There was a problem hiding this comment.
zit wat in, doe anders met 2x enter pas
|
@JesseHoekema aan de slag |
|
vibe coding D: |
|
👍 |
|
ik vind dit niet tellen als vibecoden tbh |
|
en jullie gebruiken copilot om dit te reviewen dus |
ja nou dat is het wel als je met copilot het toepast maar dan kan je het beter hebben over vibecode tolerantie |
Een geen idee knop link dinges toegevoegd als je het antwoord niet weet
