search

migtools / mig-ui

Openshift Migration UI
Apache License 2.0
10 stars 31 forks source link

capitalization #463

Closed apinnick closed 5 years ago

apinnick commented 5 years ago

"Add cluster" - Change to "Add Cluster", to be consistent with "Add Repository" and "Add Plan"

"Add Repository" window: "Url" not all caps. Is this intentional?

vconzola commented 5 years ago

@apinnick New PatternFly guidance is that all text elements should be sentence-style (except product names and proper nouns), so it's actually "Add Repository" and "Add Plan" that should be changed to "Add repository" and "Add plan".

apinnick commented 5 years ago

Oh my. I'll update the documentation. Thanks for the update. That will solve a lot of problems (like whether to capitalize prepositions) and it will make the UI more consistent.

"URL" is an acronym, so it's exempt from the sentence case directive because acronyms have their own capitalization convention. If PatternFly hasn't specified an exception for acronyms, someone should submit a request or query.

apinnick commented 5 years ago

In Add migration plan > Persistent Volumes, the dropdown list for the persistent volume options ("copy"/"move") is lower case. Should be capitalized.