Closed kghost closed 3 years ago
Hi mate!
Those are some really great changes. 👍 Thing is I'm actually currently in the process or re-writing everything into a combined UI package (Frames, actionbars, etc).
So I will kind of be discontinuing updating oUF_Lumen and focus on this new UI.
I will for sure use some elements that you added, like Masque support, (haven't really thought about adding that ever). oUF_Lumen was ok in terms of code but had some glaring flaws, like no Raid Frames (it had years ago, but had to drop it for lack of time to maintain it with the upcoming API changes).
So, I'll tell you what, like you say, it would be quite the effort to integrate everything into the upstream. I guess it would be better if you maintain your fork, and if you want you can even rename your fork into its own thing and publish it (if you wish so).
I can, and will probably had a last pass on the code before I "drop it", any PRs will be welcome anyway but it will be hard for me to maintain it since I won't be using it actively. But I will look at your changes and most probably incorporate the most obvious ones into oUF_Lumen.
Is that fair enough?
I used to work on the develop branch, and trying to improve this addon, but it turns out that my branch is diverging very quickly.
You can check my changes here.
There are a few major changes I committed:
aurawatchers
element, which is used to monitor hots applied by the player, very useful for resto druidAnd lots of minor changes:
Masque
to theme all iconsI'm using LuaFormatter to format the lua code
I want to push all of them, but it will be a very tedious work to push them one by one. If you prefer these changes, we can work together, and figure out how to cooperate.