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

A couple of features for Options #117

Merged
merged 4 commits into from Apr 2, 2019
Merged

A couple of features for Options #117

merged 4 commits into from Apr 2, 2019

Conversation

@asajeffrey
Copy link
Member

asajeffrey commented Mar 21, 2019

A couple of changes to option-handling:

  • allow demo apps to provide default options
  • add an option to disable the UI
@asajeffrey asajeffrey force-pushed the asajeffrey:pf3-options branch from f782ace to 9c8c247 Mar 21, 2019
asajeffrey added a commit to asajeffrey/pathfinder that referenced this pull request Mar 21, 2019
asajeffrey added a commit to asajeffrey/pathfinder that referenced this pull request Mar 22, 2019
asajeffrey added a commit to asajeffrey/pathfinder that referenced this pull request Mar 22, 2019
asajeffrey added a commit to asajeffrey/pathfinder that referenced this pull request Mar 25, 2019
@asajeffrey asajeffrey force-pushed the asajeffrey:pf3-options branch from 9c8c247 to 7a2b280 Mar 26, 2019
@asajeffrey
Copy link
Member Author

asajeffrey commented Mar 26, 2019

Rebased.

asajeffrey added a commit to asajeffrey/pathfinder that referenced this pull request Mar 26, 2019
asajeffrey added a commit to asajeffrey/pathfinder that referenced this pull request Mar 26, 2019
@asajeffrey
Copy link
Member Author

asajeffrey commented Mar 27, 2019

Review ping?

asajeffrey added a commit to asajeffrey/pathfinder that referenced this pull request Mar 28, 2019
asajeffrey added a commit to asajeffrey/pathfinder that referenced this pull request Mar 28, 2019
@pcwalton
Copy link
Collaborator

pcwalton commented Apr 2, 2019

I think it'd be good to allow making the UI invisible while the app is running, not just as a command line switch. Maybe allow the Tab key to cycle through "full UI", "only debug window", "no UI" options, and then add a command line switch to start in one of those modes.

@asajeffrey asajeffrey force-pushed the asajeffrey:pf3-options branch from 7a2b280 to 0166ff9 Apr 2, 2019
@asajeffrey
Copy link
Member Author

asajeffrey commented Apr 2, 2019

Done.

@pcwalton pcwalton merged commit 2f651ed into servo:pf3 Apr 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.