Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ScreenShot ( overlayApi.takeScreenShot ) feature and UXUI Update #27

Open
wants to merge 133 commits into
base: master
Choose a base branch
from

Conversation

lalafellsleep
Copy link

No description provided.

@lalafellsleep
Copy link
Author

lalafellsleep commented Jan 8, 2018

🤔 CODE

LogParseOverlay why fired event 'onOverlayDataUpdate' include 'CombatantData'?

* Type for LabelOverlay
* Detach OverlayType
* Merge MessageType Enum
* Fix Conflict
* No Changes
* USING XPATH RESX FIXER
* Change font to Microsoft NeoGothic
* Remove Data refresh interval?
* ScreenShot to Screenshot
* Modify Margin, Padding
* size unification
* Button Size, Style
* Removed Unused value, using header
* Release version fix
@hibiyasleep
Copy link
Owner

Logparse was intended to addition of MiniParse, so it can be edited to borrow miniparse-related code from miniparse classes.

@lalafellsleep
Copy link
Author

Ok understand, just curious.

quisquous added a commit to quisquous/OverlayPlugin that referenced this pull request Oct 8, 2022
Previously, OverlayPlugin supported two plugin ids,
77 for normal releases and 86 for preleases, accessible
by double clicking on the update button.

This logic should probably be cleaned up and removed but
this is a minimal patch to allow for a broader OverlayPlugin
release with minimal risk.

EQAditu suggested after some testing that the new OverlayPlugin
should use a different id to avoid some error cases.

If OverlayPlugin wants to handle prereleases in the future,
we can add this back in.
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.

3 participants