Skip to content

feat(bags): Nest by Expansion in All Items view#320

Open
liamcooper wants to merge 1 commit into
EllesmereGaming:mainfrom
liamcooper:feat/bags-nest-by-expansion
Open

feat(bags): Nest by Expansion in All Items view#320
liamcooper wants to merge 1 commit into
EllesmereGaming:mainfrom
liamcooper:feat/bags-nest-by-expansion

Conversation

@liamcooper
Copy link
Copy Markdown
Contributor

Summary

Adds an optional Nest by Expansion mode for the All Items bag view. When enabled, each sidebar category (and merged category groups) shows its items under indented expansion sub-headers (e.g. Midnight, Dragonflight, Wrath of the Lich King), using C_Item.GetItemInfo expansion metadata. A sub-header is shown for every expansion represented in that category, including when only one expansion is present. Uncached or ambiguous items are grouped under Unknown.

The setting lives under EllesmereUI → Bags → EXTRAS, paired on the same row as Show Pinned & Recent Tips, and defaults to off.

Screenshots

All Items (expansion nesting enabled)

All Items view with expansion sub-headers under each category

Bags settings

Bags module EXTRAS section with Nest by Expansion toggle

Implementation notes

  • EllesmereUIBags.lua: expansion id/label helpers, bucket builder, sub-header pool, RenderSection path for All Items when the option is on.
  • EUI_Bags_Options.lua: toggle UI and refresh on change.
  • Screenshots for this PR are stored under .github/pr-screenshots/ for stable image URLs in the description.

Testing

  • Toggle off: All Items matches prior flat layout per category.
  • Toggle on: categories show expansion rows; single-expansion categories still show one expansion row.
  • /reload after changing the option updates the bag grid.

Add a default-off "Nest by Expansion" option under Bags > EXTRAS.
When enabled, the All Items inventory view indents items under
per-expansion sub-headers within each category (and merged groups),
using item expansion metadata from the client API.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant