System statistics network widget doesn't remember Maximum

[troffasky]

Expected Behavior

Changes to "Maximum" slider should be saved on Close

Current Behavior

Changes to "Maximum" slider are forgotten when Close is clicked

Steps to Reproduce (for bugs)

1. Add system statistics widget to panel
2. Set Type to Network
3. Adjust Maximum slider
4. Click Close

System Information

• Distribution & Version: Debian SID
• Kernel: 4.8.0-2-amd64 compile lxqt-panel, error..., What package should be installed, but sysstat lib is installed. #1 SMP Debian 4.8.11-1
• Qt Version: 5.7.1+dfsg-3+b1
• liblxqt Version: 0.11.1-1
• Package version: 0.11.1-1

[pmattern]

Confirmed as depicted above running yesterday's master checkouts of all LXQt components including lxqt/lxqt-panel@94a534b1 on Arch Linux.
At first glance other settings of the plugin do not seem to suffer from the problem.

@troffasky
It would be nice if you could have another close look at the various other settings of the plugin to see whether really this one setting only is affected.

[troffasky]

"Logarithmic scale" checkbox is remembered, associated steps are not. All other settings seem to be remembered.

[tsujan]

I confirm this too. I'm sure that the setting was saved before but I don't know when this regression started because I saved the setting about 2 years ago ;)

[palinek]

Please, have a look on lxqt/lxqt-panel#395

[pmattern]

Running lxqt/lxqt-panel@c708e7ae only values ≥1kB/s are saved correctly. Values <1kB/s, albeit eligible by the slider, always fall back to 1B/s when reopening the dialogue.

Besides not mentioned in this issue's title it turned out the number of steps of the "Logarithmic scale" isn't saved either. Any chance you can have a look at this, too?

[tsujan]

Values <1kB/s, albeit eligible by the slider, always fall back to 1B/s when reopening the dialogue.

I think the minimum value of the maximum should be changed to a reasonable number, 10kKB/s, for example, or even more.

EDIT: 128kB seems more sensible to me.

the number of steps of the "Logarithmic scale" isn't saved either.

I haven't tested it but if so, it should be about a missing line of code too.