Skip to content

Improve tracker dock bgs#5383

Merged
leggettc18 merged 3 commits intoHarbourMasters:develop-blairfrom
Malkierian:improve-tracker-dock-bgs
Apr 12, 2025
Merged

Improve tracker dock bgs#5383
leggettc18 merged 3 commits intoHarbourMasters:develop-blairfrom
Malkierian:improve-tracker-dock-bgs

Conversation

@Malkierian
Copy link
Contributor

@Malkierian Malkierian commented Apr 10, 2025

Modifies the flow for applying opacity to docked tracker windows to apply only when they are docked to the "Main - Deck" dockspace, rather than each other.

Build Artifacts

@leggettc18
Copy link
Contributor

Looks good, but need to run clang-format

…hen they are docked to the "Main - Deck" dockspace.
@Malkierian Malkierian force-pushed the improve-tracker-dock-bgs branch from bdf8a33 to e9d6173 Compare April 11, 2025 17:55
@leggettc18
Copy link
Contributor

image
Is this new behavior? Doesn't seem quite right.

@Malkierian
Copy link
Contributor Author

Is that actually docked to the main viewport, or did you dock that while the menu was open?

@leggettc18
Copy link
Contributor

I don't believe the menu was open no. It also only seems to happen with the Check tracker.

@leggettc18
Copy link
Contributor

Actually Entrance tracker seems to do it too, just can't manually edit it's opacity so it's only partially see-through.

@leggettc18
Copy link
Contributor

Item Tracker is fine though, only difference I'm aware of is the Item Tracker doesn't have a close button.

@leggettc18
Copy link
Contributor

OK yeah forget what I said about Check Tracker, it happens with every window I've tried except for the Item Tracker

@Malkierian
Copy link
Contributor Author

What do you mean by "every window"?

@leggettc18
Copy link
Contributor

Every ImGui window that you can pop out and dock to the sides of the main gameplay window has this behavior. Check Tracker, Entrance Tracker, Save Editor, etc. It's most noticeable with the Check Tracker because you can make it fully transparent, but all those other windows do a similar echo of the game screen at the moment they are docked. I should also note this actually may not be new behavior, it also happened for me after downloading a different build for another PR.

@Malkierian
Copy link
Contributor Author

What you're seeing behind the check tracker currently is the last state of the framebuffer from the game before the game was limited to the new space calculated from the dock. The Check Tracker is the way it is because it can go transparent, but you shouldn't be seeing anything behind any window that doesn't have color customization that allows transparency.

@Malkierian
Copy link
Contributor Author

The reason the check tracker wasn't covered was because I didn't realize its calls to BeingFloatingWindows wasn't the same as the Item Tracker's.

@Malkierian
Copy link
Contributor Author

And, the reason the entrance tracker is doing it is because it defaults to a bit of transparency for some reason.

@leggettc18
Copy link
Contributor

It's faint, but I can see it behind other windows as well. They are normally semi-transparent when they are entirely inside the SoH window, so if they get docked they still have this semi-transparency. Looks like this isn't new behavior to this PR though, so I can just make an issue for this to be solved later.

image

@leggettc18
Copy link
Contributor

That being said, what is something I can do that wouldn't work before this PR that does work after it (or at least behaves differently)?

@Malkierian
Copy link
Contributor Author

That part of it really is outside the scope of this PR, the framebuffer issue has been longstanding for months.

@leggettc18
Copy link
Contributor

Right right, like I said I'll make an issue for that if there isn't already one, but what is something I can do that wouldn't work before this PR that does work after it (or at least behaves differently)? I think I just don't have a clear idea of what is actually different with this new flow.

@Malkierian
Copy link
Contributor Author

Before this, having the item tracker fully transparent in windowed mode wouldn't go opaque when docked, and would therefore have ghosting with the tooltips.
image

@Malkierian
Copy link
Contributor Author

Also, the item tracker has never had a close button in windowed mode that I can remember. Certainly didn't in 9.0.0 or 8.0.6

@leggettc18
Copy link
Contributor

Yeah it's never had a close button, I wasn't implying that it was incorrect for it not to have one.

Can you elaborate how to achieve that behavior with the ghosting? I'm running one of the other currently open PRs trying to get this to happen so I can confirm, but I can't seem to replicate the ghosting behavior at all.

@Malkierian
Copy link
Contributor Author

Set the item tracker window opacity to 0, dock it, and move the mouse around over the items.

@leggettc18
Copy link
Contributor

I downloaded the build of your Cosmetics Reset PR, extracted it to a clean directory, and this is the behavior I'm getting. I'm not trying to say you're wrong, but clearly I'm missing something here.

2025-04-11.15-36-24.mp4

@Malkierian
Copy link
Contributor Author

Oh, right, this modifies the routine so that when you dock tracker windows with themselves, they don't go opaque, but that the opaque clamping only happens when you dock the trackers with the main dockspace that changes the resolution of the game output.

@leggettc18
Copy link
Contributor

And by "tracker windows" are you meaning just the Item Tracker windows? Because as posted above, the Check Tracker remains transparent when docked to the main game window.

@Malkierian
Copy link
Contributor Author

Malkierian commented Apr 11, 2025

The reason the check tracker wasn't covered was because I didn't realize its calls to BeginFloatingWindows wasn't the same as the Item Tracker's.

@Malkierian
Copy link
Contributor Author

So this also brings the check tracker up to date with what the item tracker was doing with the last PR, as well as applying the same constraints this PR applies to the item tracker.

@leggettc18
Copy link
Contributor

Ok I think I get it now. Regardless these changes don't appear to cause any issues so I'll call it good lol.

@leggettc18 leggettc18 merged commit fc0cbbe into HarbourMasters:develop-blair Apr 12, 2025
6 checks passed
@Malkierian Malkierian deleted the improve-tracker-dock-bgs branch April 13, 2025 02:51
krazyjakee pushed a commit to krazyjakee/OOT that referenced this pull request Sep 6, 2025
* Modify the flow for applying opacity to docked tracker windows only when they are docked to the "Main - Deck" dockspace.

* Add opacity clamping for check tracker.
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.

2 participants