nyxt icon indicating copy to clipboard operation
nyxt copied to clipboard

Broken frame-element-select

Open hendursaga opened this issue 2 years ago • 4 comments

After opening https://azlyrics.com/ with latest Nyxt, running toggle-debug-on-error and then select-frame-new-buffer, the frame source selection fails entirely, turning that entire buffer (as in, the prompt buffer is still visible) completely black and freezing everything.

<WARN> [17:22:10] Warning: JavaScript error: GError: Domain: "WebKitJavascriptError", Code: 699, Message: https://www.azlyrics.com/:45:49: ReferenceError: Can't find variable: selection

I assume it has something to do with https://github.com/atlas-engineer/nyxt/blob/master/source/mode/document.lisp#L576 ?? I'm no ParenScript wizard (yet). I do see that #1651 has new activity..

While you're at it, https://github.com/atlas-engineer/nyxt/blob/master/source/mode/document.lisp#L643 seems like duplicated code, as we already have that PS macro..

hendursaga avatar May 24 '22 21:05 hendursaga

An update: on 855f8ba374c513c8f795a0d66afe319c983a48f0, I can now run select-frame-new-buffer and it opens the cookie consent frame, but now I'm getting error messages with no real message to them, like

<WARN> [11:39:12] Warning: JavaScript error: GError: Domain: "WebKitJavascriptError", Code: 699, Message:

Not sure how to proceed to debugging this.

hendursaga avatar Aug 30 '22 15:08 hendursaga

@aartaka @aadcg What's the status on this?

Ambrevar avatar Sep 14 '22 07:09 Ambrevar

@hendursaga Was it working properly some time in the past?

Ambrevar avatar Sep 14 '22 07:09 Ambrevar

@aartaka @aadcg What's the status on this?

I guess it should still be broken, because we haven't even touched it while refactoring hint-mode.

aartaka avatar Sep 14 '22 07:09 aartaka

I have not used this feature before, but let me attempt to fix it.

aadcg avatar Sep 26 '22 11:09 aadcg

@hendursaga what would be the expected behavior with respect to the fact that the whole buffer gets black?

aadcg avatar Sep 26 '22 14:09 aadcg

@aadcg I actually haven't used this feature before I found out it was broken - I was looking for bugs to squash! :bug:

Now it doesn't show any frames to select, but the rest of the buffer does go black, and the error "message" is the same, although I've noticed I could C-g out of it and things return to normal.

Testing with something like https://iframe-html.com/ still doesn't work - it won't show any frames to select.

hendursaga avatar Sep 28 '22 18:09 hendursaga

This feature is intended to let you drag a square on screen and select all of those elements in the square. The elements appear the prompt buffer where you can further filter them.

jmercouris avatar Sep 28 '22 19:09 jmercouris

For more information; please see here: https://nyxt.atlas.engineer/article/element-frame.org

jmercouris avatar Sep 28 '22 19:09 jmercouris