Skip to content

Click on video thumbnail to open video#964

Open
antonpetrov145 wants to merge 1 commit intoglanceapp:devfrom
antonpetrov145:video-thumbnail-click
Open

Click on video thumbnail to open video#964
antonpetrov145 wants to merge 1 commit intoglanceapp:devfrom
antonpetrov145:video-thumbnail-click

Conversation

@antonpetrov145
Copy link
Copy Markdown

@antonpetrov145 antonpetrov145 commented Feb 20, 2026

Added

cursor: pointer

to the css for the widget

Added onclick event for thumbnail

refers to #963

2026-02-20_18-08

@antonpetrov145 antonpetrov145 changed the base branch from main to dev February 20, 2026 16:14
@michaelwschultz
Copy link
Copy Markdown

Yes! This drives me nuts. haha thanks for adding. 🙏 Going to pull this into my fork.

@luissantos
Copy link
Copy Markdown

Why didn't you add an a tag around the image or around everything?

@antonpetrov145
Copy link
Copy Markdown
Author

antonpetrov145 commented Mar 19, 2026

@luissantos Mainly because I didn't want to introduce new tags in the html, but wrapping the image with a tag was my first intention

@antonpetrov145
Copy link
Copy Markdown
Author

@luissantos thank you, because I am not sure if in future there will not be some global style for a tags or some other styes or tags that will interfere with this, so IMO better to have the onclick action for now.

@luissantos
Copy link
Copy Markdown

@luissantos thank you, because I am not sure if in future there will not be some global style for a tags or some other styes or tags that will interfere with this, so IMO better to have the onclick action for now.

Thanks. For the explanation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants