Skip to content
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.

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

Add option to run headless #96

Open
RichiCoder1 opened this issue Nov 12, 2018 · 2 comments
Open

Add option to run headless #96

RichiCoder1 opened this issue Nov 12, 2018 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@RichiCoder1
Copy link

Both firefox and chrome (with some noise from Edge) support headless, and the selenium images referenced in the readme allow disabling running the display server if the inner browsers support headless. Would be great to either A) have a headless option or B) have the abilities to append capabilities.

@L0wry
Copy link
Contributor

L0wry commented Nov 13, 2018

Hey @RichiCoder1, personally we're not a fan of headless browsers as we have had bad experiences in the past where they have introduced flake or allowed bugs through.
If you think about headless browsers as a concept they are a step further away from how your end user interacts with the page and as such introduces some level of risk.

However, adding support for headless would be super easy, I'll get something on the backlog.

@L0wry L0wry added enhancement New feature or request good first issue Good for newcomers labels Nov 13, 2018
@RichiCoder1
Copy link
Author

I may look into adding this then. I agree with the risks of headless for automation testing, but since this is purely rendering and (for most cases) doesn't include interaction, I think this is something headless could be good for.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants