Firefox options unnecessary prefix

[vaswix]

Hi, When adding arguments to specify the path to the firefox profile, you need to specify -profile and path/to/profile separated by commas, otherwise the profile does not open. I also tried -profile=path/to/profile and --profile=path/to/profile, but this does not open the profile, so the only option that works for firefox, I have this firefox_arg='-profile,path/to/profile', but since in the same way, when checking firefox_arg_item, prefix '--' is added to the beginning of the path, which is why firefox cannot find the folder, the result is the string '--path/to/profile' because of this, I suggest checking that the string is the path and not adding prefix '--'

[mdmintz]

That's not the optimal way to solve the problem you're trying to solve.

• Your code has flake8 linter issues. (Eg. Line length above accepted limit.)
• Your code is setting a variable to itself.
• You're not utilizing os.sep to simplify file-separator differences between different OS.
• You added : into the conditional where it shouldn't be included (Eg. https://stackoverflow.com/a/77901568/7058266)

The original problem has been resolved in 4.33.0 with this commit: 9636868

Also, I've been moving toward the principles outlined in https://github.com/readme/featured/how-open-is-open-source:
"Open-Source, not Open-Contribution" (with some special exceptions).