Re: [Fofredux-devel] options page redone

[Kevin <ke...@dr...>]

Evan Roth wrote:
> hey guys,
>
> i've redone the options page in preparation for the redesign of the site:
> - according the design images, i've changed all the yes/no radios into
> checkboxes
> - it is now valid xhtml without tables (except for a few lines coming
> from header.php, which are still on the todo list)  [so khaled can go
> nuts]
> - i've reorganized the list of items a bit, beginning to group them in
> various sections.  i've just thrown it together quickly, so we can
> change that as needed
> - on the backend, i've redone the saving of items, since we were
> always saving a value, regardless of what it is.  i've put a bunch of
> checks in there, so we only change it when it's actually been changed.
> this also means that if a user sticks with a default value, it will
> not show up in the database
>
> I've added 1 bug that i've found in the process to the tracker at
> sourceforge.  for whatever reason, you guys drop the table and set it
> back up in install.php.  in the past, this was probably not an issue
> with only 2 options...but now that our list is growing, we can't
> afford to do this as it will lose all the user's settings.
>
The explicit drop/recreate of the config table was due to the table 
originally having a different structure.  That was a few months ago, so 
I don't think it's needed anymore.

-Kevin