search

DucNg / fip-player

FIP client with metadata
Do What The F*ck You Want To Public License
5 stars 3 forks source link

Duplicate entry in gui #7

Closed atouminet closed 1 year ago

atouminet commented 1 year ago

In some cases, a radio entry will duplicate on the gui.

Input sequence to produce the bug (in vim notation):

/contemp<CR><CR><Esc>j<CR>/rock<CR>

Which shows

Screenshot from 2022-12-19 15-19-17

DucNg commented 1 year ago

Thanks for reporting this!

I think this happens because ids in the list change when filtering. Selecting the item from the filtered view actually updates the wrong item because the id is not the good one (when doing m.list.SetItem(m.playingItemIndex, previousItem)). It causes the wrong item to get updated with infos from the item you've just selected. No idea on how to fix this for the moment. Needs some thinking 🧠

DucNg commented 1 year ago

@atouminet I attempted to fix the issue on this PR can you test?