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

UI related issues and suggestions #21

Open
DannyAcosta710 opened this issue Oct 27, 2016 · 43 comments
Open

UI related issues and suggestions #21

DannyAcosta710 opened this issue Oct 27, 2016 · 43 comments

Comments

@DannyAcosta710
Copy link

Both PC (when the player window is small) and mobile, the transparent hamburger menu makes it so that it's hard to read the options
image

It's even more noticeable on mobile, however
wp_ss_20161027_0002

@thecodrr
Copy link
Owner

@DannyTalent I made the whole menu a solid color instead of transparent for precisely this reason. What's do you say? What color or combination would be better suited?

@DannyAcosta710
Copy link
Author

The current color looks fine, just not transparent. Maybeeee blur would make it look better. Accent color would probably work fine, too, and/or a really dark grey, kinda like what the old Settings app for W10 had (R: 23 G: 23 B: 23) with white text.

PS: I noticed some files from source code are in spanish, would you prefer me to talk in spanish? Cuz I speak spanish too lol

@thecodrr
Copy link
Owner

No, no. English is absolutely fine 😄

The blur would have a very adverse effect on performance. Maybe I will try solid white-grey color or we can simply use accent color. Later on, we can make a setting for this.

@DannyAcosta710
Copy link
Author

Alright

Really? I mean myTube uses it (beta) on mobile, and it isn't really that slow... And the blur changes with the video, so I imagine there's quite a lot of processing happening..
Yeah, dark/light mode setting would be very nice

@thecodrr
Copy link
Owner

Even so, this is a rather a risky thing and nothing we can't do without. I think we should reduce the transparency to a minimum and use the same setting.

@DannyAcosta710
Copy link
Author

Right. What about moving the whole screen to the right? As in, not making the menu, instead of coming from the left and overlapping on top of the screen. I think the ESL app does this

@thecodrr
Copy link
Owner

You mean just overlapping the whole of the screen with the menu in the middle?

@DannyAcosta710
Copy link
Author

Well, that could work, too, I was thinking of moving the normally visible controls to the right, kinda like the old wp8.1 pivots... But yeah what you said sounds like a good idea. And probably easier to code!

@thecodrr
Copy link
Owner

The search bar will have to be moved to somewhere more convenient. Besides that, all is well.

@thecodrr
Copy link
Owner

inline

Did you mean this?

@DannyAcosta710
Copy link
Author

Exactly that, yeah

Ot: Hey if I made a small svg mock up of the app and suggested a few things (design wise) would you consider it? I really wanna help with the development of this app, it's pretty fun

@thecodrr
Copy link
Owner

Of course, I would love that! Actually I would really appreciate that.

@DannyAcosta710
Copy link
Author

Alrighty, cuz I feel that some transparent dark screens would make it look a bit better but I'm not so sure, unless I see it. In a few days I'll send you one if I have something decent!

@thecodrr
Copy link
Owner

Perfect!

@thecodrr
Copy link
Owner

@DannyTalent I was thinking of moving the player bar to the bottom like in Groove. What do you say?

@DannyAcosta710
Copy link
Author

I think the current layout on mobile is perfect, maybe you can put it on PC as well!

@DannyAcosta710
Copy link
Author

Although it might be bad for ux... What about making it stay on top but have the buttons (next, pause, prev, shuffle, etc.) be on top of the progress bar so that there's some extra space on top

@thecodrr
Copy link
Owner

I changed the mobile layout a bit, but basically its the same. I cannot transfer it to PC because I don't think it will look good on bigger screens.

What will we do with the extra space? Not a bad idea, btw.

@DannyAcosta710
Copy link
Author

You said you tried putting it on top but couldn't because of problems with the action center.. That extra space would help not accidentally swiping it when trying to use the seek bar

@thecodrr
Copy link
Owner

The problem with action center occurs on mobiles because of the notification bar.

I hid the whole play bar to free up all the space I could, I think its more...user-friendly that way. For two reasons: first that a user is not playing with the play bar all the time, so when he does want it he can press the button and voila!

