levels.text = $('input[name="Levels"]:checked + label');
levels.value = $('input[name="Levels"]:checked');
Those don't get you the right values for the levels object, they both
return a jQuery object.
Accessing the levels object as this.value won't work either.
Try this:
levels.each(function(){
levelsCsv.push({'text':this.value,'value':$(this).next().text()})
})
On Feb 20, 5:22 am, shapper <mdmo...@gmail.com> wrote:
> Hi,
>
> I followed your tips but I still have a few problems.
> The subject is working fine but when I do the same to the levels it
> does not work.
>
> I think the problem, but I am not sure, is in:
> levels.each(function(){
> levelCsv.push(this.value);//array to hold the levels
> });
>
> I changed everything else. I updated my
> example:http://www.27lamps.com/Beta/List/List3.html
>
> Other problem I notice is that when two levels are added the word "e"
> is not added between the both.
> Only when 3 levels are added. On my first example I had a function
> that was doing it:http://www.27lamps.com/Beta/List/List.html
>
> I just don't know how to integrate this on your code.
> I don't need to use my FriendlyLevels example. I just need to have the
> "e" every time there is more then 1 level.
>
> How can I do it?
>
> Thank You,
> Miguel
>
> - The subject is working fine.
> -
>
> On Feb 20, 1:50 am, mkmanning <michaell...@gmail.com> wrote:
>
> > You could modify the subject variable being put into the array to make
> > it an object: subject = {}
> > then add the option's value and text to it:
>
> > subject.value = $('#Subject option:selected').val();
> > subject.text = $('#Subject option:selected').text();
>
> > or in one line to replace what's there now:
> > subject = {'text':$('#Subject option:selected').text(),'value':$
> > ('#Subject option:selected').val()}
>
> > then put it in the array the same way: temptheme.push(subject)
>
> > To access it later when rebuilding just get the attributes of the
> > object:
> > var li = $('<li>').addClass('Themes').html(t[0].text+'<br />' etc...
> > for the display
>
> > $('<input>').attr({'type':'hidden','name':'Themes['+i+'].Subject'}).val
> > (t[0].value).appendTo(li); etc.... for the value
>
> > Use the same object structure for storing the checkbox names/values
>
> > On Feb 19, 4:36 pm, shapper <mdmo...@gmail.com> wrote:
>
> > > Hi Michaell,
>
> > > Sorry for the delay but when I started using JQuery 1.3 I solved this
> > > problem but I got a few others on my application.
>
> > > Anyway, there is just something else that I am not able to solve.
>
> > > When I add a theme I need to:
>
> > > 1. For the Select (Subject), add the selected value to hidden input
> > > but the text to the visible text.
>
> > > 2. For the CheckBoxes (Levels) add LevelsCsv (formed with values) to
> > > the hidden inputs.
> > > But the visible text should use the text in the labels associated
> > > to which check box ...
>
> > > I had something like that in my code:
>
> > > 1. For the subject I add something like:
> > > $subject = $('#Subject option:selected');
> > > $theme.append($subject.text()).append('<br />');
>
> > > And on the hidden input I was using $subject.val()
>
> > > 2. For the levels I was using:
> > > $levelsTexts = $('input[name="Levels"]:checked + label');
> > > $levelsValues = $('input[name="Levels"]:checked');
>
> > > So the text goes into the visible part of the theme and the value goes
> > > to the hidden inputs. See?
>
> > > I have been trying to add this to your code but I haven't be able.
>
> > > I added a new version of the List where the values and texts are
> > > different in the Select.
> > > And where each CheckBox has the associated label with the text in it.
>
> > >http://www.27lamps.com/Beta/List/List3.html
>
> > > Could you, please, help me out?
>
> > > Thank You,
> > > Miguel
>
> > > On Feb 18, 5:33 pm, mkmanning <michaell...@gmail.com> wrote:
>
> > > > The problem is with the version of jQuery you're using. Update to 1.3
> > > > (and don't include a space in the URI).
>
> > > > Try your code with the Google js api and it will work (I just did,
> > > > even with the duplicate jQuery wrapper functions).
>
> > > > On Feb 18, 9:28 am, shapper <mdmo...@gmail.com> wrote:
>
> > > > > Yes,
>
> > > > > I tried that to but the problem persists. I updated the new
> > > > > version:http://www.27lamps.com/Beta/List/List2.html
>
> > > > > I am able to add 1 theme to the list ...
> > > > > Then all the other themes I add are not added to the list ...
>
> > > > > But I think they are being added to the themes array.
>
> > > > > The index also does not change ...
> > > > > ... But the index input is on the Html code.
>
> > > > > I just can't find the problem.
>
> > > > > Thanks,
> > > > > Miguel
>
> > > > > On Feb 18, 5:21 pm, mkmanning <michaell...@gmail.com> wrote:
>
> > > > > > if you're using the ready function, remove the inner (function($)
> > > > > > { ... })(jQuery); wrapper
>
> > > > > > On Feb 18, 9:13 am, shapper <mdmo...@gmail.com> wrote:
>
> > > > > > > Hi,
>
> > > > > > > I would like to include the script on the head of the document and
> > > > > > > using an external JS file.
>
> > > > > > > So I inserted it inside the Document.Ready function. I upload a
> > > > > > > new
> > > > > > > example showing the problem I have:
>
> > > > > > >http://www.27lamps.com/Beta/List/List2.html
>
> > > > > > > In this example I didn't use the external JS file.
> > > > > > > However I placed the code inside the Document.Ready and on the
> > > > > > > head of
> > > > > > > the document.
> > > > > > > When I use the external JS file I copy it just like that to the
> > > > > > > file
> > > > > > > and add the include.
>
> > > > > > > Anyway, try to add a theme. The first is added but the second not.
> > > > > > > Or better, I think it is added to the list but not to the page ...
>
> > > > > > > Could you tell me what am I doing wrong?
>
> > > > > > > Thanks,
> > > > > > > Miguel
>
> > > > > > > On Feb 18, 4:49 pm, mkmanning <michaell...@gmail.com> wrote:
>
> > > > > > > > Scripts block other assets from downloading, so the most
> > > > > > > > appropriate
> > > > > > > > place for them (most of the time), is at the close of the body.
> > > > > > > > This
> > > > > > > > allows your page to render, and then allows you to add behavior
> > > > > > > > as
> > > > > > > > progressive enhancement. It's generally good practice; Google
> > > > > > > > progressive enhancement/graceful degradation for more on this.
> > > > > > > > You can
> > > > > > > > still include your script, just include it at the close of the
> > > > > > > > body.
> > > > > > > > Wrap it in a domready function if you like, I usually do that;
> > > > > > > > one
> > > > > > > > consequence of including your scripts at the end of the page is
> > > > > > > > the
> > > > > > > > DOM has been rendered already--which is why mine works :).
>
> > > > > > > > If you want to keep the script external AND in the head, then
> > > > > > > > you'll
> > > > > > > > have to wrap the code in a domready function.
>
> > > > > > > > On Feb 18, 8:25 am, shapper <mdmo...@gmail.com> wrote:
>
> > > > > > > > > Hi,
>
> > > > > > > > > I was trying to integrate your code on my web application and
> > > > > > > > > the
> > > > > > > > > following happens:
> > > > > > > > > I am able to add a theme. But when I try to add a second
> > > > > > > > > theme I get
> > > > > > > > > the following error:
>
> > > > > > > > > [Exception... "'type property can't be changed' when calling
> > > > > > > > > method:
> > > > > > > > > [nsIDOMEventListener::handleEvent]" nsresult: "0x8057001e
> > > > > > > > > (NS_ERROR_XPC_JS_THREW_STRING)" location: "<unknown>" data:
> > > > > > > > > no]
> > > > > > > > > data()()JQuery-1.2.6.js (line 696)
> > > > > > > > > remove()()JQuery-1.2.6.js (line 1929)
> > > > > > > > > (?)()()JQuery-1.2.6.js (line 1330)
> > > > > > > > > each()()JQuery-1.2.6.js (line 762)
> > > > > > > > > each()()JQuery-1.2.6.js (line 155)
> > > > > > > > > remove()()JQuery-1.2.6.js (line 1329)
> > > > > > > > > each()()JQuery-1.2.6.js (line 751)
> > > > > > > > > each()()JQuery-1.2.6.js (line 155)
> > > > > > > > > (?)()()JQuery-1.2.6.js (line 1348)
> > > > > > > > > (?)()()JQuery.L...-1.0.3.js (line 178)
> > > > > > > > > empty()()JQuery-1.2.6.js (line 1340)
> > > > > > > > > each()()JQuery-1.2.6.js (line 751)
> > > > > > > > > each()()JQuery-1.2.6.js (line 155)
> > > > > > > > > (?)()()JQuery-1.2.6.js (line 1348)
> > > > > > > > > (?)()()JQuery.L...-1.0.3.js (line 178)
> > > > > > > > > buildThemesList()Create (line 453)
> > > > > > > > > (?)()()Create (line 450)
> > > > > > > > > handle()()JQuery-1.2.6.js (line 2093)
> > > > > > > > > (?)()()JQuery-1.2.6.js (line 1875)
> > > > > > > > > [Break on this error] jQuery.cache[ id ][ name ] :
>
> > > > > > > > > Any idea why?
>
> > > > > > > > > And a few questions:
>
> > > > > > > > > 1. Shouldn't I place the code inside Document.Ready?
> > > > > > > > > I followed your example and didn't place it inside it but
> > > > > > > > > I don't
> > > > > > > > > understand why ...
> > > > > > > > > 2. Why do you place the script after the body?
> > > > > > > > > I am placing the script inside a js file and then I
> > > > > > > > > include it in
> > > > > > > > > the head of the document.
>
> > > > > > > > > Thanks,
> > > > > > > > > Miguel
>
> > > > > > > > > On Feb 18, 7:13 am, mkmanning <michaell...@gmail.com> wrote:
>
> > > > > > > > > > Here's a wholly different approach for consideration.
> > > > > > > > > > Rather than try
> > > > > > > > > > and keep track of LI elements and regex their names with
> > > > > > > > > > the current
> > > > > > > > > > index, just use an array placeholder and rebuild the list.
> > > > > > > > > > Chances are
> > > > > > > > > > if the user is required to add items, the list isn't going
> > > > > > > > > > to get
> > > > > > > > > > unmanageably huge (at least I'd hope not from a usability
> > > > > > > > > > standpoint).
> > > > > > > > > > Below is a refactored example, and you can see it live
> > > > > > > > > > here:http://actingthemaggot.com/test/add_theme.html
>
> > > > > > > > > > Refactored (uses markup
> > > > > > > > > > fromhttp://www.27lamps.com/Beta/List/List.html):
> > > > > > > > > > (function($) {
> > > > > > > > > > var themes = [], //this will be an array of arrays
> > > > > > > > > > to keep track, we
> > > > > > > > > > can use its indexing to rebuild the themes list, and it can
> > > > > > > > > > be
> > > > > > > > > > serialized if we want to post the data
> > > > > > > > > > ol = $('#Themes').bind('click',function(e){//event
> > > > > > > > > > delegation for
> > > > > > > > > > removing themes, livequery's fine, but this works too
> > > > > > > > > > e = $(e.target);
> > > > > > > > > > if(e.is('a')){
> > > > > > > > > > var li_index =
>
> ...
>
> read more »
