Skip to content

Fix ChatTypingPanel and ChatActionLabel notifications not displaying Take 2#6009

Merged
kwvanderlinde merged 4 commits into
RPTools:developfrom
Baaaaaz:bug/6006-overlays-do-not-display
Jul 11, 2026
Merged

Fix ChatTypingPanel and ChatActionLabel notifications not displaying Take 2#6009
kwvanderlinde merged 4 commits into
RPTools:developfrom
Baaaaaz:bug/6006-overlays-do-not-display

Conversation

@Baaaaaz

@Baaaaaz Baaaaaz commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Requirements for Contributing a Bug Fix or Enhancement

fixes #6006
properly fixes #5907

Description of the Change

  1. Undoes changes made in Fix ChatTypingPanel and ChatActionLabel notifications not displaying #5999 for issue [Bug]: Typing Notifications not displaying properly. #5907.
  2. zoneRenderPanel is now always added at the back.
  3. Resurrects the chat action label (no one seemed to miss it however as it has been gone for a while and no-one complained about it!). Chat action label is now added above overlays (so it remains clickable if overlays are visible)
  4. If the chat action label is visible in full screen mode (where chat is not visible anyway), when exiting full screen it now checks if the chat window (the command panel) is already open and if so hide the chat action label.

Layer Order is now:

  1. PointerToolOverlay (TOP)
  2. JLabel (chat action)
  3. HTMLOverlayPanel
  4. ChatTypingNotification
  5. ZoneRenderer (BOTTOM)

Possible Drawbacks

Minor - clicking the chat action label when in full screen mode will open the chat window (if not already open) on exiting full screen mode.

Documentation Notes

n/a

Release Notes

  • Fixed an issue where chat notifications were not being displayed.

This change is Reviewable

@github-actions github-actions Bot added the bug label Jul 11, 2026
@kwvanderlinde kwvanderlinde added this pull request to the merge queue Jul 11, 2026
@kwvanderlinde kwvanderlinde moved this from Todo to To-Be-Merged in MapTool 1.19 Jul 11, 2026
Merged via the queue into RPTools:develop with commit cbc43ff Jul 11, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from To-Be-Merged to Merged in MapTool 1.19 Jul 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

[Bug]: Overlays do not display [Bug]: Typing Notifications not displaying properly.

2 participants