Bug 441564 - Action to stop playback and move to start of current track
Summary: Action to stop playback and move to start of current track
Status: CONFIRMED
Alias: None
Product: Elisa
Classification: Applications
Component: general (other bugs)
Version First Reported In: unspecified
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-26 10:05 UTC by empyreal
Modified: 2022-02-23 17:30 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description empyreal 2021-08-26 10:05:08 UTC
Consider adding this funcktion. Suggest Key_C shortcut.
Comment 1 empyreal 2021-08-26 15:57:48 UTC
Just added. Works as expectred, but I get this:
"Could not convert argument 0 at"
         "onTriggered@qrc:/qml/ElisaMainWindow.qml:82"
"Passing incompatible arguments to C++ functions from JavaScript is dangerous and deprecated."
"This will throw a JavaScript TypeError in future releases of Qt!"
Comment 2 Nate Graham 2021-08-26 18:51:14 UTC
To be clear, you want an action that does only this, rather than having the "previous track" be smart and go to the beginning of the current track unless you're within the first 5 seconds of the track?
Comment 3 empyreal 2021-08-26 19:13:07 UTC
Exactly. Press C and start current track.
And I made it. Now trying to figure out what to do with this error.

Actually I made all necessary key changes and enjoy usability =)

Z - prev, X - rwd, C - start current, V - fwd, B - next, Space - Play/Stop
Comment 4 Nate Graham 2021-08-26 19:14:55 UTC
Feel free to submit a merge request with your change!
Comment 5 empyreal 2021-08-26 19:34:28 UTC
I submitted already few requests =)
Comment 6 empyreal 2021-08-27 00:00:47 UTC
Just added. Works as expectred, but I get this:
"Could not convert argument 0 at"
         "onTriggered@qrc:/qml/ElisaMainWindow.qml:82"
"Passing incompatible arguments to C++ functions from JavaScript is dangerous and deprecated."
"This will throw a JavaScript TypeError in future releases of Qt!"

Solved creating special variable instead using qint64.

I see that I wasted time on github and you prefer only gilab.
Comment 7 Nate Graham 2021-08-27 16:32:13 UTC
Yes, GitHub has a read-only mirror of KDE source code repos, but all the development happens at https://invent.kde.org/