wow/OmicronFrames
1
0 Fork 1
Code Issues 8 Pull Requests Actions Packages Projects Releases Wiki Activity

Bundle of fixes #10

Merged
omicron merged 11 commits from Andreas/OmicronFrames:bundle-of-fixes into main 2026-10-29 09:10:53 +00:00
Conversation 0 Commits 11 Files Changed 9
+400 -115
Andreas commented 2026-10-29 09:00:52 +00:00
Contributor
Copy link
Copy source

It's your fixes, minus the debuff hack. And then it's my druid config.

It's your fixes, minus the debuff hack. And then it's my druid config.
Andreas added 11 commits 2026-10-29 09:00:52 +00:00
Enhance IconIndicator with tooltips, fix stacks/applications mixup abeac379f3
update config for TBC shaman play 2a24e64554
Fix type in AuraList:UpdateAura 991bc5d0fe 
Due to a typo AuraTrigger:UpdateAura always received nil as its first
(before) parameter. This parameter is currently unused so it did not
trigger an error.
Fixed typos and logic error in UnitFrame:Disable 14aeb2c43e 
This function is currently unused so these changes are not tested. The
Disable function now always hides the UnitFrame and calls
UnregisterUnitWatch with the correct parameters.
Add raid pets to the hardcoded configuration b473f7bf3d
Make UnitGroup trigger sort on UNIT_PET 575e829025
Change the sort algorithm to use UnitExist instead of frame:IsShown 80859c1e44 
When UNIT_PET fires, pet units will exist but the UnitWatch will not yet
have shown the frame.
Replace Trigger:SetState by Trigger:SetStateAndData b9f1bad81d 
Update the use of the old API in most triggers except AuraTrigger. This
means the code isn't in a working state at this commit.
Add multiple spell ids support to AuraTrigger 7f3269949f 
Before AuraTrigger only supported a single spell id but now multiple
spells can activate the same trigger.

Related changes:
- Change the glue between AuraList and AuraTrigger to match this new functionality.
- Update the default frame config
Fix font and status bars to use new API arguments 76851011fe
Update config with TBC druid settings 949e3686aa
Andreas requested review from omicron 2026-10-29 09:00:52 +00:00
omicron merged commit 949e3686aa into main 2026-10-29 09:10:53 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
omicron
Labels
Clear labels
Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Internals
Internals changes
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Medium
3
The priority is medium
Priority
Low
4
The priority is low
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
Status
Abandoned
3
Somebody has started to work on this but abandoned work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
omicron
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wow/OmicronFrames#10