[V7.2][International][Grischu] Override for InventoryAndStashPlugin.cs

[Grischu]

Shows a blue S in the bottom right corner if item is in at least one saved itemset.
Credits for default plugin goes to KillerJohn.



Link



Place this File into /plugins/Grischu/Overrides

[JackCeparou]

Not a bad idea, but it's doable without forking the whole inventory plugin.

[bm206]

+1 for a steal by KJ for default plugin.

[bm206]

Not a bad idea, but it's doable without forking the whole inventory plugin.

And +1 for this idea!

[Grischu]

Not a bad idea, but it's doable without forking the whole inventory plugin.

I know, but i secretly hope KJ will put it into Default Plugin *Hint*

[JackCeparou]

I know, but i secretly hope KJ will put it into Default Plugin *Hint*

Ok
One remark so far :

        private void DrawItemArmorySet(IItem item, System.Drawing.RectangleF rect) 
        { 
            if (!ArmorySetEnabled) return; 
            var setstring = ""; 
            foreach (IPlayerArmorySet set in Hud.Game.Me.ArmorySets) 
            { 
                if (set.ContainsItem(item)) 
                { 
                    setstring = "S"; 
                } 
            } 
            if (!String.IsNullOrEmpty(setstring)) 
            { 
                var textLayout = ArmorySetFont.GetTextLayout(setstring); 
                ArmorySetFont.DrawText(textLayout, rect.X + 2, rect.Bottom - rv / 35.0f - textLayout.Metrics.Height); 
            } 
        }  

=>

        private void DrawItemArmorySet(IItem item, System.Drawing.RectangleF rect) 
        { 
            if (!ArmorySetEnabled) return; 
            if (!Hud.Game.Me.ArmorySets.Any(set => set.ContainsItem(item))) return; 
 
            var textLayout = ArmorySetFont.GetTextLayout("S"); 
            ArmorySetFont.DrawText(textLayout, rect.X + 2, rect.Bottom - rv / 35.0f - textLayout.Metrics.Height); 
        }  

[prrovoss]

yep, it should be marked somehow which items are used in an armory build. good idea.

[Grischu]

I have done it in HoveredItemInfoPlugin.cs too, but there i have a problem. I only get 4 sets, not 5

[JackCeparou]

I have done it in HoveredItemInfoPlugin.cs too, but there i have a problem. I only get 4 sets, not 5

Just tested, you are right, the set.ContainsItem(item) return always false on the 5th.
Also, the set detection seems to not work on amulet.

[JackCeparou]

A little placement issue with your current implementation (no clue how to resolve)
It overlap AncientRank on enchanted rings.

[Grischu]

I also don't know. Upper right corner is the cube decorator. This is a bad position too

[prrovoss]

hm maybe a thin frame around the item?

edit
might look ugly for equipped items...

[Stormreaver]

instead maybe you can just draw a thin teal border around the item or a thin corner at lower left?

[Grischu]

How is it like this

[Mendacium]

Nice! (I.e. your plugin, and the new position/design.)