Scratch-Client-4 / itchy-ionic

📱 A mobile app for Scratch
https://itchy.micahlindley.com
23 stars 4 forks source link

Design improvements, loves, and favorites #33

Closed micahlt closed 3 years ago

micahlt commented 3 years ago

An overview of this PR:

You can test this version of Itchy using the APK in the zip below: 0-7-4.zip

moeenio commented 3 years ago

0.7.4? Imo it's a pretty major release

Glad to see you're working on this again.

micahlt commented 3 years ago

0.7.4? Imo it's a pretty major release

That's fair. I was going to bump up to 0.8.0 when I added studios, but do you think now is a more appropriate time? Also, any other feedback before I merge?

Glad to see you're working on this again.

Things have gotten kind of hectic with my job, plus I'm trying to find out where I'm going to college. I'm trying to get this done before I graduate 😅

moeenio commented 3 years ago

do you think now is a more appropriate time

Many small improvements is better than a rushed major feature.

Also, any other feedback before I merge?

Looking at it tomorrow (GMT+1).

micahlt commented 3 years ago

Awesome - thanks a lot!

moeenio commented 3 years ago

Looks good overall!

I'd use some blinking placeholders for when a project is loading though, instead of the "loading... :D". But that might be for another time.

Also the text in the FAB on the project page is not legible in light mode :

(This screenshot uses an unshared project, opened from the My stuff page on the website; handling of those could be improved)

moeenio commented 3 years ago

Also, using "open your profile" in the settings loads https://scratch.mit.edu/users/undefined 's profile :p

micahlt commented 3 years ago

Cool, thank you! I'll fix the major issues related to the features in this PR (blinking placeholders is just a design thing that could take a while) and merge.