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

What casing should we used for checkboxes and radio buttons? #953

Closed
samreid opened this issue Sep 6, 2018 · 17 comments
Closed

What casing should we used for checkboxes and radio buttons? #953

samreid opened this issue Sep 6, 2018 · 17 comments

Comments

@samreid
Copy link
Member

samreid commented Sep 6, 2018

Skimming through several simulations, I saw several cases where checkboxes or radio buttons were title cased, like this:

image

image

image

In phetsims/faradays-law#124 we corrected the casing for "Field lines", and @arouinfar pointed out that Trig Tour has casing like this:

image

Should this trig tour case be changed? Would you like to assign someone (design person, QA person, intern) to look through strings in all sims to see if other occurrences should be changed?

@ariel-phet
Copy link

@samreid consistency is nice and all, but I don't think it is worth the effort to skim through all sims for changes, the trig tour one (since noticed) can be changed, I will create an issue.

@amanda-phet
Copy link

amanda-phet commented Sep 6, 2018 via email

@samreid samreid reopened this Sep 6, 2018
@samreid
Copy link
Member Author

samreid commented Sep 6, 2018

@amanda-phet pointed out one way we might proceed. Another possibility would be to note in a pre-publication checklist so we don't publish new sims with this inconsistency. @ariel-phet up to you if you want to re-close or follow an alternate strategy.

@ariel-phet
Copy link

@KatieWoe please look through all current HTML5 sims (or delegate ) and look for this inconsistency (we will want title case). For any sim that needs to be corrected, please make an issue in the sim repo and then close this issue.

@ariel-phet ariel-phet assigned KatieWoe and unassigned ariel-phet Sep 6, 2018
@amanda-phet
Copy link

https://phetsims.github.io/binder/

The "sim examples" section seems broken. @mbarlow12 do you know anything about this?

@arouinfar
Copy link

@amanda-phet the Sim Examples sections are loading for me, though RadioButtonGroup does not appear to include VerticalAquaRadioButton (yet), so we're missing the string-based radio buttons.

@pixelzoom
Copy link
Contributor

pixelzoom commented Sep 7, 2018

For the record, I disagree with this "convention" of requiring labels to use title case.

@samreid
Copy link
Member Author

samreid commented Sep 7, 2018

I agree, there are cases where lowercase is warranted.

@KatieWoe
Copy link

KatieWoe commented Sep 7, 2018

Should I ask a designer to confirm before making an issue, or continue making issues for instances?

@samreid
Copy link
Member Author

samreid commented Sep 7, 2018

Perhaps use your judgment, and if you see something that looks odd or inconsistent, create an issue for the designer? For instance in Area Model looks like the casing was done purposefully and consistently, so doesn't seem problematic to me:

image

But, for instance, if only one of these words was lowercase, that would be problematic:
image

@arouinfar
Copy link

@KatieWoe please continue to make issues, but assign them to the lead designer for final review.

@pixelzoom stated in phetsims/function-builder#122

And I know of no convention that requires every control label to start with an uppercase letter.

This does not extend to every control. I, too, would disagree with that. The accordion boxes and panel titles in @samreid's #953 (comment) would not fall under this issue.

However, as we work on creating a Style Guide, we are going to conventionalize some things. Checkboxes and Radio Buttons in panels should have consistent casing. The majority of sims have used title casing, so we decided to go with that. There is no good reason why Trig Tour should use "Special angles" and not "Special Angles", for example.

However, I think there can be some exceptions, particularly in cases like "simplify" in Function Builder. That checkbox is on its own and was likely lowercase to reduce visual weight.

@pixelzoom
Copy link
Contributor

pixelzoom commented Sep 7, 2018

The more you use uppercase, the less effective it becomes at adding "weight". Kind of like the overuse of bold text that we were guilty of in Java sims. That said, I think the convention used in the Area Model sims is vastly superior.

@KatieWoe
Copy link

@arouinfar I have checked all the checkboxes in the the style guide example. However, there is no vertical aqua radio button group in the style guide.
@ariel-phet should I check the sims on master for this aspect, or should the issue be held until vertical aqua radio buttons are added to the style guide?

@ariel-phet
Copy link

@KatieWoe I think the issue should be held until they are added to the style guide

@ariel-phet ariel-phet removed their assignment Sep 11, 2018
@arouinfar
Copy link

Thanks @KatieWoe! I agree with @ariel-phet. Let's wait until verticalAquaRadioButtonGroup is in the style guide.

@arouinfar arouinfar removed their assignment Sep 11, 2018
@pixelzoom
Copy link
Contributor

@ariel-phet Now that a whole bunch of issues have been created, what is the timeframe for deciding whether they will be (or even need to be) addressed?

@ariel-phet
Copy link

@pixelzoom I think we can just address these issues when we republish sims. There is no need to prioritize this work otherwise. Closing this issue now that specific issues have been created.

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

6 participants