?
avatar brianteeman
brianteeman
10 Mar 2017

Steps to reproduce the issue

Make sure the plugin is set to have Save Warning ON

screenshotr14-55-33

Expected result

Warning when you close a content item without saving

Actual result

No warning

System information (as much as possible)

3.7 staging

Additional comments

If it doesnt do anything then we should remove the option
If it should do something then there is a bug

avatar brianteeman brianteeman - open - 10 Mar 2017
avatar joomla-cms-bot joomla-cms-bot - change - 10 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 10 Mar 2017
avatar dgt41
dgt41 - comment - 10 Mar 2017

screen shot 2017-03-10 at 17 00 12

Works here! You need to have some data altered in the editor area. Works with browser's back, close and reload

avatar brianteeman
brianteeman - comment - 10 Mar 2017

And what happens when you change the setting to off? I still got that message so I assumed it was not coming from tinyMCE

avatar dgt41
dgt41 - comment - 10 Mar 2017

It works correctly here, the browser doesn't throw an alert and you loose all the data :)

avatar dgt41
dgt41 - comment - 10 Mar 2017

From a UX perspective even having that option is wrong! It should be ALWAYS ON

avatar brianteeman
brianteeman - comment - 10 Mar 2017

I still get the warning ;(

On 10 March 2017 at 15:07, Dimitri Grammatikogianni <
notifications@github.com> wrote:

From a UX perspective even having that option is wrong!


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#14466 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8ay4GWBUNqHjVoVPFZz1oGAxb77hks5rkWdDgaJpZM4MZem2
.

--
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar dgt41
dgt41 - comment - 10 Mar 2017

Which browser are you using?

avatar brianteeman
brianteeman - comment - 10 Mar 2017

tested on both edge and chrome on windows 10

avatar dgt41
dgt41 - comment - 10 Mar 2017

I must be drunk or something, I get that here as well, when disabled the alert keeps appearing

avatar shivamdiehard
shivamdiehard - comment - 10 Mar 2017

After disabling the save warning option to off i also get the alert when i tried to move away without saving the content in the editor.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14466.

avatar C-Lodder
C-Lodder - comment - 10 Mar 2017

I really dont see why this needs to be a parameter.

@dgt41 - is there a way to only display that message if any changes have been made on the page?

avatar dgt41
dgt41 - comment - 10 Mar 2017

That's the default, if you create a new article (or open an existing one) and close the tab without typing anything in the editor you get no warning...

avatar C-Lodder
C-Lodder - comment - 10 Mar 2017

So why is this a parameter then?

avatar brianteeman
brianteeman - comment - 10 Mar 2017

Especially when it doesn't work!

On 10 Mar 2017 4:43 p.m., "Lodder" notifications@github.com wrote:

So why is this a parameter then?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#14466 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8c3RIthZY7YnlIzrPg986kXVDcJvks5rkX1-gaJpZM4MZem2
.

avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Mar 2017
Category com_plugins
avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Apr 2017
Status New Discussion
avatar tonypartridge
tonypartridge - comment - 29 May 2017

Can we agree to remove the parameter since it doesn't work anyway!

avatar brianteeman
brianteeman - comment - 29 May 2017

@dgt41 says it works ??

avatar tonypartridge
tonypartridge - comment - 29 May 2017

@dgt41 said:

I must be drunk or something, I get that here as well, when disabled the alert keeps appearing

avatar brianteeman brianteeman - change - 14 Jun 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-06-14 19:53:51
Closed_By brianteeman
avatar brianteeman brianteeman - close - 14 Jun 2017

Add a Comment

Login with GitHub to post a comment