53

Well, I believe this to be a tad minor.. but still a bug nonetheless.

I recall that sometime today, a new display method for close votes was enabled.. next to the close button, instead of close 1, it now displayed close (1/5), or similar.. The problem comes in voting to close on a question that already has such. It instead comes out to be close (1/5) (2/5); example below:

enter image description here

I suppose I'm wondering if this is indeed a bug, or the intended behavior.. if not, well, here's the report.

| improve this question | |
  • 15
    Yep, reproduced. I'd say it's a bug. – Bart Mar 14 '13 at 8:01
  • 4
    Agreed - just came on here to post that. – Duncan Jones Mar 14 '13 at 8:39
  • Best guess is that it inserted the 2/5 after recieving an update without clearing the original – Riking Mar 14 '13 at 8:47
  • 6
    Obligatory eeeeeeeek! :D – Anirudh Ramanathan Mar 14 '13 at 8:59
  • 7
    I find your lack of freehand circles disturbing. – J. Steen Mar 14 '13 at 9:04
  • 2
    @J.Steen better? ;-) – Shadow Wizard Wearing Mask Mar 14 '13 at 9:48
  • 1
    @ShaWizDowArd You oscillate the cockles of my heart. – J. Steen Mar 14 '13 at 9:54
  • @Marc Is this fixed? I still can reproduce it (or by fixed - does it mean next build) – Jon Clements Mar 14 '13 at 13:33
  • 2
    @JonClements it will be fixed in the next deploy. If you look in the bottom-right of the page, it says "rev 2013.3.14.856". When it says something else the answer will be "yes" – Marc Gravell♦ Mar 14 '13 at 13:55
  • @MarcGravell awesome - I'll get ya a pint next time I'm around your area of the woods mate ;) – Jon Clements Mar 14 '13 at 14:00
  • @MarcGravell - The issue is not resolved yet. I just voted to close a question and it show me once again. :( – hims056 Mar 15 '13 at 8:50
  • 2
    @hims056 "When it says something else". It doesn't say something else yet, so the new revision hasn't been deployed yet. – J. Steen Mar 15 '13 at 9:10
  • @J.Steen - Oh yeh. I extremely need a cup of coffee. – hims056 Mar 15 '13 at 9:13
  • @hims056 Coffee is very good. I'll have a cup myself, I think. =D – J. Steen Mar 15 '13 at 9:14
12

Ah yeah, this sounds a bit familiar. Given that I don't think parentheses ever appear in these links except for in this particular case, that regular expression can probably be genericized to just

a.text().replace(/\s?\([^)]+\)$/, "")

anyway. Barring that,

a.text().replace(/\s?\(\d\/\d\)$/, "")

should work too.

There's also extra whitespace on either side of the text, which makes the links display slightly inconsistently from the other options.

| improve this answer | |
  • 1
    fixed - I went with /\w?\(\d(\/\d)?\)/ to go with the two formats we might expect – Marc Gravell♦ Mar 14 '13 at 12:27
  • That's fair. I still maintain that \w? should be \s? though, since it seems to want to remove the space it re-adds during the subsequent string concatenation. – Tim Stone Mar 14 '13 at 12:29
  • 1
    After years of seeing those, regular expressions still look like Orz Nemesis battleship to me. :-P – Shadow Wizard Wearing Mask Mar 14 '13 at 12:37
  • 1
    @ShaWizDowArd: Trade-off of power in the expression vs. readability. I think APL is the same. – nhahtdh Mar 14 '13 at 12:45
  • 2
    Just happened to me a minute ago. Not fixed. – j08691 Mar 14 '13 at 13:37
  • @j08691 not just me then ;) – Jon Clements Mar 14 '13 at 13:40
  • @j08691 The fix probably hasn't been deployed yet. I also just saw the problem. – andyb Mar 14 '13 at 14:50
  • @andyb - the status is marked as completed though. – j08691 Mar 14 '13 at 14:50
  • Completed does not mean deployed. I guess it all depends on what the definition of "complete" is. The defect is fixed in 2013.3.14.856. Current SO version is 2013.3.14.553 so it not deployed to SO yet. It is deployed to meta though (see right column of the footer). If you are still seeing it on meta still then I might agree with you :-) – andyb Mar 14 '13 at 14:53
  • 6
    I don't like this feature anyway; it makes the spacing all inconsistent. – Lightness Races in Orbit Mar 14 '13 at 17:49

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .