FUDforum
Fast Uncompromising Discussions. FUDforum will get your users talking.

Home » FUDforum Development » Bug Reports » Selected and Unselected mix-up?
Show: Today's Messages :: Polls :: Message Navigator
Switch to threaded view of this topic Create a new topic Submit Reply
Selected and Unselected mix-up? [message #20933] Wed, 03 November 2004 15:05 Go to next message
Abraxa is currently offline  Abraxa   Germany
Messages: 72
Registered: August 2004
Location: Germany
Karma: 0
Member
Heya Ilia =)

I don't know if you did it on purpose or not but I found something that made me wonder a little.
In draw_select_opt.tmpl, you define two sections:

{SECTION: selected_option}
<option value="{VAR: vls[$i]}" {VAR: normal_tmpl}>{VAR: nms[$i]}</option>
{SECTION: END}

{SECTION: unselected_option}
<option value="{VAR: vls[$i]}" selected {VAR: selected_tmpl}>{VAR: nms[$i]}</option>
{SECTION: END}


As you can see, the unselected section has the "selected" property while the selected section doesn't - isn't it supposed to be the other way round, judging from the names you gave the sections?
The mix-up doesn't have any consequences since in draw_select_opt.inc.t you do a "!=" comparison instead of "==" but still I wonder if that's something that maybe should be fixed for reasons of code logic?

(btw, same thing occurs in draw_radio_opt.*)

-Abraxa
Re: Selected and Unselected mix-up? [message #20935 is a reply to message #20933] Wed, 03 November 2004 17:17 Go to previous messageGo to next message
Ilia is currently offline  Ilia   Canada
Messages: 13241
Registered: January 2002
Karma: 0
Senior Member
Administrator
Core Developer
This is something that is addressed through recent template optimizations and revisions.

FUDforum Core Developer
Re: Selected and Unselected mix-up? [message #20936 is a reply to message #20933] Wed, 03 November 2004 18:05 Go to previous message
Abraxa is currently offline  Abraxa   Germany
Messages: 72
Registered: August 2004
Location: Germany
Karma: 0
Member
oh ok =)

-Abraxa
  Switch to threaded view of this topic Create a new topic Submit Reply
Previous Topic: SQL error on upgrade
Next Topic: Moderator can't post in password protected forum
Goto Forum:
  

-=] Back to Top [=-
[ Syndicate this forum (XML) ] [ RSS ]

Current Time: Thu Nov 28 16:09:37 GMT 2024

Total time taken to generate the page: 0.04234 seconds