Browse Source

fix cursor issue

Michael 2 years ago
parent
commit
20fd0d8901

+ 1 - 1
client/adventureMap/AdventureMapInterface.cpp

@@ -175,7 +175,7 @@ void AdventureMapInterface::dim(Canvas & to)
 			Rect targetRect(0, 0, GH.screenDimensions().x, GH.screenDimensions().y);
 			ColorRGBA colorToFill(0, 0, 0, std::clamp<int>(backgroundDimLevel, 0, 255));
 			if(backgroundDimLevel > 0)
-				to.drawColor(targetRect, colorToFill);
+				to.drawColor(targetRect, colorToFill, false);
 			return;
 		}
 	}

+ 2 - 2
client/render/Canvas.cpp

@@ -166,11 +166,11 @@ void Canvas::drawText(const Point & position, const EFonts & font, const ColorRG
 	}
 }
 
-void Canvas::drawColor(const Rect & target, const ColorRGBA & color)
+void Canvas::drawColor(const Rect & target, const ColorRGBA & color, const bool replace)
 {
 	Rect realTarget = target + renderArea.topLeft();
 
-	CSDL_Ext::fillRect(surface, realTarget, CSDL_Ext::toSDL(color));
+	CSDL_Ext::fillRect(surface, realTarget, CSDL_Ext::toSDL(color), replace);
 }
 
 SDL_Surface * Canvas::getInternalSurface()

+ 2 - 2
client/render/Canvas.h

@@ -93,8 +93,8 @@ public:
 	/// renders multiple lines of text with specified parameters
 	void drawText(const Point & position, const EFonts & font, const ColorRGBA & colorDest, ETextAlignment alignment, const std::vector<std::string> & text );
 
-	/// fills selected area with solid color
-	void drawColor(const Rect & target, const ColorRGBA & color);
+	/// fills selected area with solid color, allows replacing or overdrawing
+	void drawColor(const Rect & target, const ColorRGBA & color, const bool replace = true);
 
 	/// Compatibility method. AVOID USAGE. To be removed once SDL abstraction layer is finished.
 	SDL_Surface * getInternalSurface();

+ 12 - 6
client/renderSDL/SDL_Extensions.cpp

@@ -799,15 +799,21 @@ void CSDL_Ext::fillSurface( SDL_Surface *dst, const SDL_Color & color )
 	fillRect(dst, allSurface, color);
 }
 
-void CSDL_Ext::fillRect( SDL_Surface *dst, const Rect & dstrect, const SDL_Color & color )
+void CSDL_Ext::fillRect( SDL_Surface *dst, const Rect & dstrect, const SDL_Color & color, const bool replace)
 {
 	SDL_Rect newRect = CSDL_Ext::toSDL(dstrect);
-
 	uint32_t sdlColor = SDL_MapRGBA(dst->format, color.r, color.g, color.b, color.a);
-	SDL_Surface * tmp = SDL_CreateRGBSurface(0, newRect.w, newRect.h, dst->format->BitsPerPixel, dst->format->Rmask, dst->format->Gmask, dst->format->Bmask, dst->format->Amask);
-	SDL_FillRect(tmp, NULL, sdlColor);
-	SDL_BlitSurface(tmp, NULL, dst, &newRect);
-	SDL_FreeSurface(tmp);
+
+	// replacing or overdrawing (relevant for transparency)
+	if(replace)
+		SDL_FillRect(dst, &newRect, sdlColor);
+	else
+	{
+		SDL_Surface * tmp = SDL_CreateRGBSurface(0, newRect.w, newRect.h, dst->format->BitsPerPixel, dst->format->Rmask, dst->format->Gmask, dst->format->Bmask, dst->format->Amask);
+		SDL_FillRect(tmp, NULL, sdlColor);
+		SDL_BlitSurface(tmp, NULL, dst, &newRect);
+		SDL_FreeSurface(tmp);
+	}
 }
 
 STRONG_INLINE static uint32_t mapColor(SDL_Surface * surface, SDL_Color color)

+ 1 - 1
client/renderSDL/SDL_Extensions.h

@@ -58,7 +58,7 @@ using TColorPutterAlpha = void (*)(uint8_t *&, const uint8_t &, const uint8_t &,
 	void blitSurface(SDL_Surface * src, SDL_Surface * dst, const Point & dest);
 
 	void fillSurface(SDL_Surface * dst, const SDL_Color & color);
-	void fillRect(SDL_Surface * dst, const Rect & dstrect, const SDL_Color & color);
+	void fillRect(SDL_Surface * dst, const Rect & dstrect, const SDL_Color & color, const bool replace = true);
 
 	void updateRect(SDL_Surface * surface, const Rect & rect);