Skip to content

Change "Update" button to "Preview"#80

Closed
arnoesterhuizen wants to merge 2 commits into
xwp:masterfrom
arnoesterhuizen:master
Closed

Change "Update" button to "Preview"#80
arnoesterhuizen wants to merge 2 commits into
xwp:masterfrom
arnoesterhuizen:master

Conversation

@arnoesterhuizen

Copy link
Copy Markdown
Contributor

...to help guide users to press the "Save & Publish" at the top of the
customizer.

If accepted, I can also change the help text to reflect the renaming of the button.

...to help guide users to press the "Save & Publish" at the top of the
customizer.
@westonruter

Copy link
Copy Markdown
Contributor

@arnoesterhuizen thanks for the PR. I think we actually had it say Preview before. However, a Preview button is actually misleading because it makes it seem like clicking it is optional. However, if you don't click the button then the setting will never be updated and their changes will not be saved. So I think something more forceful is needed.

@arnoesterhuizen

Copy link
Copy Markdown
Contributor Author

Maybe using "Apply"? I guess this would need more user testing.

Feel free to delete the pull request if this isn't viable right now.

@westonruter

Copy link
Copy Markdown
Contributor

@shaunandrews what do you think? Is "Apply" better than "Update"?

@shaunandrews

Copy link
Copy Markdown
Contributor

I like Apply.

@westonruter

Copy link
Copy Markdown
Contributor

@arnoesterhuizen you can go ahead and change it to using "Apply". How would you change the help text?

@arnoesterhuizen

Copy link
Copy Markdown
Contributor Author

Just changing the text in the readme files to refer to the "Apply" button. The screenshots I won't be able to help with necessarily, but I can try.

westonruter added a commit that referenced this pull request Feb 4, 2014
@westonruter

Copy link
Copy Markdown
Contributor

Thanks! Merged into develop.

@westonruter westonruter closed this Feb 4, 2014
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