Malopieds / InnerTune

A fork of InnerTune, a Material 3 YouTube Music client for Android
GNU General Public License v3.0
490 stars 21 forks source link

Persistent Search Bar #26

Open SiddhantRakh opened 7 months ago

SiddhantRakh commented 7 months ago

Checklist

Steps to reproduce the bug

Search something. Scroll to view more results. The search bar stays on top.

Expected behavior

The search bar should hide as it does on the home screen.

Actual behavior

The search bar persists looking a bit odd.

Screenshots/Screen recordings

Screenshot_20240329-200117_InnerTune Screenshot_20240329-200127_InnerTune

Logs

Sorry I can't figure out how to provide logs.

InnerTune version

0.5.5

Android version

Android 12

Additional information

No response

Malopieds commented 7 months ago

Good catch, I'll try to investigate !

W1zardK1ng commented 7 months ago

Just a suggestion. If we could make it search bar section persistent with dark background, we can access the options below the search(All, songs, videos etc) even if we scrolled down.

Malopieds commented 7 months ago

Well, there are 2 options, we either:

SiddhantRakh commented 7 months ago

Well, there are 2 options, we either:

  • keep the idea where the search bar and filter are hidden when scrolling down, but if we scroll up both reappears (like in the home screen for example)
  • we make it always accessible but with a background not transparent

Well I personally feel that making it always accessible is better option as most of the options of search are associated with the search bar and having it stay at top with a nice fade behind will definitely be better both functionally and visually.

BUT I strongly recommend implementing this behaviour everywhere inside the app i.e. on both home screen and the search screen is the way to go. As most of the sorting options in the artists and playlists tab makes more sense to be visible all the time.

Lastly whatever option to choose is your preference but whatever gets implemented should be done on both home and search screen just for the same of simplicity and uniformity.

Malopieds commented 7 months ago

Yes I agree with you on the fact that it should be consistent. I will try to implement it as soon as I can.

SiddhantRakh commented 7 months ago

Thanks. I hope that the suggestions I make turn out to be helpful for the app.