[Xrystal]

Originally Posted by Seerah You're missing some code here. Where does it actually (attempt to) hide/show the minimap? I only see "self", which I assume is referring to nUI_MinimapManager?

Sorry, nUI's code is so convoluted that I was trying to extract the important parts.

The minimap is parented to the MinimapManager. By default the Minimap is visible and what it used to do was hide the minimap and replace it with the focus frame.

I'm trying to create a minimised version of the effect but struggling at the moment.

The ADDON_LOADED event registers callbacks which include applyOptions and applyScale. applyOptions has another set of code that does the show/hide toggle of the minimap frame based on the focus attribute flag being true. Again frame is being used which has always been the manager. Minimap:Hide/Show has never been called directly so can't figure out why it would have changed recently. Unless of course no one has ever used the /focus while using nUI. I know it used to work but the last time I seriously played was back in Cataclysm days. I've rarely set foot into a dungeon or raid since then, but I am sure other's have, which is why I was thinking of a recent change for there to only be a report now, 2 years after the latest expansion with on approx 3,000 users.

if UnitExists( "focus" ) 
        then frame:Hide();
        else frame:Show();
    end

Aha .. I think I may have figured it out ..

One of the changes I made at the start of the expansion was to move the Minimap and not the MinimapCluster as per several discussions on the forum due to several error messages that occurred.

Before BFA

MinimapCluster:EnableMouse( false );
    MinimapCluster:SetParent( frame );
    MinimapCluster:SetFrameStrata( frame:GetFrameStrata() );
    MinimapCluster:SetFrameLevel( frame:GetFrameLevel()+1 );

What I forgot to do was do the equivalent to the Minimap frame. And yes, that resolved the issue.

Thanks for alerting me to that vital part I clearly missed in my earlier perusal rofl.