Banners color editing #3868
No reviewers
Labels
No Label
#P1 CRITICAL
#P2: HIGH
#P3: elevated
#P4 priority: medium
#P6: low
#Review
annoying
API
bug
code quality
combat
commands
compatibility
configurability
contribution inside
controls
core feature
creative mode
delayed for engine release
documentation
duplicate
enhancement
environment
feature request
gameplay
graphics
ground content conflict
GUI/HUD
help wanted
incomplete feature
invalid / won't fix
items
looking for contributor
mapgen
meta
mineclone2+
Minecraft >= 1.13
Minecraft >= 1.17
missing feature
mobile
mobs
mod support
model needed
multiplayer
Needs adoption
needs discussion
needs engine change
needs more information
needs research
nodes
non-mob entities
performance
player
possible close
redstone
release notes
schematics
Skyblock
sounds
Testing / Retest
tools
translation
unconfirmed
mcl5
mcla
Media missing
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: VoxeLibre/VoxeLibre#3868
Loading…
Reference in New Issue
No description provided.
Delete Branch "VanicGame/MineClone2:banner-colors"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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.
looks good
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?
Corrected colors, updated description
Looks good! You've got my merge approval one tweaking the white banner
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
Banners color editingto WIP: Banners color editingWIP: Banners color editingto Banners color editing@VanicGame Thanks for the PR and responding to feedback :). I have squash merged this. @chmodsayshello thank you for reviewing this one.