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

rename ?autoFill to ?autofill #179

Closed
pixelzoom opened this issue Jul 21, 2020 · 8 comments
Closed

rename ?autoFill to ?autofill #179

pixelzoom opened this issue Jul 21, 2020 · 8 comments
Assignees

Comments

@pixelzoom
Copy link
Contributor

pixelzoom commented Jul 21, 2020

For phetsims/qa#515 & phetsims/qa#514

The autoFill query parameter is new for PhET-iO, see #121.

While working on #178, I discovered that the correct spelling is "autofill", see https://en.wikipedia.org/wiki/Autofill.

Should we change the spelling to "autofill" ?

@arouinfar
Copy link
Contributor

@pixelzoom yes, let's use autofill. Let me know when the change has been made, and I'll propagate it to the client docs.

@arouinfar arouinfar assigned pixelzoom and unassigned arouinfar Jul 22, 2020
pixelzoom added a commit to phetsims/ph-scale-basics that referenced this issue Jul 22, 2020
pixelzoom added a commit that referenced this issue Jul 22, 2020
pixelzoom added a commit that referenced this issue Jul 22, 2020
pixelzoom added a commit that referenced this issue Jul 22, 2020
pixelzoom added a commit to phetsims/ph-scale-basics that referenced this issue Jul 22, 2020
pixelzoom added a commit to phetsims/ph-scale-basics that referenced this issue Jul 22, 2020
@pixelzoom
Copy link
Contributor Author

Changed in master, patched into 1.4 branches for ph-scale and ph-scale basics.

In addition to the query parameter, I also renamed occurrences in code (e.g. autofillEnabledProperty) and occurrences in overrides.js (e.g. featuring phScale.microScreen.model.autofillEnabledProperty).

Assigning to @arouinfar to propagate to client docs. When that has been done, please unassign, and label this issue with "status:fixed-awaiting-deploy" so that it gets schedule for regression testing in the next RC.

@pixelzoom pixelzoom assigned arouinfar and unassigned pixelzoom Jul 22, 2020
@pixelzoom
Copy link
Contributor Author

@arouinfar if there are changes to client docs that need to be patched into the 1.4 branches for ph-scale and ph-scale basics, let me know and I'll handle that.

@arouinfar
Copy link
Contributor

Thanks, I've updated the client guides to use autofill. I'll go ahead and close this since #167 is the guide-specific issue.

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Jul 29, 2020

This needs to remain open until verified by QA in the next RC. Labeled fixed-awaiting-deploy.

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Aug 4, 2020

To verify in ph-scale and ph-scale-basics for brand=phet:

  1. Run the sim with autofill=false and verify that autofill is off.
  2. Run the sim with autofill=true and verify that autofill is on.

To verify in ph-scale and ph-scale-basics for brand=phet-io:

  1. Run the sim in Studio
  2. Search for "autofill"
  3. Verify that all hits are "autofill" or "Autofill"

@brooklynlash
Copy link

For brand=phet on Chromebook, the autofill spellings are correct, but on default with no extra query parameter, ph-scale basics does not autofill and ph-scale does autofill. Is this a problem or intended?

@pixelzoom
Copy link
Contributor Author

@brooklynlash Thanks for reporting. That's actually the desired behavior; ph-scale autofills, ph-scale-basics does not. From ph-scale-basics-main.js:

// If autofill query parameter was not in the URL, change the default.
if ( !QueryStringMachine.containsKey( 'autofill' ) ) {
  PHScaleQueryParameters.autofill = false;
}

Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants