Skip to content

World marker categories#437

Open
Ivinedra wants to merge 13 commits into
una-xiv:mainfrom
Ivinedra:World-Marker-Categories
Open

World marker categories#437
Ivinedra wants to merge 13 commits into
una-xiv:mainfrom
Ivinedra:World-Marker-Categories

Conversation

@Ivinedra

Copy link
Copy Markdown
Contributor

Add support for world marker categories, both in the widget and in the settings menu:

Widget Image Setting Image

Ivinedra and others added 11 commits February 7, 2026 08:55
Add categorized, collapsible grouping for world marker UI and settings. Introduces WorldMarkerCategoryDefinitions (new) to define category order and assignments, updates MarkerControlWidget and SettingsWindowMarkerSettingsModule to build grouped UI from the registry, and adds collapsible group behavior and label prefixes to MenuPopup. Also updates templates/styles (markers_module.xml, _popup_menu.xml) and i18n entries for category labels and some marker names to support the new grouping and UI. These changes reorganize marker lists to improve discoverability and allow collapsing less-used groups.
Because VS is yelling at me.
This reverts commit ba0e5f1.
Adds support for the Elementals found in the various Eureka zones.
Was one of the only markers we didn't have that AetherCompass used to
provide so figured it'd be nice to port it over since it's fairly
trivial.

Also adds support for marker icons that arent 32x32, it'll still default
to those values so it doesn't break any existing markers but they can be
overridden in the specific marker factory with IconWidth and IconHeight.
Had to implement this cause the icon that AC used (and by extension also
this) is the buff icon so it's 24x32.
<img width="185" height="135" alt="image"
src="https://github.com/user-attachments/assets/b282ff8f-1cf2-4137-b00c-b4bea76a2eb3"
/>
Adds support for the Elementals found in the various Eureka zones.
Was one of the only markers we didn't have that AetherCompass used to
provide so figured it'd be nice to port it over since it's fairly
trivial.

Also adds support for marker icons that arent 32x32, it'll still default
to those values so it doesn't break any existing markers but they can be
overridden in the specific marker factory with IconWidth and IconHeight.
Had to implement this cause the icon that AC used (and by extension also
this) is the buff icon so it's 24x32.
<img width="185" height="135" alt="image"
src="https://github.com/user-attachments/assets/b282ff8f-1cf2-4137-b00c-b4bea76a2eb3"
/>

@Bloodsoul Bloodsoul left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the nice work. I added some remarks.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fails for every marker without a set category.

"OccultCoffers",
"OccultSurveyPointMarkers"
]
};

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be really nice to have each marker put their own category in there. Maybe something like it is done with the toolbar widgets https://github.com/una-xiv/umbra/blob/main/Umbra/src/Toolbar/Widgets/Library/Accessibility/AccessibilityWidget.cs#L7
And if a Marker does not define a category, it goes in "General" automatically. This also solves the problem, that we cannot know what markers someone has installed via custom plugins.

Comment thread Umbra/i18n/en.json
"Markers.PartyMembers.Config.ShowName.Description": "Whether to show the party member names on the world markers.",
"Markers.PartyMembers.Config.IconType.Name": "Job icon type",
"Markers.PartyMembers.Config.IconType.Description": "The type of icon that is used on the world marker. Only applicable if 'Show party member icons' is enabled.",
"Markers.EurekaCoffers.Name": "Eureka Bunny Coffers",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't mind repeating the category in the names. It's also better to have them here when someone renames the categories in the future.

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.

3 participants