Confusing text on unsaved content warning #1223

Closed
btubbs opened this Issue May 16, 2012 · 3 comments

Comments

Projects
None yet
3 participants
@btubbs

btubbs commented May 16, 2012

When you try to save a page without first saving the plugins, you get a Javascript alert message with this text:

"Not all plugins are saved. Are you sure you want to save the page?
All unsaved plugin content will tried to save."

This message has broken grammar, which confuses users into thinking that clicking OK is safe, rather than dangerous. Better text would be this:

"Not all plugins are saved. Are you sure you want to save the page? You may lose your changes. To prevent that, cancel this message and save the individual plugin(s) first."

yakky added a commit to yakky/django-cms that referenced this issue Jun 9, 2012

@beniwohli

This comment has been minimized.

Show comment Hide comment
@beniwohli

beniwohli Jul 23, 2012

Contributor

@ojii can you look at this? Not sure how this interacts with transifex

Contributor

beniwohli commented Jul 23, 2012

@ojii can you look at this? Not sure how this interacts with transifex

@ojii

This comment has been minimized.

Show comment Hide comment
@ojii

ojii Jul 23, 2012

Collaborator

with English we have to pull manually via git and then push to transifex. for some reason TX doesn't allow editing the "source" language.

Collaborator

ojii commented Jul 23, 2012

with English we have to pull manually via git and then push to transifex. for some reason TX doesn't allow editing the "source" language.

beniwohli added a commit to beniwohli/django-cms that referenced this issue Aug 3, 2012

@beniwohli

This comment has been minimized.

Show comment Hide comment
@beniwohli

beniwohli Aug 3, 2012

Contributor

I cherry-picked @yakky's translation. We have to do a transifex run sometime before 2.3.1 anyway

Contributor

beniwohli commented Aug 3, 2012

I cherry-picked @yakky's translation. We have to do a transifex run sometime before 2.3.1 anyway

@beniwohli beniwohli closed this Aug 3, 2012

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