Skip to content
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

Change the size of Youtube video #8

Closed
mendax1234 opened this issue May 28, 2022 · 18 comments
Closed

Change the size of Youtube video #8

mendax1234 opened this issue May 28, 2022 · 18 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@mendax1234
Copy link

l use Logseq 0.7.0 now and l find that l can't change the size of Youtube video.
Here is the youtube video link:https://www.youtube.com/watch?v=6Af6b_wyiwI
Here is my code to use it in the Logseq{{video https://www.youtube.com/watch?v=6Af6b_wyiwI}}
l hope the plugin can be updated to suit the latest version of Logseq. Thanks!

Daniel

@vyleung
Copy link
Owner

vyleung commented May 28, 2022

hi Daniel, i’m actively working on supporting the new {{video }} macro 🙂

@vyleung vyleung added the enhancement New feature or request label May 29, 2022
vyleung added a commit that referenced this issue Jun 3, 2022
- feat: add keyboard shortcuts to 1) play/pause and 2) skip forward/backward timestamp for youtube and local videos
- feat: increasing the width of videos no longer shows the overflow scroll bar (addresses #5)
- feat: local videos can be resized (addresses #8)
- refactor: update `startFloat(e)` to support the new `{{video URL}}` macro in Logseq version 0.7.0+ (addresses #7)
vyleung added a commit that referenced this issue Jun 3, 2022
- feat: add keyboard shortcuts to 1) play/pause and 2) skip forward/backward timestamp for youtube and local videos
- feat: increasing the width of videos no longer shows the overflow scroll bar (addresses #5)
- feat: local videos can be resized (addresses #7)
- refactor: update `startFloat(e)` to support the new `{{video URL}}` macro in Logseq version 0.7.0+ (addresses #8)
@vyleung
Copy link
Owner

vyleung commented Jun 3, 2022

added in v.2.0.0! let me know if you run into any issues

@mendax1234
Copy link
Author

Thanks! It works for youtube video now, but it doesn't work for the local video, even the start float function is not compatiable with the local video. Can you fix this? Thanks!

@vyleung
Copy link
Owner

vyleung commented Jun 3, 2022

can you share your local video syntax? the one i currently have supported is ![](local video path)

@mendax1234
Copy link
Author

[:video {:controls true :src "path"}]

@vyleung
Copy link
Owner

vyleung commented Jun 3, 2022

gotcha, i haven't seen this syntax before - i'll work on supporting it

@mendax1234
Copy link
Author

Thanks, how can l use your syntax, since mine is from the logseq directly? If l wan to use your syntax by using "/..."how can l do that ? Can you have a look at it ,thanks!

@mendax1234
Copy link
Author

l have tried your syntax and the video still can't be shown.

vyleung added a commit that referenced this issue Jun 3, 2022
- fixed: support all syntax for embedding local videos (addresses #8)
@vyleung
Copy link
Owner

vyleung commented Jun 3, 2022

can you update to v.2.0.1 and try again using your original syntax?

@mendax1234
Copy link
Author

Sure!

@mendax1234
Copy link
Author

mendax1234 commented Jun 3, 2022

It seems that you haven't updated the 2.0.1 release. It is still 2.0.0 in both you Github release and in the marketplace. @vyleung

@vyleung
Copy link
Owner

vyleung commented Jun 3, 2022

oops - i pushed the commit, but forgot to create a release 😅 please check again

@mendax1234
Copy link
Author

Thanks , it works now , but it seems that there is a bug with this button.
image
After the first click on the local video, it seems that the status of the video is wrong .
For example , like below
image
Hope this can be fixed!

vyleung added a commit that referenced this issue Jun 3, 2022
- fixed: clicking on local videos show incorrect play/pause icon [created `playPauseControls_Listener()` instead of using `playPauseControls()`] (addresses #8)
@vyleung
Copy link
Owner

vyleung commented Jun 3, 2022

hopefully v.2.0.2 fixes this issue

@mendax1234
Copy link
Author

ok 😂 l am trying it now.

@mendax1234
Copy link
Author

Thanks , this bug is fixed now , but l think l have found a new one. If l play the video first and then start floating, it seems that the state of the button is not consistent again.
image
Hope this can be fixed!

@vyleung vyleung added the bug Something isn't working label Jun 3, 2022
vyleung added a commit that referenced this issue Jun 3, 2022
- fixed: playing a video then activating the plugin shows incorrect icon (addresses #8)
@vyleung
Copy link
Owner

vyleung commented Jun 3, 2022

try v.2.0.3 - thanks for finding all these bugs btw!

@mendax1234
Copy link
Author

mendax1234 commented Jun 3, 2022

It works perfectly now! Thanks for fixing all the bugs 😄. BTW, hope the translation function can be added soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants