option to disable the javascript messages about password-manager HOT 12 CLOSED

I looked into it a while ago, the problem is, that after the alert the whole page is reloaded, so it's not that easy to implement.
It would be better practice to just reload the table and tags after editing/adding/deleting an account - then showing a notification instead of an alert would be trivial.

from password-manager.

We can use Ajax. That is, put all dynamic contents in a seperate php file.
But yes, it's not a trivial work.

On Thursday, 16 June 2016, Benjamin Häublein [email protected]
wrote:

I looked into it a while ago, the problem is, that after the alert the
whole page is reloaded, so it's not that easy to implement.
It would be better practice to just reload the table and tags after
editing/adding/deleting an account - then showing a notification instead of
an alert would be trivial.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#55 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AEbvNAuHbgmJ733eUbyY7Hrebkdpj4Tuks5qMUIqgaJpZM4I2Zb3
.

from password-manager.

Well For me the messages are not needed anyway. Can't you just disable them?

from password-manager.

Temp solution for you: open password.php using notepad, search for 'alert'
and delete all of them.

On Thursday, 16 June 2016, Nick [email protected] wrote:

Well For me the messages are not needed anyway. Can't you just disable
them?

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#55 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AEbvNI3uS6nV9WhsqCMRc-2YXX03-Uiiks5qMfgLgaJpZM4I2Zb3
.

from password-manager.

Yeah I already did this for now.
Just thought this is a general improvement that I should report.

from password-manager.

I'll give it a try.

from password-manager.

Thanks Benjamin 👍

from password-manager.

@nickbe I think I've got it, but as this is a major change I'd be glad if you tested it before I'll merge it to the master branch. You just need the files src/password_ajax.php and src/password.php from my development branch

from password-manager.

I'll test this later today and report the results to you.

from password-manager.

There seems to be a problem with refreshing the view after deleteing an entry.
Entry is still visible and can be even openend, but after F5 it's gone.

Otherwise the messages look really fine this way.

from password-manager.

@nickbe thanks. It should work now. I just pulled everything to master.

from password-manager.

Will update the live system tonight.
Thanks!!!

from password-manager.