ItemLevel Changes

[bthurlow]

Fix #195.
Advanced design based on pull 178.

Item Level Text can be set to 9 different positions through the configuration screen.

Upgrade Icon can be set to 9 different positions through the configuration screen.

[Alyred]

This works really well and should be integrated into the main branch. May need a little alignment/organization on the options page.

[p5hema2]

Implemented in my own fork and tested on 7.3.

Works like a charm.

Pls Merge

[AcidWeb]

I merged this changes to my ElvUI fork.

[DoubleHighC]

Hey there!! Just found this post and the changes work perfect!

Thank you very much for that and it would be awsome if I would find those changes in the next update!

Thanky you again! :-)

[Alyred]

I've made a few minor updates to this pull for the BfA stat squish, some re-coloring and the like. I set the "blue" and "purple" colors in the InventoryItemLevels method to be more like the Blizzard Rare and Epic colors, respectively, but do think they may be a bit hard to read. It's late here though and my eyes are tired. I'm considering changing them to more of a shade of the Cyan and Pink colors that were originally in the list.

Happy to see any comments.

Sorry my line levels aren't exact, I've added some comments

I'm not sure if can attach or propose additional files in this pull (sorry, only an armchair/script coder and haven't used GitHUB much for actual WORK... ;) so here's my adjustments, at around line number 262 in modules/ItemLevel.lua:

local colors = {
-- { upper bound, r, g, b }
{ 45, 0.55, 0.55, 0.55 }, -- gray, formerly 150
{ 75, 1.00, 1.00, 1.00 }, -- white, 999
{ 105, 1.00, 0.00, 0.00 }, -- red, formerly 250
{ 135, 1.00, 1.00, 0.00 }, -- yellow, 350
{ 165, 0.00, 1.00, 0.00 }, -- green, 372
{ 195, 0.00, 1.00, 1.00 }, -- cyan, 385
{ 225, 0.0, 0.439, 0.867 }, -- blue, 397, { 206, 0.00, 0.80, 1.00 }
{ 265, 0.639, 0.208, 0.933 }, -- purple, 403, { 221, 1.00, 0.50, 1.00 }
{ 295, 1.00, 0.70, 0.00 }, -- orange, formerly 300
{ 325, 1.00, 0.75, 1.00 }, -- pink, 410
}

Additionally, I modified the "level ranges" method colors to the following values (around line 370):

local maxLevelRanges = {
[60] = { 50, 62 }, -- [60] = { 66, 92 },
[70] = { 80, 94 }, -- [70] = { 100, 164 },
[80] = { 95, 105 }, -- [80] = { 187, 284 },
[85] = { 106, 115 }, -- [85] = { 333, 416 },
[90] = { 116, 135 }, -- [90] = { 450, 616 },
[100] = { 135, 150 }, -- [100] = { 615, 735 },
[110] = { 190, 265 }, -- [110] = { 805, 905 },
[120] = { 290, 320 }, -- new for BfA... completely estimated.
}

Finally, I made a change to line 204 to set the max iLevel for the "minimum iLevel threshold to show" to 250.

[aleczk]

I really wish this option, isn't anyone going to accept this pull? I'm struggling with things like this.

[aleczk]

I didn't even remember I posted this. If anyone encounters this, the way to 'fix' atm is manually changing the location at the addons folder.
Interface\AddOns\AdiBags\modules\**ItemLevel.lua**

Change the line 104 from:
text:SetPoint("TOPLEFT", button, 3, -1)

to:
text:SetPoint("BOTTOMLEFT", button, 3, 1)

You can also change the numbers after 'button' to change the position in screen. I set my y axis to 1 so it doesn't look weird. It was by default -1 because it meant to be showed on top.

Now it looks like this.

(and it does NOT go back to default when the AdiBags update)

[Talyrius]

Closed in favor of #496.