Lots of UX changes + feed component

Scratch-Client-4 / itchy-ionic

📱 A mobile app for Scratch

https://itchy.micahlindley.com

23 stars 4 forks source link

Lots of UX changes + feed component #36

Closed micahlt closed 3 years ago

micahlt commented 3 years ago

In this pull request:

Better messages
Combined user, studio, and project search
Feed (aka What's Happening) component for logged in users on the explore page
Fixed stretched Itchy logo on auth modal
Friendly timestamps for messages
Much better UI for project pages
Love and favorite projects for logged in users
Unread indicators for messages

micahlt commented 3 years ago

@locness3 I'd really appreciate a review before I merge this, but if you're too busy everything appears to be fine on my end.

moeenio commented 3 years ago

@micahlt do you have any apk? I mostly gave up on developement for this project and i don't have an available dev environement.

micahlt commented 3 years ago

@micahlt do you have any apk? I mostly gave up on developement for this project and i don't have an available dev environement.

Yep - let me zip one up and upload it.

micahlt commented 3 years ago

Here's the APK inside a zip (GitHub doesn't let me upload straight APK files).

0-8-7.zip

moeenio commented 3 years ago

Explore page is stuck on infinite loading when signed out, for me.
A list with square rounded pictures is IMO not suitable for searching for projects. Thumbnails like on the Explore page should be used (and searching for users/projects/studios should be seperated).
Pressing system back button on the project page closes the app (i think that's related to the fact we use modals? #6)
IMO, Feed deserves its own tab with more listed items

moeenio commented 3 years ago

Also, no way to mark messages as read.

micahlt commented 3 years ago

Explore page is stuck on infinite loading when signed out, for me.

Same problem - thanks for investigating!

A list with square rounded pictures is IMO not suitable for searching for projects. Thumbnails like on the Explore page should be used (and searching for users/projects/studios should be seperated).

So just remove the border radius?

Pressing system back button on the project page closes the app (i think that's related to the fact we use modals? #6)

That is an issue, but not related to this PR.

IMO, Feed deserves its own tab with more listed items

I agree, but again that's a feature that needs to be added later on.

Also, no way to mark messages as read.

Another feature coming soon.

moeenio commented 3 years ago

So just remove the border radius?

Well, no :

Thumbnails like on the Explore page should be used (and searching for users/projects/studios should be seperated).

i.e. a Scratch project search shouldn't look like the Contacts app

micahlt commented 3 years ago

i.e. a Scratch project search shouldn't look like the Contacts app

Ah, I see - okay. Well, that's a much harder thing to do since the ProjectThumbnail (I think that's what I named it) component doesn't take screen width into account. For now, this search will have to do and I'll work on the UI later.

micahlt commented 3 years ago

All right, everything is working now. Merging!