DejaVu Wrote:I never noticed that as my SubNav is at the bottom of the page and is not too intrusive, that being said, it will not let me delete the last entry in the settings page anyway. Is that something you did to try to rectify it?
No really, it was just a bug in my JS function which I already fixed, but it breaks things when saving.
Quote:Deleting the entry manually from config.ini removed the SubNav fine here.
Ye editing it manually will always work fine.
Quote:One thing I managed to do was screw it up saving one, and the config.ini was full of 'arrays'. Cannot recreate this ATM.
Ye, I've seen that one happening a few times. Still trying to find a fix.
Quote:As for the reoccurring save errors, I was hoping that was an easy fix.
It just seems like too many requests are being sent, could they just not be suppressed somehow?
Ye don't worry about those. That's just me trying to get some feedback from javascript.
Quote:I'm going to make a youtube video of setting mine up and you can see what I mean.
There you go - on XBMCLive accessed by Windows 7.
http://www.youtube.com/watch?v=_H7NlRX2jiI
Just watched the video. Ye, the version you have of the settings page is definitely not very good. I've been making quite a few changes. The annoying alerts don't come up as often anymore. Only "settings saved" or "error: description of the error" which I think is a bit more helpful. I'm still having problems with a few things but all in all I still think this is easier than editing the php file. In a couple of weeks I expect it to be quite polished and possibly ready.
I've been thinking about the alerts tho. They get annoying because there are so many sections to save. Maybe we can find another system to give feedback about the saving actions. What do you think?
Also, in regards to reloading, the code is mostly there already. Just have a look in the onclick function for the save button on the Nav section. It's just that I was reloading the frame where the Nav was in the "frameset" version but in your version you can just run a window.reload or something and it will work fine.
Try and update my latest couple of commits to your repo if you want, but know that I've been changing it a lot so it may not be worth it yet. You might want to wait for my next push tomorrow or the day after. I'm commenting the code to make it easier to understand. I have also noticed your servecheck is a bit different. Have you updated it? If not, get the one I've made which already ensures a copy from default-config.INI is made and not PHP.