lorca
lorca copied to clipboard
zserge
remove `--enable-automation` flag to disable browser is controlled message
Simply removing this flag will cause the "save password" bubble to pop up again unfortunately.
@cgostuff If a user needs passwords, it can be added later. This seems to be the more reasonable default. There are cases where the message "The browser is controlled" is a deal breaker.
@christopher-kleine I believe one of the main uses of webviews like this library is a login flow for desktop applications. Consequently I think that the "save password" bubble popping up every time you log in with Lorca is equally a deal breaker. Both should be possible to disable at the same time.
@cgostuff This is the more reasonable default. The ui.New function is written to allow users to add in additional custom arguments but does not allow the user to restrict defined default arguments. With this change users that need to not have the banner will not have the banner and users that want to not have the "save password" bubble can still supply the parameter to not have the "save password" bubble.
@georgemcarlson I agree, having this configurable is better. I will open a new issue about the "save password" bubble, surely there must be some programmatic way of disabling it other than command line switches.
I made an alternate pull request here that should accommodate everyone. It leaves the current lorca UI constructor implementation intact and creates two new lorca UI constructors (lorca.NewChromium and lorca.NewEdge) which do not automatically include the default arguments. This will let users have more control in the future over what arguments are passed to the lorca UI constructor. https://github.com/zserge/lorca/pull/175