-
Notifications
You must be signed in to change notification settings - Fork 971
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hotkeys-Issues solved #297
Comments
Hi @hersche When I click on your link I got this... But any way, can you please make a pull request? will be great having you as a contributor :) |
don't know whats wrong, typo or so, i'm sorry. here's the link again, i've tested it in the preview.. edit: original link is weird longer, have to be typos |
Great, you use the video.js plugin. that is nice. The only thing I would make different I would make it as an youphptube plugin. some times people dont want it, and as a plugin they can disable or enable thu feature. |
mh, don't see through your plugin-structure yet and i think it's may better useable with settings? |
Hello cool !
Where get this plugins please ? |
it's not a plugin, it's hardcoded right now. apply these changes on your youphptube-code and there you go: hersche@29e823f or you mean this? https://github.com/DanielnetoDotCom/YouPHPTube/tree/master/plugin/YouPHPFlix |
Please make this as a plugin visible coded so i can learn it too. This script better as plugin because it will not be overwritten when new version updated as well. |
@gamersalpha go to your plugin page , if your YouPHOTube is up dated you should have a plugin called YouPHPFlix @abamilod if @hersche can not build it as a plugin as soon as I get some free time I will build it and release it free and open :) |
@DanielnetoDotCom you are the best man! |
@DanielnetoDotCom i think i get/understand the main-plugin-structure, like getName, getHeaderCode etc.. what i don't get: the youphpflix-plugin use the "getFirstPage()"-method - this seems to place the plugin into the startpage, /cat/-site, etc, but not on the page where the video itself plays. that's well for this plugin(s). in my case, i need the opposite, my plugin should only be active, when the user is on a concrete video, so there where the movie plays, what's for shure not the "firstPage". i found no method like "getVideoPage()" where i could return my scriptcontent. maybe this would also work with the getHeaderCode-method, but this would be unproper, because of the first page - also, i'm not shure if it's not included too early then. so what's the right method for this case? my reference for plugin-structure was https://github.com/DanielnetoDotCom/YouPHPTube/blob/master/plugin/Plugin.abstract.php plugin/YouPHPTube.php is a bit too complex to understand for me atm, but shure there would be a solution-szenario.. |
well, i got it. the if-condition is may not very great, but it works. also, i don't know how to set und use options/parameters, personaly i would like to disable up and down for volume and use it default. to set such stuff would be the best. close, if you want, extend how you like.. best regards and have fun edit: i know now also how to use options, there's still stuff to do, but the plugin should work better with last commit.. hersche@0e955b1 |
@hersche |
@hersche |
@hersche Actually you can use tab key to navigate to player control (tab 3x) until it reaches volume bar and use arrow key for volume but using + and - would be better tho. |
you should be able to change volume with up/down-arrows only (no tab/control-showup needed). but + - would be also nice - up/down is just default and i have to look for the right key-codes and similiar stuff.. |
This is nice. I think using arrow up/down for volume better than -/+. thumbs up (I was looking for volume hotkey actually) :-) |
i don't think so. when i personaly press up/down-arrows, i except, the website will scrolling. instead i mute the movie. youtube, which we "want to clone", also don't handle it this way. if you really like it the way it is, i could make it reverseable via plugin-options - but i have to do it at all first ;) |
This issue was closed due to inactivity. Feel free to reopen if you have more feedback or are interested it working on it |
oh, it's basicly implemented, merged and working, so it's absolutly closeable.. |
Hello,
don't know how i should handle, i could make a pull-request but don't know how to do this for a single commit. Feel free to handle it as you like.
My commit hersche@29e823f5c8d408cb5c86fsdf8f45306a4bb9cb1eb47 solves the point of space for pause/resume, f for fullscreen, etc. May need finetuning!
That should solve #59 and may others?
Have fun and best regards. Also many thx for the person wrote the youphpflix-plugin, it's genoius!
The text was updated successfully, but these errors were encountered: