shdwp / xivFaderPlugin

XIV Dalamud plugin to automatically hide UI elements.
15 stars 16 forks source link

Cleaned up mentions of Quest Log, implemented DutyList and Server Inf… #50

Closed siklr closed 2 years ago

siklr commented 2 years ago

This resolves issue #24 and removes the ignore option for QuestLog and renames QuestLog to DutyList to be consistent with in-game naming of the UI element.

siklr commented 2 years ago

The second commit was added due to _ToDoList's position being updated whenever one would queue for a duty or withdraw from a queue. It seems that those actions would overwrite the X value to the original value, but Y would remain as -9999. Since the code originally only checked for X, it would be in a state where the addon was visible when it wasn't.

If there is a better way to implement this, please feel free to change the code.

Aelexe commented 2 years ago

Thank you for the contribution siklr. I'll re-add the references to Quest Log as they break backwards compatibility with older configs without implementing a config migration.