search

MeesCode / mmjs

TUI mp3 player designed for excessively large music libraries
MIT License
1 stars 1 forks source link

Add seeking #2

Closed fwsmit closed 2 years ago

fwsmit commented 2 years ago

Ik wil seeking toevoegen, maar ik kon niet helemaal uitvogelen hoe ik dit toevoeg aan de interface. Ik zat te denken aan "<" en ">" gebruiken voor naar voren en terug spoelen, maar ik kon niet vinden hoe ik die kon gebruiken.

MeesCode commented 2 years ago

Hoi, Je kan de keybindings toevoegen aan de ui in de interface.go file. Een voorbeeld voor het handlen van character inputs kan je hier vinden: https://github.com/MeesCode/mmjs/blob/5496d3f8138b61b3611d252b930c4ba741192734/tui/interface.go#L482-L491

Zou hem in deze switch stoppen zodat hij globaal werkt: https://github.com/MeesCode/mmjs/blob/5496d3f8138b61b3611d252b930c4ba741192734/tui/interface.go#L392

Top dat je er aan mee werkt!

MeesCode commented 2 years ago

@fwsmit heb ook wat andere issues toegevoed mocht je interesse hebben

fwsmit commented 2 years ago

Tnx voor de pointer. Ik had het al gezien, maar het zag er vrij sketchy uit :) Nu werkt de feature voor mij. Het enige is dat er soms error op het scherm komt. iets met (niet helemaal leesbaar, want is overschreven met de progress bar):

───────1801] er4o12 dequantization failed!

MeesCode commented 2 years ago

Het issue van de progress bar die niet snel update als je naar een andere track gaat is wat mi betreft een apart issue, dus deze feature kan gewoon gemerged worden :+1: