zaproxy / browser-extension

Apache License 2.0
24 stars 7 forks source link

added floating window to indicate and stop recording #63

Closed aryangupta701 closed 1 year ago

aryangupta701 commented 1 year ago

Screenshot from 2023-08-19 15-24-03

Screenshot from 2023-08-19 15-23-55

aryangupta701 commented 1 year ago

I will create a separate PR for reporting the recorded actions over this floating window

thc202 commented 1 year ago

This should be rebased and use/extract constants. (It has conflicts too.)

thc202 commented 1 year ago

It seems the events done to the floating window might also end up being recorded to the scripts, e.g.:

    {
      "windowHandle": "windowHandle1",
      "type": "id",
      "element": "ZapfloatingDiv",
      "index": 4,
      "enabled": true,
      "elementType": "ZestClientElementClick"
    }
aryangupta701 commented 1 year ago

ohhh Thank you. Forgot to test that.

aryangupta701 commented 1 year ago

ready for review

thc202 commented 1 year ago

Sometimes the floating window is almost hidden, e.g.: floating partially hidden

aryangupta701 commented 1 year ago

fixing this

aryangupta701 commented 1 year ago

if someone drag and drop to a particular location it will stay at that location even if the window size changes that may lead to the loss of the floating window. Thinking about how we can fix this

aryangupta701 commented 1 year ago

Fixed :)

thc202 commented 1 year ago

Looks better but it's still partially hidden: partial

(That's the default window size when launching the browser from ZAP.)

aryangupta701 commented 1 year ago

Fixing..

aryangupta701 commented 1 year ago

I think this is again a mac related issue. For me it works fine. I can't reproduce this behavior @thc202

thc202 commented 1 year ago

What information can I provide?

aryangupta701 commented 1 year ago

I can suggest some hit and trials. Can you test them please?

  1. try changing https://github.com/zaproxy/browser-extension/pull/63/files#diff-326f362eacd2a9bea9debec8ede73c13ba68fdcbbb2c69a9f40a90d5afc3d5b0R246 replace -105 with -100
aryangupta701 commented 1 year ago

I am able to reproduce this issue. I wonder why the chrome browser launched from ZAP and the the one which I directly use is rendering css differently.

aryangupta701 commented 1 year ago

Ohh I got it. This is because of the website only. It has some css which is being applied over this floating window as well. So I guess I will need to use frames.

aryangupta701 commented 1 year ago

ready for review

thc202 commented 1 year ago

Thank you!