Issue 7010

Title
Add settings on sao
Priority
feature
Status
testing
Superseder
Widget buttons should be ignored when tabbing (issue 6739)
Nosy list
acaubet, albertca, ced, pokoli, reviewbot, yangoon
Assigned to
pokoli
Keywords
review

Created on 2017-12-13.16:45:09 by pokoli, last changed 2 months ago by acaubet.

Messages

Author: [hidden] (acaubet)
Date: 2021-11-10.16:12:04

ping?

Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2017-12-18.13:15:37
> I think the session key could be stored in the sessionStorage to survive to a reload:https://developer.mozilla.org/en-US/docs/Web/API/Window/sessionStorage

I've created issue7018 for managing sessions. 

I will update this review to store configuration in localstorage.
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2017-12-18.10:36:29
I think all the configuration should be stored in the localstorage of the browser (similar the configuration file of tryton).
https://developer.mozilla.org/en-US/docs/Web/API/Storage/LocalStorage
I think it can be used safely: https://caniuse.com/#search=Storage

I think the session key could be stored in the sessionStorage to survive to a reload: https://developer.mozilla.org/en-US/docs/Web/API/Window/sessionStorage
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2017-12-18.10:05:57
> Why not storing that information on the server?

Storing on the server will provide the same values for sao an tryton.

In case of tryton, the settings are preserved for all the servers you conect to. We will lose this funciontality if we store it on the server.

Another concern is how to store it on the server?
Author: [hidden] (albertca)
Date: 2017-12-18.01:06:25
Why not storing that information on the server?
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2017-12-13.17:18:33
> - configuration is not persistent

It can be persited by using a cookie. But don't know if it's worth. 

> - add an extra menu which bloat the UI

The menu should be show somewhere otherwise the option is not possible to be changed. Maybe we can find a better place. 

I placed the option in the same place as on tryton (near the favourites menu).
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2017-12-13.17:14:46
I do not think it is the right way to manage such things.
There are many issues with this way:
- configuration is not persistent
- add an extra menu which bloat the UI
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2017-12-13.16:45:09
From msg37265
History
Date User Action Args
2021-11-10 16:12:04acaubetsetmessages: + msg71590
nosy: + acaubet
2018-01-27 14:55:05yangoonsetnosy: + yangoon
2017-12-18 14:50:53reviewbotsetmessages: + msg37339
2017-12-18 13:15:37pokolisetmessages: + msg37337
2017-12-18 10:36:29cedsetmessages: + msg37332
2017-12-18 10:05:58pokolisetmessages: + msg37331
2017-12-18 01:06:26albertcasetnosy: + albertca
messages: + msg37330
2017-12-13 17:18:34pokolisetmessages: + msg37279
2017-12-13 17:14:47cedsetnosy: + ced
messages: + msg37278
2017-12-13 17:12:36pokolisetstatus: chatting -> testing

Showing 10 items. Show all history (warning: this could be VERY long)