Skip to content
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

💻 new customize adventure page #5535

Merged
merged 46 commits into from
Jul 3, 2024
Merged

💻 new customize adventure page #5535

merged 46 commits into from
Jul 3, 2024

Conversation

hasan-sh
Copy link
Collaborator

@hasan-sh hasan-sh commented May 15, 2024

Fixes #5086

@hasan-sh
Copy link
Collaborator Author

@Felienne as discussed in the meeting, we need to decide what descriptions we want to put in the solution example and tips tabs:

image

NB: the solution example has the editor and the tips is empty!

Copy link
Member

@jpelay jpelay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Felienne told me to review this since she's busy.

Seeing the UI I think that the Preview button might be made available on the "Adventure" tab, rather than in the General settings one. I also think the "I agree that my adventure might be made publicly available on Hedy." section can be switched over to the settings tab.

image

On the solution example section we can put something like:

This is where the solution of your adventure goes. This can be used if you want to share this adventure with other teacher's, so they can know what your suggested solution is.

If both of these tabs have the editor, maybe we can add the help button to both of them as well

image

@hasan-sh
Copy link
Collaborator Author

hasan-sh commented Jun 26, 2024

@jpelay I like and agree with all suggestions!

@hasan-sh
Copy link
Collaborator Author

Will disable the tips section for now until it's needed and known what content to put in there!

@Felienne
Copy link
Member

Will disable the tips section for now until it's needed and known what content to put in there!

Disabling seems a bit of a heavy strategy?

@Felienne
Copy link
Member

Will disable the tips section for now until it's needed and known what content to put in there!

Disabling seems a bit of a heavy strategy?

I think we want something along the lines of: "Here you can add tips to help other teachers help their students"?

@hasan-sh
Copy link
Collaborator Author

I think we want something along the lines of: "Here you can add tips to help other teachers help their students"?

Sure. How would creating tips look like? Where and how do we list or represent them?

Disabling seems a bit of a heavy strategy?

I didn't get any note about it!! Also i thought we could implement it in a separate PR in accordance with the new hedy design that's being created!

@Felienne
Copy link
Member

I think we want something along the lines of: "Here you can add tips to help other teachers help their students"?

Sure. How would creating tips look like? Where and how do we list or represent them?

I am lost here, sorry. In the screenshot above, there is already a tab called "tips". I thought you were looking for text to put there to expain what the tips are for. Help me understand what text you need and I can provide it to you.

Please remember I have not looked at this PR, so if you need my input, you need to sketch a bit of context and ask me clear questions (Or ask Jesus, he has seen more of this work, and I am sure he has great ideas too!)

I didn't get any note about it!!

I think this has to do with the fact that I don't really understand the question well (again, I have not looked at the PR, just jumping in for being tagged) But surely we can do it in a different PR is that makes more sense, just open an issue so we don't forget.

@Felienne
Copy link
Member

Felienne commented Jul 1, 2024

And on a whole different note, I don't really agree with the design of the tabs. Not only because the new design for the Hedy tabs is not implemented yet, so until then teachers might be very confused. But still I think this can be done in a more understandable way. @Felienne @gisellandrade what do you think? Just addressing this now, but can of course be done in a different PR.

Hi @Annelein!

Do you want to make changes to these tabs (that are a bit different from the adventure tabs, because they are here representing different aspects of an adventure)? I am open to looking at what you want it to look like, but I can also imagine we redesign this with the new homepage together.

@Annelein
Copy link
Collaborator

Annelein commented Jul 2, 2024

Do you want to make changes to these tabs (that are a bit different from the adventure tabs, because they are here representing different aspects of an adventure)? I am open to looking at what you want it to look like, but I can also imagine we redesign this with the new homepage together.

Hi @Felienne ! I think it should be a different than the adventure tabs design, so not redesign it with the homepage but septeratly. Because these aren't adventure tabs but they have a whole different meaning/purpose right? Let me know if you get what I mean.

I was thinking something like this for example:
Screenshot 2024-07-02 at 13 49 16

@Annelein
Copy link
Collaborator

Annelein commented Jul 2, 2024

Hey @hasan-sh ! Looks good, I've added a few comments and was wondering, is the whole page covered in the Cypress tests now?

@hasan-sh what about the Cypress tests?

@hasan-sh
Copy link
Collaborator Author

hasan-sh commented Jul 2, 2024

Hi @Felienne ! I think it should be a different than the adventure tabs design, so not redesign it with the homepage but septeratly. Because these aren't adventure tabs but they have a whole different meaning/purpose right? Let me know if you get what I mean.

I was thinking something like this for example: Screenshot 2024-07-02 at 13 49 16

I like this since it aligns more with the new redesign, do you wanna push your commit here before we merge?

Copy link
Contributor

mergify bot commented Jul 3, 2024

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).

@mergify mergify bot merged commit bdbdfac into main Jul 3, 2024
12 checks passed
Copy link
Contributor

mergify bot commented Jul 3, 2024

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).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

💻 Allow teachers to add model examples to their custom adventures
4 participants