Opened 6 years ago

Closed 6 years ago

#3606 closed defect (bug) (fixed)

Fix regression of user note voting using Ajax

Reported by: keesiemeijer's profile keesiemeijer Owned by: obenland's profile obenland
Milestone: Priority: normal
Component: Developer Hub Keywords: has-patch


In changeset [6400] the fix for dynamically added voting links in #1745 was reversed. This means the voting no longer uses Ajax to handle the vote submission.

The following patch adds the fix back and adds aria-hidden="true" to the empty span elements for the voting arrows.

Attachments (1)

3606.patch (1.7 KB) - added by keesiemeijer 6 years ago.
Fix regression

Download all attachments as: .zip

Change History (3)

6 years ago

Fix regression

This ticket was mentioned in Slack in #meta by sergey. View the logs.

6 years ago

#2 @obenland
6 years ago

  • Owner set to obenland
  • Resolution set to fixed
  • Status changed from new to closed

In 7276:

Developers: Keep using AJAX for multiple votes

Fixes a regression where the AJAX handler was not working for dynamically added voting links.

Props keesiemeijer.
Fixes #3606.

Note: See TracTickets for help on using tickets.