search

njs50 / poe_ext

Path of Exile Inventory Management and Crafting Helper (Chrome Plugin)
http://njs50.github.io/poe_ext/
27 stars 14 forks source link

Add locationTable directly to popover in inventory view #38

Closed JonDum closed 11 years ago

JonDum commented 11 years ago

Screen shot 2013-02-17 at 4 18 43 PM

I found myself consistently looking for the location table when I hovered over an item to see its stats rather than having to hover, close the hover, and hit show/hide.

So I added a copy of the location table to the popover, quick and simple.

No errors, checkout my fork and test if you need to.

In the future I may suggest caching the location tables for each item instead of recreating the html each time getLocationTable is called. Not sure if it's even much of a performance impact though.

njs50 commented 11 years ago

looks good to me. it'll go out in the next update, which probably wont be too far away.