Changes between Version 3 and Version 6 of Ticket #6823


Ignore:
Timestamp:
07/31/2018 09:09:01 PM (14 months ago)
Author:
mjethani
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #6823

    • Property Status changed from reviewing to closed
    • Property Resolution changed from to fixed
    • Property Summary changed from Remove unnecessary usage of the u flag in regular expressions to Remove unnecessary usage of the /u flag in regular expressions
  • Ticket #6823 – Description

    v3 v6  
    22See #6821 for the background. 
    33 
    4 In a couple of places we are using the `u` flag in regular expressions where it is not really needed (it does not affect `\s`). 
    5  
    6 Since Chrome 49 does not support the `u` flag, we should remove it. 
     4In a couple of places we are using the `/u` flag in regular expressions where it is not really needed, as it does not affect `\s` (whitespace matching). Since Chrome 49 does not support the `/u` flag, we should remove it. 
    75 
    86=== What to change === 
    9 In `lib/snippets.js` and `lib/common.js` remove the unnecessary usage of the `u` flag in the two places where it is being used. 
     7In `lib/snippets.js` and `lib/common.js` remove the unnecessary usage of the `/u` flag in the two places where it is being used. 
    108 
    119=== Hints for testers === 
    12 Same as #6781 (focus on whitespace here) and #6619. 
     10Same as #6781 and #6619, but focus on whitespace in the snippet filter and the CSS selector respectively.