Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing Golden bow sprite glowing pixels #8542

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

shprotru
Copy link
Contributor

@shprotru shprotru commented Apr 4, 2024

fix #8520

after this fix:

vokoscreenNG-2024-04-06_00-08-20.mp4

@shprotru
Copy link
Contributor Author

shprotru commented Apr 4, 2024

Please, help me to find the place where the small artifact icons should be processed.

@zenseii zenseii added improvement New feature, request or improvement assets Game data related stuff pixel-precision Fine-tuning of UI elements and assets labels Apr 4, 2024
@zenseii zenseii added this to the 1.1.0 milestone Apr 4, 2024
@zenseii
Copy link
Collaborator

zenseii commented Apr 5, 2024

Hi, @shprotru. Could you please leave an image, preferably large, so that we can easily see the change done in this PR?

Copy link
Owner

@ihhub ihhub left a comment

Choose a reason for hiding this comment

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

Hi @shprotru , I left one suggestion. You also need to modify ICN::ARTFX and ICN::OBJNARTI resources.

@@ -3911,6 +3911,11 @@ namespace fheroes2
case ICN::ARTIFACT:
LoadOriginalICN( id );
if ( _icnVsSprite[id].size() > 99 ) {
// This fixes "Golden Bow" (#64) large artifact icon glowing yellow pixel
if ( _icnVsSprite[id][64].width() == 64 && _icnVsSprite[id][64].height() == 64 ) {
Copy link
Owner

Choose a reason for hiding this comment

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

This check is not needed as Copy function will do the same.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@shprotru
Copy link
Contributor Author

shprotru commented Apr 5, 2024

Hi @ihhub ! I added ICN::ARTFX, PR already updated and ready to merge. Where I can see ICN::OBJNARTI in the game? At the global map Golden Bow artifact was checked by @Branikolog and there is no any glowing pixels.

@shprotru shprotru requested a review from ihhub April 6, 2024 13:36
@ihhub ihhub modified the milestones: 1.1.0, 1.1.1 Apr 8, 2024
@ihhub ihhub modified the milestones: 1.1.1, 1.1.2 Jul 13, 2024
Copy link
Collaborator

@Districh-ru Districh-ru left a comment

Choose a reason for hiding this comment

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

Hi, @shprotru.
I've checked this fix and all is OK - the glowing pixels are properly replaced with the correct ones:

изображение изображение

Could you please, sync this PR with a master branch: we've added extra artifact image for Editor recently and used case ICN::ARTFX: in agg_image.cpp in the 4121 line.
And then updete this PR: the fix from current lines 3931-3934 should be moved the recently added case ICN::ARTFX: to properly merge this PR.

Co-authored-by: Sergei Ivanov (Districh) <113276641+Districh-ru@users.noreply.github.com>
@Districh-ru Districh-ru self-requested a review July 28, 2024 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assets Game data related stuff improvement New feature, request or improvement pixel-precision Fine-tuning of UI elements and assets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Golden bow sprite has glowing pixel
4 participants