![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 16 years ago
Closed 16 years ago
#13204 closed task (blessed) (fixed)
Fix ajax on add menu item boxes
| Reported by: | koopersmith | Owned by: | filosofo |
|---|---|---|---|
| Priority: | high | Milestone: | 3.0 |
| Component: | Menus | Version: | 3.0 |
| Severity: | blocker | Keywords: | has-patch |
| Cc: | Focuses: |
Description
The ajax that switched between add menu item tabs was broken and causing a reload.
The patch fixes switching between tabs, but clicking on links to view paged results (in view all) still causes reloads.
Attachments (4)
Change History (18)
#4
@
16 years ago
- Resolution fixed
- Status closed → reopened
Paged results still cause reloads, and need to be ajaxified.
#7
@
16 years ago
Latest patch shifts jQuery extensions into their own function and modifies the addSelectedToMenu extension to make one ajax call instead of one per item.
#10
@
16 years ago
We still need to add ajax for paged results, but since we're using checkboxes it's less of an issue. That said, if a user clicks on a page link they will lose the menu order (as it equates to a no-save, see #13337) so this is probably worth adding for 3.0.
Note:
See TracTickets
for help on using tickets.
(In [14332]) Fix ajax on add menu item boxes. props koopersmith, see #13204