search

ICT-Infer / timelog

Time sheet time tracking
ISC License
1 stars 0 forks source link

Query string parameter no-default-true is ugly. #4

Closed yarwelp closed 9 years ago

yarwelp commented 9 years ago

We should get rid of it.

Proposed solution: Redirect request with empty query string to same page but with a query string with default values.

yarwelp commented 9 years ago

Maybe not redirect, though -- that takes extra time for little gain.

yarwelp commented 9 years ago

I also don't want to surprise the user in a negative way, though.

It might seem strange to the user that they manually add an option and suddely other options are unselected.

It might seem, then, that I might have to go with radio buttons instead of check boxes. That's too bad, though, because I prefer check boxes over radio buttons for truth values. That will make URLs longer and filled with annoying values which are the default values unless I then strip parameters which are set to their defaults and redirect which, once again, would mean more time between user action and result in browser.

One might also do as is common with cli long-options and prefix options which default to true with "no" but that can look a bit messy as well.

yarwelp commented 9 years ago

Hm, nope, not closing this ticket yet. The first commit to resolve part of this was god. The second one to take care of the rest was not.