minj / foxtrick

FoxTrick is a browser extension for the Hattrick online football manager game, currently available for Firefox, Google Chrome, as well as for Opera.
https://www.foxtrick.org
GNU General Public License v3.0
71 stars 49 forks source link

Update event icons in MatchReportFormat #1286

Closed minj closed 9 years ago

minj commented 9 years ago

From: llramone_nonumber PostID: 16751289.23 To: Everyone Re: 16751289.1 Datetime: 2015-03-19 10:51 Message:

I cant see the Reorganize and Nerveous icons at Match Report, check minutes 90 and 83 on this match (5924940) Cache cleared, all the others icons are working fine, should I do something else?

Thanks!


FoxTrick 0.14.0.3 release mozilla - Stage: false - Skin: normal - en - ltr - 1280 x 773 - Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:36.0) Gecko/20100101 Firefox/36.0 - MacIntel

minj commented 9 years ago

current icons: https://github.com/minj/foxtrick/tree/master/content/resources/img/matches

minj commented 9 years ago

Event descriptions Icon definitions Event<->icon map

ratsrats commented 9 years ago

I sure like to support Foxtrick. Give me some time, I'll look in to the subject : )

ratsrats commented 9 years ago

"There are quite a few match events that could benefit from additional icons."

I could not make up for which match events new icons are to be made. From browsing the list img/matches, there appear to be a lot of different sizes for icons.

  1. Could you supply me with a list of events that need an icon?
  2. What are minimal and maximum measurements for the icons?
minj commented 9 years ago
    22: 'Incomplete lineup, players from neighborhood used',
    25: 'Regional derby',
    26: 'Neutral ground',
    27: 'Away is actually home',
    30: 'Spectators/venue - rain',
    31: 'Spectators/venue - cloudy',
    32: 'Spectators/venue - fair weather',
    33: 'Spectators/venue - sunny',
    45: 'Half time results',
    46: 'Hat-trick',
    60: 'Underestimation',
    63: 'Remove underestimation on break',
    64: 'Reorganize',
    66: 'Decrease underestimation on break (equal score)',
    67: 'Decrease underestimation on break (up by 1 goal)',
    69: 'Remove underestimation',
    72: 'Extra time decided',
    80: 'New captain',
    81: 'New set pieces taker',
    331: 'Tactic Type: Pressing',
    332: 'Tactic Type: Counter-attacks',
    333: 'Tactic Type: Attack in the middle',
    334: 'Tactic Type: Attack on wings',
    335: 'Tactic Type: Play creatively',
    336: 'Tactic Type: Long shots',
    343: 'Tactic: Attack in the middle used',
    344: 'Tactic: Attack on wings used',
    500: 'Both teams walkover',
    501: 'Home team walkover',
    502: 'Away team walkover',
    503: 'Both teams break game (2 players remaining)',
    504: 'Home team breaks game (2 players remaining)',
    505: 'Away team breaks game (2 players remaining)',
    700: 'Event-o-Matic: Taunt Opponent',
    701: 'Event-o-Matic: Praise Opponent',
    702: 'Event-o-Matic: Plead for Fan Support',
    703: 'Event-o-Matic: Build Positive Atmosphere',
    704: 'Event-o-Matic: Honour Club Legacy',

These are missing. I'm not sure each and every one of them needs an icon though. I'll leave that to your discretion. Any addition is good obviously.

As for the sizes: there are no standards there as usual :D I see 12x12, 14x14, 16x16, even 19x16. I'd say we should be going for 16x16. Or 20x16 if you need a wider icon.

ratsrats commented 9 years ago

Browsing through offered lists I encountered:

bruised: '/Img/Icons/bruised.gif',

I was not able to find this icon. There is an icon folder, in lower case. The uppercase Icon folder is not visible to me. I would like to see some icons before continuing.

The 'tacticType' icons will be the first to take on.

minj commented 9 years ago

FT icons are only where you see Foxtrick.resourcePath. Otherwise we reuse HT icons: https://www.hattrick.org/Img/Icons/bruised.gif ;)

2015-05-11 12:55 GMT+03:00, ratsrats notifications@github.com:

Browsing through offered lists I encountered:

bruised: '/Img/Icons/bruised.gif',

I was not able to find this icon. There is an icon folder, in lower case. The uppercase Icon folder is not visible to me. I would like to see some icons before continuing.

The 'tacticType' icons will be the first to take on.


Reply to this email directly or view it on GitHub: https://github.com/minj/foxtrick/issues/1286#issuecomment-100841070

ratsrats commented 9 years ago

aim aim used aow aow used

These are the first suggestions.

ratsrats commented 9 years ago

Suggestions for: aim aim used aow aow used

minj commented 9 years ago

Are you sure they need to be different? I'd prefer the ones corresponding to 'used' for both cases

ratsrats commented 9 years ago

That's all right ; )

ratsrats commented 9 years ago

here are some more:

play creatively counter attack

icon play creatively icon counter attack

ratsrats commented 9 years ago

here are some more:

counter attack 2 longshot

icon counter attack 2 icon longshot

ratsrats commented 9 years ago

here are some more:

pressing 20 px pressing 30 px pressing 1

icons pressing 20 and 30 px.

ratsrats commented 9 years ago

her is an other one:

magic wand 2

icon play creatively 2

ratsrats commented 9 years ago

here is an other one:

pressing 3

icon pressing 3

minj commented 9 years ago

It's a bit unclear to me what these are supposed to symbolize:

as for the muscle, I think it would be clearer if it was rotated ~90deg CW

ratsrats commented 9 years ago

counter attack 2

Before playing CA, you have to build a wall, to stop the attack : ) In Germany, building a wall or producing concrete is synonym for building a strong defense. In England one defender is called Ronny Concrete for the same reason.

pressing 5

pressing 5

minj commented 9 years ago

I wanted to nudge you a bit :smile:

I would prefer to have these icons for next version since this requires no code changes and should be failproof.

E. g. icons are definitely needed for events that effect ratings (like underestimation)

minj commented 9 years ago

@ratsrats If you don't have the time, just say so. It's fine.

ratsrats commented 9 years ago

@minj you must have fallen of the radar.

ratsrats commented 9 years ago

Would you be so kind to produce a list of icons that are due:

underestimation . .

size will be: 20x20 px

minj commented 9 years ago

I think these are most important:

25: 'Regional derby',
26: 'Neutral ground',
60: 'Underestimation',
63: 'Remove underestimation on break',
64: 'Reorganize',
66: 'Decrease underestimation on break (equal score)',
67: 'Decrease underestimation on break (up by 1 goal)',
69: 'Remove underestimation',

Those remove/decrease underestimation could be the same icon obviously

ratsrats commented 9 years ago

I will have a go at these.

ratsrats commented 9 years ago

reorganise _ 01 decrease underestimation _ 01 remove underestimation _ 01 underestimation _ 01 neutral ground _ 22 regional derby _ 22

ratsrats commented 9 years ago

from top to bottom:

reorganise decrease underestimation remove underestimation underestimation neutral ground regional derby

minj commented 9 years ago

Cool thanks, I'll include them in new release soon. I'll also add you to our contributor list as a designer

ratsrats commented 9 years ago

thank you, the pleasure is all mine.