Re: Admin actions.js performance

2017-08-07 Thread Adam Johnson
I've made a ticket here: https://code.djangoproject.com/ticket/28472#ticket . You can follow https://docs.djangoproject.com/en/dev/intro/contributing/ to make a Pull Request on GitHub against this ticket 👌 On 7 August 2017 at 09:19, ecas wrote: > Is there any way to propose this optimization to

Re: Admin actions.js performance

2017-08-07 Thread ecas
Is there any way to propose this optimization to be considered? I have never posted code to the Django community. El divendres, 4 agost de 2017 15:20:55 UTC+2, Collin Anderson va escriure: > > I can confirm it's slightly faster in my case (maybe 20-50ms for 58 rows, > 3 inputs per row). > > I ca

Re: Admin actions.js performance

2017-08-04 Thread Collin Anderson
I can confirm it's slightly faster in my case (maybe 20-50ms for 58 rows, 3 inputs per row). I can get it ~15ms faster by making it just one event handler, though it's a bigger change: $('#result_list').on('change', 'td:nth-child(n+2) :input', function() { On Fri, Aug 4, 2017 at 7:29 AM, ecas wr

Admin actions.js performance

2017-08-04 Thread ecas
Hi, I was doing some loading time benchmarking in Chrome and I noticed that the javascript that sets up the actions in the admin page takes some time. I pinpointed the problem to this jquery: $('form#changelist-form table#result_list tr').find('td:gt(0) :input').change(function() {