ESOUI

ESOUI (https://www.esoui.com/forums/index.php)
-   AddOn Search/Requests (https://www.esoui.com/forums/forumdisplay.php?f=165)
-   -   Combat Reticle feature (https://www.esoui.com/forums/showthread.php?t=8307)

and 01/27/19 06:26 AM

Combat Reticle feature
 
Hi All,

I would like to to request an add-on related to the combat reticle.

I think it would be great to have the basic reticle (or an enhanced one, what Combat Reticle or Rainbow Reticle add-ons provide) changing colour based on what weapon set I am equipping (front/back).

I reckon it is not impossible to do and I think it might be useful.

Many thanks!

Baertram 01/27/19 06:52 AM

I'd rather ask you to ask the dev of the existing addons to add the feature instead of building another new addon then. Seems easier and faster to accomplish Imo.

and 01/27/19 07:27 AM

Quote:

Originally Posted by Baertram (Post 36951)
I'd rather ask you to ask the dev of the existing addons to add the feature instead of building another new addon then. Seems easier and faster to accomplish Imo.

I did, actually.

The original dev of Combat Reticle suggested to use the forums :)

I have also contacted the dev of Rainbow Reticle, waiting for an answer.

Baertram 01/27/19 09:36 AM

:D I'm using rainbow reticle. Will have a look if such feature would be hard to implement but I doubt it.
You'd need to check the active weapon pair, react on a change (event is available therefor) and change the color then. Settings to define for both weapon bars: the color for out of combat and in combat, reticle over enemy in combat or not, etc. The lam settings would most probably the most work :(

and 01/27/19 01:03 PM

Quote:

Originally Posted by Baertram (Post 36954)
:D I'm using rainbow reticle. Will have a look if such feature would be hard to implement but I doubt it.
You'd need to check the active weapon pair, react on a change (event is available therefor) and change the color then. Settings to define for both weapon bars: the color for out of combat and in combat, reticle over enemy in combat or not, etc. The lam settings would most probably the most work :(


Lovely :)

Both CR and RR are great, and I was honestly surprised that they didn't have this feature.
At the moment I managed to mofify a very old addon (SWI), showing a big coloured number based on the set I use, but I really don't know how to touch complex addons... :(

BTW, what are the "lam settings"?

Thanks!

Baertram 01/27/19 03:01 PM

Lam = LibAddonMenu-2.0 > Addon settings

Baertram 01/28/19 06:21 AM

Just checked Rainbow Reticle and there are no color settings so this makes no sense to implement this here, as the reticle will only change the color for friends, guildmates, etc.
Would be better to implement in CombatReticle then Iguess, which I'm not using. Will have a look at it.

and 01/28/19 02:42 PM

Would it be better to have a simple addon doing just that or does integrating it into CR is eaier?

Baertram 01/28/19 03:38 PM

If you have another addon which changes the color it will most-likely interfere with other reticle addons which change the color, so its harder to get them working if they are not in the same addon.
I've started to work on a CR change to add the settings for weapon pair colors (and noticed there are some global variables leaking from CR which might even break other addons by accident).

Don't expect anything soon as I need to change and test it ingame and teh servers were not available half of the day today. Next days I'm off for work so just wait please and as it is ready for a test I'll link you the version here.

and 01/29/19 11:48 AM

Quote:

Originally Posted by Baertram (Post 36962)
If you have another addon which changes the color it will most-likely interfere with other reticle addons which change the color, so its harder to get them working if they are not in the same addon.
I've started to work on a CR change to add the settings for weapon pair colors (and noticed there are some global variables leaking from CR which might even break other addons by accident).

Don't expect anything soon as I need to change and test it ingame and teh servers were not available half of the day today. Next days I'm off for work so just wait please and as it is ready for a test I'll link you the version here.

It makes perfect sense, thanks for the explanation :)

Soon? No worries, of course, you are being so helpful that I really can't have any expectations.
Actually, I have so many add-ons that I can be considered a solid benchmark :)

Drakanwulf 01/29/19 03:38 PM

Baertran, would you add me to that link list also, please? Thanks...

Baertram 02/03/19 09:14 AM

Here is the link for my CombatReticle beta version including different reticle color settings for weapon pair main and backup:
CombatReticle beta weapon pair colors

Please test it a lot and tell me if everything works, or not. Tell me how to rebuild any bugs in detail please so I can fix them. If we got a version which is stable and performant I'll release it as a patch.

I also changed the internal stuff so there is a global table CombatReticle now, and some other errors like global variables named "ctrl" or "settings" were removed/renamed as well to fix compatibility issues with other addons.

Libraries were removed as well so please be sure to load LibStub and LibAddonMenu-2.0 as standalone libraries!

Drakanwulf 02/03/19 04:13 PM

Downloaded for testing on Live and PTS. Thanks for the fix, Baertram.

The standalone libraries can be downloaded from these ESOUI locations:
https://www.esoui.com/downloads/info44-LibStub.html
https://www.esoui.com/downloads/info7-LibAddonMenu.html

