Problem/Motivation

Spotted while reviewing #994360: #states cannot check/uncheck checkboxes elements, states.js contains this comment:

      // Note: WebKit nightlies don't reflect that change correctly.
      // See https://bugs.webkit.org/show_bug.cgi?id=23789

This comment was introduced in the first version of states.js in #557272: FAPI: JavaScript States system in 2009. The bug was closed as fixed in 2011. Therefore, we should remove this outdated comment.

Steps to reproduce

Proposed resolution

Remove the comment.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3347492-2.patch438 byteslongwave

Comments

longwave created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Needs review
StatusFileSize
new 3347492-2.patch438 bytes
dww’s picture

dww
we/he/they
commented
Status: Needs review » Reviewed & tested by the community

Simple, clear, easy, good. 😉 Thanks!

  • lauriii committed dee67cc1 on 10.1.x 
    Issue #3347492 by longwave: Remove outdated comment about WebKit from...
lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
commented
Status: Reviewed & tested by the community » Fixed

Committed dee67cc and pushed to 10.1.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.