search

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.32k stars 289 forks source link

Change the place of the 'succesfully saved' message? #677

Closed MarleenGilsing closed 3 years ago

MarleenGilsing commented 3 years ago

I don't work on a big screen (like most schoolkids I reckon), so I don't see the message that I've succesfully saved my program. I need to scroll down to see it, especially in the higher levels where the instructions are getting longer.

I would like to see it when I click the button, so I don't click it 5 times.. Would it be possible to move the message closer to the save button?

Felienne commented 3 years ago

@fpereiro This is a good point Marleen makes, can you move this into the unified modal too?

fpereiro commented 3 years ago

@Felienne should I leave also the old green notification in place, or remove it altogether?

Felienne commented 3 years ago

Let's remove the green box all together (although we can at one point make the modal have colors too).

Is it possible in the modal that we have now to dissapear after a while (rather than requiring a click)? That would be nice.

fpereiro commented 3 years ago

Yes, we can piggyback a timer to close it into this issue, as long as it doesn't require user input.

чт, 26 авг. 2021 г. в 13:28, Felienne @.***>:

Let's remove the green box all together (although we can at one point make the modal have colors too).

Is it possible in the modal that we have now to dissapear after a while (rather than requiring a click)? That would be nice.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Felienne/hedy/issues/677#issuecomment-906322183, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI3GUXKX22SLFSI47UT5U3T6YQMRANCNFSM5CZHRHJA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

fpereiro commented 3 years ago

I think we can close this after merging #754 :)