Hi Chris, Whatever happens, you should always be able to see one empty file input (disabled if you've reached the limit).
I'll look into it.... Thanks, Diego A. On Apr 9, 9:22 pm, Christopher Jordan <[EMAIL PROTECTED]> wrote: > Oh! I see. Well, thanks for setting me straight. I didn't really > understand what I was seeing then I guess. It's funny, I had five files > "uploaded" but only three of the browse controls had anything in them. > After I removed them all, shouldn't I have been left with only one > browse control? > > I've not looked at the code under the hood, and suppose it doesn't > really matter *too* much to me as long as it works. :o) Anyway, good > idea for the plugin. > > Cheers, > Chris > > > > Diego A. wrote: > > > Example 3 is in debug mode so you see what;s happening behind the scences. > > It > > doesn't hide the multiple file inputs... > > Not an error, just a demostration of how it actually works. > > > ;-) > > > Christopher Jordan wrote: > > >> Deigo, > > >> I like it too. Cool. I did notice the same as Scott about the limitation > >> error on example 2. I also noticed a problem where example 3 kept adding > >> browse boxes and buttons. I started with one, then got an additional one > >> for each file I added, and after removing all the files, I was left with > >> two brows controls right next to each other. :o' > > >> Otherwise, great work! I think I could definitely find use for this. :o) > > >> Chris > > >> Scott Sauyet wrote: > >>> Diego A. wrote: > >>>> The name says it all. Just thought somebody might find it useful. > >>>> Please reply to this thread with suggestions, comments... > > >>>>http://www.fyneworks.com/jQuery/multiple-file-upload/ > >>> Looks very nice. I love the clean markup. > > >>> One bug. Testing your demo page in FF2.0.0.3 and IE7.0.5730.11 both on > >>> WinXPsp2, I found that the "limit-3" limits me to two entries, not > >>> three. I didn't look at the code, but I'm guessing it's a typical > >>> off-by-one issue. > > >>> Thanks for sharing this, > > >>> -- Scott > > >> -- > >>http://www.cjordan.us > > --http://www.cjordan.us
