disable quickstyle in artiq_browser

[SimonRenblad]

ARTIQ Pull Request

Description of Changes

Disables quickstyle widgets in artiq_browser. quickstyle widgets will display as if option was not enabled.

Steps

All Pull Requests

• [x] Use correct spelling and grammar.
• [x] Check the copyright situation of your changes and sign off your patches (git commit --signoff, see copyright).

Code Changes

• [x] Run flake8 to check code style (follow PEP-8 style). flake8 has issues with parsing Migen/gateware code, ignore as necessary.
• [x] Test your changes or have someone test them. Mention what was tested and how.
  • [x] Test in browser

Documentation Changes

• [x] Check, test, and update the documentation in doc/. Build documentation (cd doc/manual/; make html) to ensure no errors.

Git Logistics

• [x] Split your contribution into logically separate changes (git rebase --interactive). Merge/squash/fixup commits that just fix or amend previous commits. Remove unintended changes & cleanup. See tutorial.

• [x] Write short & meaningful commit messages. Review each commit for messages (git show). Format:

  topic: description. < 50 characters total.
  
  Longer description. < 70 characters per line

Licensing

See copyright & licensing for more info. ARTIQ files that do not contain a license header are copyrighted by M-Labs Limited and are licensed under LGPLv3+.

[SimonRenblad]

Updated alternative solution:

Here is how it compares to previous proposals.

Pros:

• No copying and no mutating of argument list whatsoever
• No change of procdesc_entry required
• Logic mostly contained to the entry widget

Cons:

• EnumerationEntry now has additional set_quickstyle method, which means widget display is no longer one-to-one with its argument. This is somewhat unavoidable due to the nature of altering widgets at a "dashboard/browser level" in a way that contradicts how they were set at an "experiment level". Since "quickstyle" does not alter the resulting value in experiments in any way, this may still be acceptable.
• Still a fairly big change in terms of SLOC

[SimonRenblad]

Update:

Small change, still a mutation but now in initialize_submission_arguments.