Banners color editing #3868

Merged
ancientmarinerdev merged 5 commits from VanicGame/MineClone2:banner-colors into master 2023-09-01 00:27:10 +02:00
Contributor

Changing banner colors

Reason: I think the current colors of the banners do not match the palette of the surrounding world.
Solution: Muting Banner Tones

Testing

Check out the new colors on 12 color banners.

Attachments

Changed banner colors/Current banner appearance

UPDATE: Adjusted colors, see the third attachment

UPDATE: Changed the color of the white flag. Fixed a bug with different colors of patterns and banners, increased the contrast of folds. The last four attachments display all.

### Changing banner colors Reason: I think the current colors of the banners do not match the palette of the surrounding world. Solution: Muting Banner Tones ### Testing Check out the new colors on 12 color banners. ### Attachments Changed banner colors/Current banner appearance UPDATE: Adjusted colors, see the third attachment UPDATE: Changed the color of the white flag. Fixed a bug with different colors of patterns and banners, increased the contrast of folds. The last four attachments display all.
VanicGame added 1 commit 2023-08-02 13:01:58 +02:00
Member

looks good

looks good
Member

if we are to tune the colors based off the wool, I think this PR fails at some of them. mostly on these (cropped out of the image using an editor, then changed the banner colors):

выява

выява

what do you think?

if we are to tune the colors based off the wool, I think this PR fails at some of them. mostly on these (cropped out of the image using an editor, then changed the banner colors): ![выява](/attachments/7cf12497-44fc-43a6-8487-43a9dfb450d2) ![выява](/attachments/80c051f1-4319-4356-a881-dd4db7b4aec6) what do you think?
VanicGame added 1 commit 2023-08-02 20:38:29 +02:00
Author
Contributor

Corrected colors, updated description

Corrected colors, updated description
chmodsayshello added the
graphics
enhancement
labels 2023-08-04 17:06:15 +02:00
Member

Looks good! You've got my merge approval one tweaking the white banner

Looks good! You've got my merge approval one tweaking the white banner
VanicGame added 1 commit 2023-08-04 20:07:49 +02:00
Member

Thanks for adjusting the white banner, but as long as the result of the PR is that color palette of the banner's content is different than the ones the banners use, we'll have to mark it as WIP

Thanks for adjusting the white banner, but as long as the result of the PR is that color palette of the banner's content is different than the ones the banners use, we'll have to mark it as WIP
chmodsayshello changed title from Banners color editing to WIP: Banners color editing 2023-08-04 20:58:27 +02:00
VanicGame added 1 commit 2023-08-04 21:33:56 +02:00
chmodsayshello changed title from WIP: Banners color editing to Banners color editing 2023-08-04 21:38:42 +02:00
VanicGame added 1 commit 2023-08-04 22:00:04 +02:00
chmodsayshello approved these changes 2023-08-31 19:02:31 +02:00
ancientmarinerdev added this to the 0.85.0 - Fire and Stone milestone 2023-09-01 00:26:49 +02:00
ancientmarinerdev merged commit 7b764adbc0 into master 2023-09-01 00:27:10 +02:00

@VanicGame Thanks for the PR and responding to feedback :). I have squash merged this. @chmodsayshello thank you for reviewing this one.

@VanicGame Thanks for the PR and responding to feedback :). I have squash merged this. @chmodsayshello thank you for reviewing this one.
Sign in to join this conversation.
No reviewers
No project
No Assignees
5 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: VoxeLibre/VoxeLibre#3868
No description provided.