hedyorg / hedy

Hedy is a gradual programming language to teach children programming. Gradual languages use different language levels, where each level adds new concepts and syntactic complexity. At the end of the Hedy level sequence, kids master a subset of syntactically valid Python.
https://www.hedy.org
European Union Public License 1.2
1.26k stars 281 forks source link

[UI idea] Improvements to adventures reordering #3955

Open jpelay opened 1 year ago

jpelay commented 1 year ago

Idea incl level After #3854 is merged we need to address these improvements:

Originally posted by @Felienne

Now that we have the new layout, I think we can also move quiz and puzzle into the "dragger" instead of a tickbox below? We need to change the text for the enable/disable buttons (I can do that) If there are no adventures available, can we remove the drowdown? And empty one is a bit weird. We can even consider a button there with "add you own" We now have 2 places for all levels (in addition to the dragger) the top enable/disable and the dates list. Can we make that into one? Maybe just the date lists with a tick mark?

Also we need to add a Cypress test for the new drag & drop functionality!

Additional context Add any other context about the idea here.

Felienne commented 3 months ago

Hi @Annelein!

Maybe many of these (or all?) have been fixed already? If so feel free to close right away.

Annelein commented 1 month ago

@jpelay I don't think there is an Cypress test for dragging the adventures right?

And then all thats left is this: If there are no adventures available, can we remove the drowdown? And empty one is a bit weird.

jpelay commented 1 month ago

@jpelay I don't think there is an Cypress test for dragging the adventures right?

No, we used to use our own solution for the dragging and dropping, but we later switched to a library and it doesn't work welk with Cypress.

And then all thats left is this: If there are no adventures available, can we remove the drowdown? And empty one is a bit weird.

I think I can tackle this in #5540 or maybe in another PR but using that same element.

Annelein commented 1 month ago

@jpelay allright! Should I just create a new issue for removing the dropdown? And also a new issue for testing the drag and drop for in the future?

Then we can close this one.

Annelein commented 1 week ago

@jpelay allright! Should I just create a new issue for removing the dropdown? And also a new issue for testing the drag and drop for in the future?

Then we can close this one.

@jpelay 👆🏼