Skip to content

Conversation

@lwalter
Copy link

@lwalter lwalter commented Jan 28, 2026

Apologies for the missed code originally, the assignment of the indicator properties was in my git index but not pushed. This should be complete and functional now and verified with the following steps

./release.sh -g retail -m .pkgmeta-retail -d
  • Symlinked the resulting addon directories to my local WoW addons folder
$ ln -sf ~/_dev/Grid2/.release/Grid2 Grid2
$ ln -sf ~/_dev/Grid2/.release/Grid2Options Grid2Options
$ ln -sf ~/_dev/Grid2/.release/Grid2LDB Grid2LDB
  • Reloaded WoW and verified no Lua errors in Bugsack

Previous MR description for completeness:
Thanks for this addon and the continued support. While setting up my UI during prepatch, I noticed that OmniCC is not continuing to be supported for prepatch and Midnight releases. However one key feature of OmniCC was being able to modify the placement of cooldown text on particular frames using rules (to avoid impacting other frames). This PR introduces the functionality to move cooldown text using similar configuration values as the stack text, namely justification and x/y offsets.

I've tested this on Retail Version: 12.0.0.65560.

I'm happy to iterate as needed to get this merged.

Thanks!

OmniCC is not continuing to be supported for prepatch and Midnight
retail releases. As such it is valuable to be able to at least move the
cooldown text which OmniCC originally provided. This commit introduces
this functionality within Grid2.
@michaelnpsp
Copy link
Owner

I will eventually merge this pull, but i cannot do that just now, because i need some time to test and maybe add some extra code to make this compatible with this OmniCC alternative (from the same author than OmniCC):
tullaCTC

@lwalter
Copy link
Author

lwalter commented Jan 28, 2026

Let me know if you'd like me to uptake anything with tullaCTC. It appears rules have to be defined in Lua so as-is, tullaCTC would have to be shipped as a dependency in grid2. Unless tullaCTC supported custom rule filters as OmniCC did.

@michaelnpsp michaelnpsp merged commit bb7e037 into michaelnpsp:midnight Jan 28, 2026
@michaelnpsp
Copy link
Owner

michaelnpsp commented Jan 28, 2026

Finally i decided to merge without tullaCTC support, because blizzard is doing too many changes every few days, breaking tulallCTC things and i prefer to wait for a more stable enviroment, i already have too many things to fix in grid2, i dont want to add more noise to my issues.

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.

2 participants