Opened 6 months ago

Closed 6 months ago

Last modified 6 months ago

#22819 closed defect (bug) (fixed)

Trailing commas are bad.

Reported by: koopersmith Owned by: koopersmith
Priority: normal Milestone: 3.5
Component: General Version: 3.5
Severity: normal Keywords: has-patch commit
Cc:

Description

They make IE7 cry.

Attachments (1)

22819.diff​ (402 bytes) - added by koopersmith 6 months ago.

Download all attachments as: .zip

Change History (7)

koopersmith — 6 months ago

comment:1   miqrogroove — 6 months ago

IE7 makes me cry.

comment:2   azaozz — 6 months ago

Patch fixes it here, +1 to commit.

comment:3   gcorne — 6 months ago

Fixes it for me, commit her.

comment:4   sunnyratilal — 6 months ago

Fixed for me.

comment:5   koopersmith — 6 months ago

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

In 23124:

Trailing commas make IE7 cry. fixes #22819.

comment:6   wonderboymusic — 6 months ago

there should be an IE7 bot that just scrapes for ,\s*}

Note: See TracTickets for help on using tickets.