Skip to content

Rename "Insert" to "Add"#4393

Merged
karmatosed merged 1 commit intoWordPress:masterfrom
rileybrook:update/change-insert-to-add
Jan 11, 2018
Merged

Rename "Insert" to "Add"#4393
karmatosed merged 1 commit intoWordPress:masterfrom
rileybrook:update/change-insert-to-add

Conversation

@rileybrook
Copy link
Copy Markdown

@rileybrook rileybrook commented Jan 11, 2018

Description

Intent of PR is a copy change. To rename "Insert" to "Add" as discussed and agreed upon in #2975

  • This change only affects copy seen by users.
  • No changes have been made to functions, class names, the component named "Inserter" nor any developer documentation. Those changes would warrant separate, future issues as pointed out by @jasmussen in Rename "Insert" to "Add" #2975

Screenshot Examples of Changes

Before
screen shot 2018-01-10 at 10 34 37 am

After
screen shot 2018-01-10 at 6 10 53 pm

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows has proper inline documentation.

Copy change of"Insert" to "Add"
@jasmussen
Copy link
Copy Markdown
Contributor

Wohoo! Thanks so much for doing this! This is all looking good to me. Only point in question is whether we need to change the "Insert" term to "Add" for the table block — I'm fine with it, but that would be the only one where it sort of feels more sensible to use the old term. @karmatosed any thoughts?

Otherwise, 👍 👍 from me, and thanks again.

By the way, did the fork/branch instructions work as intended? Could you describe the steps you took so we can update the contributor instructions?

@karmatosed
Copy link
Copy Markdown
Member

👍 from me! Thank you for doing this @rileybrook - excited to get it in!

@karmatosed karmatosed merged commit f40cfc8 into WordPress:master Jan 11, 2018
@rileybrook
Copy link
Copy Markdown
Author

Thanks to both of you @karmatosed and @jasmussen. The steps taken below worked as intended.

Workflow

  • Fork Gutenberg repository to personal GitHub account
  • Clone forked repo
  • Create new branch
  • Make code changes
  • Commit code changes within newly created branch
  • Publish branch
  • Submit Pull Request

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants