New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Firefoxでもスクショが取れるようにする #38

Merged

windymelt merged 2 commits into main from firefox-screenshot

Apr 21, 2023

Owner

windymelt commented Apr 20, 2023 •

edited

Loading

solves: #

前提 / Prerequisites

Chromeには妙なバグがあり、画面の解像度が800x600くらいになってしまうことがある(バージョンによる)

なぜこのPRが必要になったか / Why do we need this PR

一時的な回避策としてFirefoxを使えるようにしたい

なにをやったか / What I did

Firefoxにもスクショを撮影するオプションがあるので、既存のインターフェイスを一部修正しつつ実装した。
Firefoxは出力先ファイル名を指定できない(指定できるという情報があるが、手元で動作しなかった)ので、Mutexを使って同時に走れなくした
Mutexを導入したのでIO型になった
-s firefoxオプションをつけるとFirefoxで稼動するようになる

補足 / Supplementary information

Firefoxは透過背景でのスクショをサポートしていないので、動画との組み合わせが今後あるなら問題になるかも

windymelt added 2 commits

April 20, 2023 21:09


          Firefoxでもスクショが取れるようにする

cd82112


          CLIからスクリーンショットバックエンドを設定できるようにした

a9d4c85

windymelt force-pushed the firefox-screenshot branch from 95ae574 to a9d4c85 Compare

April 21, 2023 07:03

windymelt merged commit 1695f31 into main

windymelt deleted the firefox-screenshot branch

April 21, 2023 07:05

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet