Web Development

« first day (47 days earlier) ← previous day next day → last day (160 days later) »

Ismael Miguel

15:27

@EBrown Greetings

EBrown

By the way pings don't work like that.

You have to send an invite for it to notify the other user.

Unless they are already in the room.

Ismael Miguel

I have no idea how to do that

But anyway

Explain me a bit more the problem you have

EBrown

I need the browser to execute the following line, then actually display the changes before the loop starts:

$("#searchingResultsFound").eq(0).html(items.length + " results found. Please wait while we render your items.");

Ismael Miguel

I have 4 questions:

1 - Does $("#searchingResultsFound") return a length of 1?

2 - Why .eq(0)? You have an #id. If you have multiple elements with that ID, change it now

3 - Does the loop take some time to execute?

4 - If there any error in your console?

EBrown

1. Irrelevant.
2. Irrelevant.
3. Yes, which is why I need it to update immediately.
4. No.

Ismael Miguel

15:34

Point 1 and 2 are extremelly important

If you have more than 1 element with id searchingResultsFound, it won't work

EBrown

I do not, but the fact that I use .eq(0) is irrelevant.

I want a guarantee that I only have one item.

Ismael Miguel

Then you are doing it wrong

Using the id selector will select the very first element with the matching id

If you want to guarantee you only have 1 element, and you are using an id selector, your HTML has serious issues

EBrown

Yes, and using .eq(0) guarantees I only have one element.

Because I don't trust JavaScript, JQuery, or anything about it.

Ismael Miguel

Using .eq(0) is both a waste and makes anyone think that your HTML is broken

And yes, repeated id's are invalid, rendering the whole document as broken and invalid

EBrown

I know. If all you are going to do is complain about eq(0) then I've no use for you.

It's there. It's staying there. End of story.

Ismael Miguel

15:41

Are you using repeated ids?

EBrown

Nope.

Ismael Miguel

Do you have your code inside a $(function(){ [...] })?

EBrown

        function OnItemsDownloadSuccess(response) {
            var xmlDocument = $.parseXML(response.d);
            var xml = $(xmlDocument);
            var items = xml.find("Table1");
            var i = 0;
            //alert(response.d);

            if (items.length > 0) {
                $("#searchingResultsFound").eq(0).html(items.length + " results found. Please wait while we render your items.");

                while (i < items.length) {
                    var item = $(items[i]);

Of which items can be anywhere from 0 to 3000+ elements.

Then, within that loop, it either removes the row, or it adds data to about 20 elements.

                        row.attr("id", "search-" + item.find("Number").text());
                        $(".item-awp", row).eq(0).html(item.find("AWP").text());
                        $(".item-brand-name", row).eq(0).html(item.find("BrandName").text());
                        $(".item-category", row).eq(0).html(item.find("Category").text());

Etc.

Ismael Miguel

Try adding a setTimeout(function(){ ... }, 1) on your code

EBrown

@IsmaelMiguel And why would I do that?

Ismael Miguel

15:47

And make a copy of whatever you need to change and shove it inside a DocumentFragment. Manipulate that and then you change the HTML of your table or whatever

@EBrown By doing that, you let the function finish and return, without grinding. Also, it gives time to render whatever the browser has to

It is something that can and should be done assyncronously (misspelled?)

EBrown

It should be as non-blocking as possible, as there is a modal to indicate progress.

Ismael Miguel

And the way I told doesn't work?

EBrown

Wrapping the entire function in setTimeout doesn't work.

Still blocks.

Ismael Miguel

I may have something for you

Q: Function queue for synchronous execution

The purpose of this code is to allow to add functions to a queue, that will execute them sequentially. With this in mind, it also allows to execute them with a defined delay, to prevent them from blocking the browser. (function(window, undefined){ 'use strict'; var queue = []; var r...