Skip to content
This repository was archived by the owner on May 13, 2026. It is now read-only.

Add auto hide main bar with configurable delay - #142

Merged
Wishmaster117 merged 2 commits into
feature/ace3-migrationfrom
feature/ace3-Milestone-8
Apr 3, 2026
Merged

Add auto hide main bar with configurable delay#142
Wishmaster117 merged 2 commits into
feature/ace3-migrationfrom
feature/ace3-Milestone-8

Conversation

@Wishmaster117

Copy link
Copy Markdown
Owner

No description provided.

@Wishmaster117
Wishmaster117 merged commit 35580cd into feature/ace3-migration Apr 3, 2026
2 of 4 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 530aa32566

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread UI/MultiBotMainUI.lua
end)
end

MultiBot.RefreshMainBarAutoHideState()

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Resync auto-hide hotspot after external bar moves

The detector hotspot is only synchronized during initialization/drag paths, so it drifts when the main bar is repositioned by other code paths (for example restoreBoundFramePoints() on ADDON_LOADED and the RTSC saved-state restore path). With auto-hide enabled and a saved non-default position, the bar can hide while the hotspot remains at the old coordinates, making it appear unrecoverable until another refresh-triggering action occurs.

Useful? React with 👍 / 👎.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant