Expand documentation for window related functions

[mhanberg]

Also fixes #628

[codecov-commenter]

Codecov Report

Merging #629 (4b523ec) into main (8120d9a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #629   +/-   ##
=======================================
  Coverage   92.02%   92.02%           
=======================================
  Files          24       24           
  Lines         941      941           
=======================================
  Hits          866      866           
  Misses         75       75           

Flag	Coverage Δ
IntegrationTest	87.67% <ø> (ø)
UnitTest	41.55% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
lib/wallaby/browser.ex	94.50% <ø> (ø)
lib/wallaby/helpers/key_codes.ex	100.00% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8120d9a...4b523ec. Read the comment docs.

[mhanberg]

@rrrene really sorry to bother you, but do you have any idea why i'd be getting these Credo errors? https://github.com/elixir-wallaby/wallaby/pull/629/checks?check_run_id=3617265849

I am sort of at a loss 😅.

(Feel free to ignore this if you don't have free time. I wanted to avoid opening an issue as I can't imagine this is a bug in credo)

[rrrene]

@mhanberg It's a Credo bug. Thx for reporting this. Working on a minimal reproducible example.

[rrrene]

@mhanberg Thanks for reporting this 😀 It is now fixed on master.

Could you try this again by setting your Credo dep to

{:credo, github: "rrrene/credo"}

Please report back if your issue is solved! 👍

[mhanberg]

@rrrene I opened another PR to trial the credo updates #630.

Looks like there are still a few false positives.