Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Harder, better, faster, stronger #78

Merged
merged 3 commits into from
Mar 6, 2013
Merged

Harder, better, faster, stronger #78

merged 3 commits into from
Mar 6, 2013

Conversation

daxxog
Copy link
Contributor

@daxxog daxxog commented Mar 6, 2013

Harder, better, faster, stronger

[proper version, merge this]

What is included in this pull request?

  • make.js file for easy building via nodejs
  • Over 2x speed increase for grid rendering!
    • 50 items with old script took took 619ms to render on my machine
    • 50 items with new script takes 255ms to render on my machine
  • Selector caching system. Other people can use this to speed up the grid even more!

Selector cache example

old code

$('th:eq(' + n + ')', g.hDiv);

new code

g.cacheSel('th:eq(' + n + ')', 'hDiv');

paulopmx added a commit that referenced this pull request Mar 6, 2013
Harder, better, faster, stronger
@paulopmx paulopmx merged commit 6c4f98c into paulopmx:test Mar 6, 2013
@NaomiN
Copy link

NaomiN commented Mar 29, 2013

Hi,

I just downloaded the latest suggested version of both flexigrid.css and
flexigrid.js (and packed versions). I see that my Add, Edit, Delete buttons
no longer have pictures - what should I do to make them show their pictures
again?

Also, I had the following code to allow double clicking on the selected row
to invoke Edit functionality:

function bindDblClick()
{
$('#flexClients tr').dblclick(function () {
edit('Edit');
});
}

This also doesn't work anymore. Finally, all my dialogs now look very
weird. They look like they are now transparent and the grid bleeds through
them. Also, the close button of the dialogs doesn't look like it used (the
x at the top, but the button with close instead). I am relatively new to
jQuery and I had my project working Ok about 2+ months ago. Today I tried
running it again and got all these problems.

I attached my js file that defines dialogs and flexigrid - may be you can
help me.

Thanks a lot again.

On Tue, Mar 5, 2013 at 8:36 PM, David Volm [email protected] wrote:

Harder, better, faster, stronger

[proper version, merge this]

What is included in this pull request?

  • make.js file for easy building via nodejs
  • Over 2x speed increase for grid rendering!
    • 50 items with old script took took 619ms to render on my machine
    • 50 items with new script takes 255ms to render on my machine
      • Selector caching system. Other people can use this to speed up
        the grid even more!

Selector cache example

old code

$('th:eq(' + n + ')', g.hDiv);

new code

g.cacheSel('th:eq(' + n + ')', 'hDiv');


You can merge this Pull Request by running

git pull https://github.com/daxxog/Flexigrid pull2

Or view, comment on, or merge it at:

#78
Commit Summary

  • Harder, better, faster, stronger
  • My changes to the JavaScript source.
  • Cleaned up a bit.

File Changes

Patch Links:

@NaomiN
Copy link

NaomiN commented Mar 29, 2013

Who is RS (that's the initials I see for the version I downloaded) and
given this code

if (btn.bimage) // if bimage defined, use its string as an image url for
this button style (RS)
$('span',btnDiv).css( 'background', 'url('+btn.bimage+') no-repeat center
left' );
$('span',btnDiv).css( 'paddingLeft', 20 );

if (btn.tooltip) // add title if exists (RS)
$('span',btnDiv)[0].title = btn.tooltip;

and the following structure in my MVC application

Content\Themes\FlexiGrid\Images\add.png

what should I put for bimage property? Also, before the latest update the
images for buttons showed up automatically - can we somehow specify default
images on the class level?

Thanks again.

On Thu, Mar 28, 2013 at 7:47 PM, Naomi Nosonovsky [email protected]:

Hi,

I just downloaded the latest suggested version of both flexigrid.css and
flexigrid.js (and packed versions). I see that my Add, Edit, Delete buttons
no longer have pictures - what should I do to make them show their pictures
again?

Also, I had the following code to allow double clicking on the selected
row to invoke Edit functionality:

function bindDblClick()
{
$('#flexClients tr').dblclick(function () {
edit('Edit');
});
}

This also doesn't work anymore. Finally, all my dialogs now look very
weird. They look like they are now transparent and the grid bleeds through
them. Also, the close button of the dialogs doesn't look like it used (the
x at the top, but the button with close instead). I am relatively new to
jQuery and I had my project working Ok about 2+ months ago. Today I tried
running it again and got all these problems.

I attached my js file that defines dialogs and flexigrid - may be you can
help me.

Thanks a lot again.

On Tue, Mar 5, 2013 at 8:36 PM, David Volm [email protected]:

Harder, better, faster, stronger

[proper version, merge this]

What is included in this pull request?

  • make.js file for easy building via nodejs
  • Over 2x speed increase for grid rendering!
    • 50 items with old script took took 619ms to render on my
      machine
    • 50 items with new script takes 255ms to render on my machine
      • Selector caching system. Other people can use this to speed up
        the grid even more!

Selector cache example

old code

$('th:eq(' + n + ')', g.hDiv);

new code

g.cacheSel('th:eq(' + n + ')', 'hDiv');


You can merge this Pull Request by running

git pull https://github.com/daxxog/Flexigrid pull2

Or view, comment on, or merge it at:

#78
Commit Summary

  • Harder, better, faster, stronger
  • My changes to the JavaScript source.
  • Cleaned up a bit.

File Changes

Patch Links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants