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

Coloured SCs #177

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

Coloured SCs #177

wants to merge 8 commits into from

Conversation

JorenC
Copy link
Contributor

@JorenC JorenC commented Mar 12, 2023

This is a work that will works towards this style - a balance between visibility and aesthetic.

It requires new SCs for all maps (although it will work for maps that don't have this).
image

  • There will be a layer ColorSCs. Inside it are objects called proSC (pro = province).
  • The client will be able to colour these according to the 'game rules'. If a map does not have this map, they will just be omitted and not coloured - the province will still be coloured.

We can use one of the Beta maps (e.g. Maharajah) which are already using this scheme. If successful, I'll update this SC with all the maps one-by-one.

Added the first test of "Beta" map. Let's see if this works
Elegant orders needs to reference a pattern in the SVG. Added it here to test.
- Changed White SCs to named SC layer
- Reduced size of units
- Changed some SC positions.
- Split the SC foreground in two layers.
- One layer is called ColorSCs
- Each object in this layer is called 'proSC' (pro = province). These objects can be called from the client and coloured.
- There is a layer above it called 'ColorSCs_foreground'.
- Objects in this layer are ornamental, making sure the 'shape' fits exactly.

I've added these to Maharajah and Spice Islands Beta's as well, to test before we push this. If it works, I'll need to change ALL the maps (but that's not a bad thing!)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant