Conversation
| <g> | ||
| <title>Observer</title> | ||
| <circle fill="#ffff00" strokeWidth="5" stroke={stroke} r={iconSize * 0.4} cy={iconSize / 2} cx={iconSize / 2} fillOpacity="0.4" /> | ||
| <circle fill="#ffff00" strokeWidth="2.5" stroke={stroke} r={iconSize * 0.4} cy={iconSize / 2} cx={iconSize / 2} fillOpacity="0.3" /> |
There was a problem hiding this comment.
You have 2 for sentries. And since this is just the border, I think both of them should have the same value 2.
| <g> | ||
| <title>Sentry</title> | ||
| <circle fill="#0000ff" strokeWidth="5" stroke={stroke} r={iconSize * 0.4} cy={iconSize / 2} cx={iconSize / 2} fillOpacity="0.4" /> | ||
| <circle fill="#0000ff" strokeWidth="2" stroke={stroke} r={iconSize * 0.4} cy={iconSize / 2} cx={iconSize / 2} fillOpacity="0.3" /> |
There was a problem hiding this comment.
Can we make this a cyan color?
|
@howardchung I am not too keen on the player color. We already have player specific switches. I think the map should have just team color. |
|
|
Okay, different thickness it is. TIL. 😄 How about #0055FF? Can you also add tooltips to each of those circles containing information about who planted the ward, when, who destroyed it if any, when if any |
|
|
I'll try to merge this into #469 |
|
@howardchung What's the plan with moving forward on this one? |
|
Needs to be synced with @micaelbergeron 's merged PR. I think we can apply the larger ward radius. Not sure if we want to do the colors as well. |
|
Being done in #587 |
4 participants
Uh oh!
There was an error while loading. Please reload this page.