Conversation
Co-authored-by: jitsedesmet <35114273+jitsedesmet@users.noreply.github.com>
|
@copilot nice work, but I think entering this page should not be part of the 'deel sleutel filtern' dropdown. Maybe we can have a button 'bekijk paddenstoelen in deelsleutels' on the main page which routes to a page with a list of all subKeys. |
Co-authored-by: jitsedesmet <35114273+jitsedesmet@users.noreply.github.com>
Done in commit Here's the flow in brief:
|
|
@copilot this is really nice!
|
Co-authored-by: jitsedesmet <35114273+jitsedesmet@users.noreply.github.com>
... Fixed in |
|
@copilot you may undo your work on supporting the waarnemingen.be images. We will tackle that issue in another PR. Additionally, it would be nice if the list overview would have the same quick links as the mushrooms page itself:
|
Co-authored-by: jitsedesmet <35114273+jitsedesmet@users.noreply.github.com>
Done in
|
|
@copilot viaually, the image and its Copyright sometimes bug out. Can we do some thing about that?
|
Co-authored-by: jitsedesmet <35114273+jitsedesmet@users.noreply.github.com>
Fixed in |
|
@copilot nice, but lets still put credits somewhere with the image. |
Co-authored-by: jitsedesmet <35114273+jitsedesmet@users.noreply.github.com>
Done in |
jitsedesmet
left a comment
There was a problem hiding this comment.
@copilot Nice work. Small question and path forward
src/lib/OneZoomPicture.svelte
Outdated
| export let credits: boolean = true; | ||
| /** When true, renders the credit text as a translucent overlay at the | ||
| * bottom of the image instead of below it. Useful inside fixed-size | ||
| * containers where a figcaption below the image would overflow. */ | ||
| export let creditsOverlay: boolean = false; |
There was a problem hiding this comment.
Nice addition! I like the overview look. I think it can replace 'credits' entirely? Since it is used no where?
There was a problem hiding this comment.
Done in 4577b51. Removed the credits prop entirely — creditsOverlay is now the only prop. Updated all three call sites: the mushroom identification page passes creditsOverlay={true}, the saved page drops the old credits={false} (default is no credits), and the species list is unchanged.
… prop Co-authored-by: jitsedesmet <35114273+jitsedesmet@users.noreply.github.com>


Users had no way to browse all species within a sub-key (e.g. "all ink caps") without stepping through the full decision tree.
Changes
Sub-key index page (
/9789050117548/soorten)Species list page (
/9789050117548/soorten?key=<subkey>)Graph traversal utility
Added
getMushroomsForSubKey()toparser.ts— recursively walks the question graph from a sub-key entry and returns all terminal mushroom species, sorted alphabetically:Home page
Added a "Bekijk paddenstoelen in deelsleutels" button that routes to the sub-key index page (
/9789050117548/soorten).Original prompt
💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.