Opened 10 years ago

Closed 4 years ago

#4120 closed Cleanup/optimization (fixed)

some admin Javascript cleanups

Reported by: arvin Owned by: andrewjesaitis
Component: contrib.admin Version: master
Severity: Normal Keywords: javascript admin
Cc: andrewjesaitis Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Here are the pure javascript cleanups from ticket 4045.

Attachments (1)

admin-js.2.diff​ (4.0 KB) - added by arvin 10 years ago.

Download all attachments as: .zip

Change History (14)

Changed 10 years ago by arvin

Attachment: admin-js.2.diff​ added

comment:1 Changed 10 years ago by anonymous

Component: Uncategorized → Admin interface
Owner: changed from Jacob to Adrian Holovaty

comment:2 Changed 10 years ago by Simon G. <dev@…>

Triage Stage: Unreviewed → Accepted

comment:3 Changed 10 years ago by James Bennett

Owner: changed from nobody to xian

Reassigning to xian since he's doing admin JS stuff.

comment:4 Changed 7 years ago by Adam Nelson

Patch needs improvement: set

xian obviously isn't doing anything with this. Patch needs to be updated to current js.

comment:5 Changed 7 years ago by Adam Nelson

Owner: changed from xian to nobody

comment:6 Changed 6 years ago by Łukasz Rekucki

Severity: → Normal
Type: → Cleanup/optimization

comment:7 Changed 6 years ago by Aymeric Augustin

UI/UX: unset

Change UI/UX from NULL to False.

comment:8 Changed 6 years ago by Aymeric Augustin

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:9 Changed 4 years ago by andrewjesaitis

Cc: andrewjesaitis added
Owner: changed from nobody to andrewjesaitis
Status: new → assigned

This was quite an old ticket, but most of the original patch still applies. Consider it spring cleaning?!

It is just js cleanup. No admin functionality. The most import change is that it removes a duplicative function from calendar.js, which is also found in core.js.

The other two commits shuffle a couple functions into core.js. This ticket seemed to be a prerequisite to addressing #4045 which I will pick up once a call is made on this ticket.

The only part of the original patch that didn't apply were the changes to js/admin/ordering.js, which appears to have been removed. This change looked to be removing a duplicative function as well.

The associated pull request -- ​https://github.com/django/django/pull/948

Last edited 4 years ago by andrewjesaitis (previous) (diff)

comment:10 Changed 4 years ago by Aymeric Augustin

Patch needs improvement: unset
Triage Stage: Accepted → Ready for checkin

Looks good to me, I don't think any of these is subject to the backwards compatibility policy.

comment:11 Changed 4 years ago by Claude Paroz <claude@…>

In d5824ef69d7b58ef71175bdb5e9420d601aa194c:

Removed quickElement from calendar.js

The function quickElement in calendar.js is a duplicate of the
quickElement function in core.js. Refs #4120.

comment:12 Changed 4 years ago by Claude Paroz <claude@…>

In 0a9ea56056ee4a7ded265dc56fff309c906473d1:

Moved cancelEventPropagation into core.js

The cancelEventPropagation function is a general utility function that
can be reused by other widgets. Refs #4120.

comment:13 Changed 4 years ago by Claude Paroz <claude@…>

Resolution: → fixed
Status: assigned → closed

In d9b12ecbe62aa7c9fb0f913e7eefc21662f8e21b:

Moved removeChildren to core.js

The function removeChildren is a general utility that other functions
might want to use.
Fixes #4120. Thanks arvin for the initial patch.

Note: See TracTickets for help on using tickets.
Back to Top