nyxt
nyxt copied to clipboard
Add `iframe` support for `hint-mode`
Hinting on the duckduckgo iframe at https://nyxt.atlas.engineer/start does not work.
@aartaka You merged some work on iframes recently, any chance it fixes this?
@aartaka You merged some work on iframes recently, any chance it fixes this?
No, the iframe work I've done was purely request-related. For this issue, the fix should be something close to the #1828 fix (which I started but haven't yet finished implementing).
Label as dom?
I don't think this is important to solve by 3.0.
Agreed.
It is inoortant! As important as #1828! We need our keyboard interfaces to work as good as mouse for Nyxt to be good enough of keyboard-driven browser. For that, we should make hints work in iframes and Shadow DOM, because more and more websites use the latter, while lots already rely of the former ヽ(o`皿′o)ノ
I agree it's important, but I can't commit to it when there are more pressing issues.
Would it be API-breaking to fix it after the 3.0 release?
I would say it is not.
Closing this as not planned after some analysis of IFrame behaviors.
I don't think iframe hinting is impossible for all cases as Saka key seems to be able to support it.
This is not a priority, but maybe we should keep this issue open for future reference. CC @jmercouris.
Interesting, I wonder how that works...
It would involve studying https://github.com/lusakasa/saka-key/blob/master/src/modes/hints/client/findHints.js#L24.