and 02/03/19 05:08 PM

I managed to play a couple of hours in Cyro, it works like a charm and I haven't noticed any issue.
I plan to test it tomorrow in PvE too.

BTW, do you need any specific test to be run?

Jolly good job!!! Many thanks :)

Drakanwulf 02/03/19 06:50 PM

I downloaded fresh copies of LibStub and LAM from the ESO libraries listed above before starting my tests.

I should not have downloaded LAM because it has not been updated since 100023, or thereabouts; it has a naked LibStub embedded within it; and it has no DependsOn: to instruct the game to find and use the latest LibStub.

So I cannot test this update until LAM has been repaired to work with stand alone add-ons. Sorry... :(

Yes, I could have repaired LAM but, when I am testing, I test as an ignorant user who has no idea how the game or Minion, etc. works. I just follow the instructions I have been given. :)

Baertram 02/03/19 06:53 PM

Quote:

Originally Posted by Drakanwulf (Post 37014)
I downloaded fresh copies of LibStub and LAM from the ESO libraries listed above before starting my tests.

I should not have downloaded LAM because it has not been updated since 100023, or thereabouts; it has a naked LibStub embedded within it; and it has no DependsOn: to instruct the game to find and use the latest LibStub.

So I cannot test this update until LAM has been repaired to work with stand alone add-ons. Sorry... :(

Yes, I could have repaired LAM but, when I am testing, I test as an ignorant user who has no idea how the game or Minion, etc. works. I just follow the instructions I have been given. :)

You are kidding me, right?
LibAddonMenu uses LiBStub from the LiBAddonMenu-2.0.txt file and everything works fine this way since months. If this bothers you and "forbids the test for you" I'm seriusoly sorry. But ONLY that I've invested time to change CombatReticle to read such a nonsense here... Sorry to say but please stop this!

You know how to fix this on your own and I won't comment this anymore now. Test and enjoy or leave me alone.

Baertram 02/04/19 03:51 AM

Quote:

Originally Posted by and (Post 37013)
I managed to play a couple of hours in Cyro, it works like a charm and I haven't noticed any issue.
I plan to test it tomorrow in PvE too.

BTW, do you need any specific test to be run?

Jolly good job!!! Many thanks :)

Thanks for the test. Nope, nothing in particular to test. Just use it and see if everything works as designed (before + new weapon bar colors).
If you notice any setting not working or any copy unction or theme or whatever there was before, just tell me.

and 02/04/19 02:39 PM

Quote:

Originally Posted by Baertram (Post 37018)
Thanks for the test. Nope, nothing in particular to test. Just use it and see if everything works as designed (before + new weapon bar colors).
If you notice any setting not working or any copy unction or theme or whatever there was before, just tell me.

To be honest, I was not using Combat Reticle before; I started from scratch :)

I have changed the basic settings for everything, including Stealth, and I have tested it in either PvE and PvP.

So far, so good!

I have noticed that the overlay doesn't inherit the new colours, I think it is by design.
It would be great to have an option to change that, unless it is really complicated.

I'll keep testing :)

Drakanwulf 02/04/19 03:58 PM

Quote:

Originally Posted by Baertram (Post 37015)
You are kidding me, right?
LibAddonMenu uses LiBStub from the LiBAddonMenu-2.0.txt file and everything works fine this way since months. If this bothers you and "forbids the test for you" I'm seriusoly sorry. But ONLY that I've invested time to change CombatReticle to read such a nonsense here... Sorry to say but please stop this!

You know how to fix this on your own and I won't comment this anymore now. Test and enjoy or leave me alone.

I think you may be loading Add-ons with the Out-of-Date checkbox turned on. Try turning it off and see what happens.

And no, I wasn't kidding. Open the latest LAM folder in ESOUI and check the manifest.txt file. Then check the rest of the LAM folder's contents. I believe you will find that the embedded, naked, LibStub.lua file is an out-of-date, r4 version, rather than the current, standalone, r5, version that exists in ESOUI.

My point is, if I fix LAM for myself, my fixes do not get back to ESOUI because I am not the author and because I am not the author, my changes could work for me but not for everyone else who plays the game.

I do not wish to get into an argument with you, Baertram, because I respect you, your add-ons, and your inputs into this forum. To that end, I have copied my more critical remarks from this thread and re-posted them under the LAM 2.0 add-on Comments thread for its author to review and act upon.

Baertram 02/05/19 03:38 AM

Quote:

I think you may be loading Add-ons with the Out-of-Date checkbox turned on. Try turning it off and see what happens.
Yes, about 99% of the users do so. I got your point but I haven't asked you to test the libraries but my work on the addon.
The "user" AND was able to test it. I doubt anyone else won't be able because of an outdated library. Most of the "users" know what outdated addons mean and what libraries are, or standalone libraries. It has been working since months now.

Thanks for bringing your "windmill fight" to the correct forum thread/comments :-)


All times are GMT -6. The time now is 07:25 AM.

vBulletin © 2024, Jelsoft Enterprises Ltd
© 2014 - 2022 MMOUI