added link to artist page

[nickm980]

Clicking on the artist name in the song player below the title of the song will navigate to the artist page. I believe this is consistent with most music player apps.

Ex) Spotify and the youtube music app have a feature where clicking on the artist name below the title of the page changes the page to the artist page.

The current way to access the artist page from ViMusic is to press the ellipses on the bottom right corner then "More of {artist}" button which requires an additional step. Instead of two steps, it should only take one. Adding the link to the artist name is a more intuitive approach and requires less friction, improving user experience because visiting the artist page is a common action.

[th3y]

You are taking the id from Database, if user has no items in database (First time), the app will crash.

You can use extras in player metadata to get the information:

var artistsInfo by remember { mutableStateOf(  binder.player.currentMediaItem?.mediaMetadata?.extras?.getStringArrayList("artistIds") ) }

Problem for me: it crash when there's more than one artist.

Update:
It was not because there's more than one artist. The artistIds was returning nothing in some songs (Displaying more than one artist), to fix this

 if (!artistsInfo.isNullOrEmpty()) {
    val goTo = artistRoute::global
    goTo(artistsInfo?.get(0))
} else {
    val goTo = albumRoute::global
    val albumId = binder.player.currentMediaItem?.mediaMetadata?.extras?.getString("albumId")
    if (!albumId.isNullOrBlank())
        goTo(albumId)
}

// ADD import it.vfsfitvnm.vimusic.ui.screens.albumRoute

As a workaround. Works than expected

[siggi1984]

hey bro vfsfitvnm, please allow your followers to further develop the app. to improve the user experience.

[owencz1998]

What about developing on harmony seems more active or innertune.
https://github.com/anandnet/Harmony-Music

https://github.com/z-huang/InnerTune

[vfsfitvnm]

This PR is not complete as it doesn't take into account that one track may have more artists, or there might be differences between "artists text" (the one showed on the screen, generally richer) and the "artist metadata" (the one showed in the menus).

I already attempted implementing this feature a year ago, and I eventually gave up as the hassle of handling edge cases wasn't worth a single tap save.

PS: Anyone is free to fork the project and continue the development there, so that any feature/code quality policy can be used 🙃

[Rayforest]

Is there any good fork of vimusic that is updated recently?

[siggi1984]

Is there any good fork of vimusic that is updated recently?

Take a look at the fork.

https://github.com/fast4x/RiMusic

[Rayforest]

Take a look at the fork.

https://github.com/fast4x/RiMusic

Thank you 🥰

[ServerMonk63]

Done

[sharunkumar]

Is there any good fork of vimusic that is updated recently?

Take a look at the fork.

https://github.com/fast4x/RiMusic

Someone needs to update this fork link in the README

[Harsha-070]

In my opinion the application was great and I was thinking of an Idea of making a widget for this application..