Lmms: Move search and refresh fields above file navigator

[Mark-Agent003]

Moving the search field and the refresh button to the top of the file navigator would help the user's experience in many ways. It would:

• Bring it to the direction of the user's eye (Then nobody has to "search" for the search)

• Make it where people actually know it exists (I've heard many times of people not even knowing it existed)

• Make the search more standard (you don't see Google's search on the bottom of the page)

(I may be able to provide some concept pictures tomorrow)

Thanks

[Mark-Agent003]

Also adding a "Search" placeholder text in the text box would help. Here's the concept image:

[Sawuare]

Great proposal!
I didn't know that there were a search field and a refresh button tbh. 😆

[michaelgregorius]

Move of the search bar to the top is implemented in pull request #3662 which is based on master.

[michaelgregorius]

Commit 96b136f adds the placeholder text "Search". This is how it currently looks:

Adding the default text was one line and much simpler than I would have thought. Qt for the win! 😄

[Mark-Agent003]

How does it look in the classic theme?

[michaelgregorius]

@Mark-Agent003 This is how it looks with the classic theme:

[RebeccaDeField]

@michaelgregorius Nicely done!

[BaraMGB]

@Mark-Agent003 why do you closed this issue?

[michaelgregorius]

Reopening this one so it can be closed once #3662 is integrated.

[Mark-Agent003]

I closed it because we already had a pull request open for this, and I didn't think we needed two separate discussions about the same thing. Do we actually need this open?

[grejppi]

Yes, because it will be closed once the pull request is merged

[Mark-Agent003]

Ok, that's fine

[michaelgregorius]

Shall I merge this one?

[tresf]

Shall I merge this one?

👍 from me.

[zonkmachine]

Tested. Works much better on top!

[michaelgregorius]

Closed with #3662.