r/p5.js-web-editor
1
0
Fork 0
Code Issues 1 Pull requests Releases 1 Wiki Activity

Make "Add sketch" copy consistent

Browse source 
Buttons and menu items should say "Add sketch" (singular) and titles
should say "Add to collection" as it's implied by context that it's
a sketch.
This commit is contained in:
Andrew Nicolaou 2020-01-15 10:50:17 +01:00
parent 903550b41c
commit 0660031a0c
4 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
client/modules/IDE/components/CollectionList/CollectionList.jsx
View file

@ -142,7 +142,7 @@ class CollectionList extends React.Component {
{ {
this.state.addingSketchesToCollectionId && ( this.state.addingSketchesToCollectionId && (
<Overlay <Overlay
title="Add sketches" title="Add sketch"
actions={<SketchSearchbar />} actions={<SketchSearchbar />}
closeOverlay={this.hideAddSketches} closeOverlay={this.hideAddSketches}
isFixedHeight isFixedHeight

2
client/modules/IDE/components/CollectionList/CollectionListRow.jsx
View file

@ -135,7 +135,7 @@ class CollectionListRowBase extends React.Component {
onBlur={this.onBlurComponent} onBlur={this.onBlurComponent}
onFocus={this.onFocusComponent} onFocus={this.onFocusComponent}
> >
Add sketches Add sketch
</button> </button>
</li> </li>
{userIsOwner && {userIsOwner &&

2
client/modules/IDE/pages/IDEView.jsx
View file

@ -380,7 +380,7 @@ class IDEView extends React.Component {
{this.props.location.pathname.match(/add-to-collection$/) && {this.props.location.pathname.match(/add-to-collection$/) &&
<Overlay <Overlay
ariaLabel="add to collection" ariaLabel="add to collection"
title="Add sketch to collection" title="Add to collection"
previousPath={this.props.ide.previousPath} previousPath={this.props.ide.previousPath}
actions={<CollectionSearchbar />} actions={<CollectionSearchbar />}
isFixedHeight isFixedHeight

2
client/modules/User/components/Collection.jsx
View file

@ -325,7 +325,7 @@ class Collection extends React.Component {
{ {
this.state.isAddingSketches && ( this.state.isAddingSketches && (
<Overlay <Overlay
title="Add sketches" title="Add sketch"
actions={<SketchSearchbar />} actions={<SketchSearchbar />}
closeOverlay={this.hideAddSketches} closeOverlay={this.hideAddSketches}
isFixedHeight isFixedHeight