Opened 3 years ago
Closed 3 years ago
#13204 closed task (blessed) (fixed)
Fix ajax on add menu item boxes
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | high | Milestone: | 3.0 |
| Component: | Menus | Version: | 3.0 |
| Severity: | blocker | Keywords: | has-patch |
| Cc: |
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)
koopersmith — 3 years ago
comment:4
koopersmith — 3 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Paged results still cause reloads, and need to be ajaxified.
- Priority changed from normal to high
- Type changed from defect (bug) to task (blessed)
koopersmith — 3 years ago
koopersmith — 3 years ago
comment:7
koopersmith — 3 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.
comment:10
koopersmith — 3 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.
comment:11
nacin — 3 years ago
- Severity changed from normal to blocker
With the AYS now, it's required for 3.0.
comment:12
filosofo — 3 years ago
- Owner changed from koopersmith to filosofo
- Status changed from reopened to accepted
Doing ajax pages.
comment:13
filosofo — 3 years ago
- Keywords has-patch added
ajaxy-pagification-of-meta-boxes.13204.diff makes the paginated meta-box links ajaxified.
comment:14
ryan — 3 years ago
- Resolution set to fixed
- Status changed from accepted to closed

(In [14332]) Fix ajax on add menu item boxes. props koopersmith, see #13204