The second reason is the hamburger menu button. It looks awful when placed right below the play bar destroying the whole beauty of the thing.

@thecodrr
Copy link
Owner

As to PCs, the second reason applies + the rectangle where currently playing song info is now showed. Placing the whole thing at the bottom will on one side make it more appropriate and on the other side we will have to get rid of the status bar. I have seen status bar right below the play bar and it looks weird.

@DannyAcosta710
Copy link
Author

Ohhh I see... It might be a little hard to discover for an average user though if it's hidden. (Never underestimate end users!) Maybe a little animation later in development might help with that tho.

Which status bar? The one that says Nothing baking? What if you have it appear and disappear, kinda like a small, thin toast notification? Only when it's needed of course, or maybe with an X button to close it...

@thecodrr
Copy link
Owner

How about if its visible on first use with the button to hide? That way the user will know that there is a playbar :)

@thecodrr
Copy link
Owner

@DannyTalent I have released a new alpha update, go check it out!

@thecodrr thecodrr changed the title Bad visibility on hamburger menu options UI related issues and suggestions Oct 28, 2016
@thecodrr
Copy link
Owner

inline

Do tell how it looks? Better or worse?

@DannyAcosta710
Copy link
Author

Yes! Loving that placement! I did get the new alpha for my phone, ty! You might wanna reopen the double song issue tho :/ it's still happening...

@thecodrr
Copy link
Owner

Strange, it wasn't appearing in the emulator at all.

@DannyAcosta710
Copy link
Author

Yep... I'll record a video tonight to show you exactly what's happening.. Can't earlier because of classes :/

@thecodrr
Copy link
Owner

Yes! That would certainly help a lot.

@DannyAcosta710
Copy link
Author

wp_ss_20161101_0001
Controls don't show up when you just opened up the app. Even if you start playing a song. Closing and opening the tab fixes it, however

@thecodrr
Copy link
Owner

thecodrr commented Nov 1, 2016

yes, I missed those figures out. Fixing it now.

@MightyK1337
Copy link

MightyK1337 commented Nov 2, 2016

While using the dark theme on the phone, the letters for jump lists and the jump list is not visible.

@thecodrr
Copy link
Owner

thecodrr commented Nov 2, 2016

@MightyK1337 Reproduced and fixed. There was also a crash while jumping to a specific letter.

@DannyAcosta710
Copy link
Author

http://imgur.com/a/CPj5y (can't upload here, on mobile) it's from bottom to top, the 1st one shows landscape mode, a little awkward with the song info and the hamburger menu... Then, when u go back to portrait, the hamburger button from the playlist, which usually is transparent (as in the top pic), goes red (accent color).

@thecodrr
Copy link
Owner

thecodrr commented Nov 7, 2016

Yes. This is certainly a bug in the UI adaptation to different screens. Fixing right away!

@MightyK1337
Copy link

Alphabets for jump lists not visible.

@thecodrr
Copy link
Owner

@MightyK1337 In dark theme or light?

@MightyK1337
Copy link

It is the dark theme.
Also - don't know if this is a issue- when the first time i play something and navigate from the app, song stops playing. I need to go back to app and then navigate to others for it to work in background.

@thecodrr
Copy link
Owner

I will surely investigate this further.

@thecodrr
Copy link
Owner

@MightyK1337 Can you explain the exact steps to reproduce this new issue?

@MightyK1337
Copy link

Sorry for the late reply.

  1. Play a song
  2. navigate away from the app like pressing the home key
  3. the song stops playing
  4. navigate back to the app, by pressing the back key or from task switcher
  5. song starts playing again.
  6. now navigate away from the app again.
  7. the song will continue to play in background

Expected behavior: song should continue to play in the background,even when navigating from the app for the first time.

@MightyK1337
Copy link

wp_ss_20161203_0001_ink_li

Remove the now playing bar highlighted, when nothing is being played. We can look at more number of songs, if it is removed. also improves usability

@MightyK1337
Copy link

wp_ss_20161203_0002_ink_li
This arrow appears after using the jumplist. pressing it does not work and after this appears, the jump list also stops working.

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

No branches or pull requests

3 participants