-
Notifications
You must be signed in to change notification settings - Fork 289
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
💻 Redesign of Adventure page within tryit endpoint #5648
Conversation
A few good points that @AnneliesVlaar mentioned in #5517 , I've also added some Few points of discussion:
Missing in the design:
Please leave a comment if there is something else you're missing or that needs to be discussed Screen.Recording.2024-07-03.at.15.13.08.mov |
Ow that is a great idea, I would love a copy button that changes into a paste button on theory. However in practice, no other tool has that (as fas as I know) so I fear it will be more confusing if we do it. We can disable the copy button if something has already been copied? But I also see how we can, for now, just aline with other tools do with is nothing.
This is an unrelated interesting question! We have long discussed pasting at the end instead of over for the yellow example buttons. I am fine with that but maybe it is out of scope here? Do we need to discuss that now?
Another very hard question!!!! I think it does not look nice that the explanation "hops" when switching tabs. However kids might not care so much? I fear that they will not scroll and then miss important explanation if we don't show it all (but on the other other hand, maybe they never read?) In conclusion I think I'd keep it as we have it, but I can easily be swayed. |
I can see that an all-in-one copy/paste button can be confusing. I was wondering, if you copy your code (to use it in a different level/adventure I guess), how do you paste it there without a paste button. Can we easily teach student to use a button to copy and
I will make separate issue about the yellow button.
|
@Annelein Here's the new buttons for the design and new modifications |
As discussed in the meeting, if @boryanagoncharenko does a sanity check and merge, @MarleenGilsing can test it on alpha! |
Yes! The url to try the new design is: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is a big PR, did my best to go through the code! LGTM
Thank you for contributing! Your pull request is now going on the merge train (choo choo! Do not click update from main anymore, and be sure to allow changes to be pushed to your fork). |
Thank you for contributing! Your pull request is now going on the merge train (choo choo! Do not click update from main anymore, and be sure to allow changes to be pushed to your fork). |
Yes, it's huuuuuge. Ill do my best to test thoroughly on Alpha! |
Thanks for the superextensive review Marleen!!! I marked the PR tobediscussed again so we can go over the issues in next week's meeting |
You're welcome! My screen resolution is 1536x864 |
Fixes #5535
https://www.figma.com/design/wrT20i7nKRBm66eVrf5A8F/Class-%2F%2F-Adventures-ideas?node-id=111-107&t=WR6z6PvgzdUO63sk-0