raethkcj / RatingBuster

An item comparison tool for WoW Classic.
GNU General Public License v2.0
57 stars 14 forks source link

1.3.7 broke ratings percentages in gear tooltips #72

Closed Hantunaar closed 2 years ago

Hantunaar commented 2 years ago

Hey so latest update (1.3.7) broke percentages in gear tooltips in french, and probably others non-english clients.

Green lines like "Equip: Improves spell hit rating by 17" no longer display the percentage value of ratings.

According to patch changes, this is related with the line {pattern = " de (%d+)[^%%]?$", addInfo = "AfterNumber",}, and more exactly with the $ thing. Removing it brought back the percentages, but whatever it did somewhere else might be clunky now.

Also, if work has to be done there, could it be possible to add a blankspace after the percentages? So it doesn't stick to the next word.

raethkcj commented 2 years ago

Sorry about that, working on a fix.

I only just installed frFR locale, but I see your point about percentages sticking to the next word. So I'll fix that as well.

Just for context, the reason I'm modifying these is to fix a handful of items that are incorrectly converting percentages to rating, for example in the set bonus at the bottom of this piece, 5% shouldn't be converted: image

In hindsight, adding $ was a bad idea, but I think I can fix it by breaking it into two separate patterns.