Skip to content

Link URL tooltips#8

Merged
BigJk merged 2 commits into
BigJk:mainfrom
PierceLBrooks:plb_href-tooltips
Jun 10, 2026
Merged

Link URL tooltips#8
BigJk merged 2 commits into
BigJk:mainfrom
PierceLBrooks:plb_href-tooltips

Conversation

@PierceLBrooks

Copy link
Copy Markdown
Contributor

This pull request will push a textual tooltip onto your cursor whenever you hover over a link within the HTML document that shows the actual URL that it links to.

@BigJk

BigJk commented May 29, 2026

Copy link
Copy Markdown
Owner

Thanks for this PR!

Some thoughts:

  • Maybe it would make more sense to just name attr to tooltip, because that's all the variable does
  • I personally would like if there was a switch in the config to globally enable or disable href tooltips
  • With your mouse entered approach we could also allow for any element that has el->get_attr("tooltip"); attribute to show a imgui tooltip. So the user is not restricted to hrefs and in case of hrefs can even overwrite what is shown as tooltip.

What do you think?

@PierceLBrooks

PierceLBrooks commented May 31, 2026

Copy link
Copy Markdown
Contributor Author

Hi @BigJk ,

These are solid suggestions, so I went ahead and included appropriate accommodations for them in my most recent commit ( https://github.com/PierceLBrooks/ImHTML/commits/a9948a8ab640cf2bc75f4def44181d40f73ce8fc ). Additionally, there is now support for image alt-text attributes that is also toggleable by the corresponding new configuration member flag assigned onto each browser container instance, and I've verified bold-face font family behaviors during the course of verifying tooltip override functionality.

Thank you very much! Take care.

  • Pierce

@BigJk BigJk merged commit 1cf3330 into BigJk:main Jun 10, 2026
1 check failed
@BigJk

BigJk commented Jun 10, 2026

Copy link
Copy Markdown
Owner

Thanks a lot!

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.

2 participants