-
-
Notifications
You must be signed in to change notification settings - Fork 385
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
#8924 fix Hero Movement button #9085
base: master
Are you sure you want to change the base?
#8924 fix Hero Movement button #9085
Conversation
to delete hero path always
Despite the fact that it technically works, it doesn't look very elegant. First of all, this button is disabled, that is, from the user's point of view, it cannot be pressed. Also, since it is disabled, there is no press and release animation. |
Hi @oleg-derevenetz. In this case, I see two options:
Could you describe how you see the solution to this problem? |
I'd prefer to add a special state for this case, just like it's done for the "visit the object under the hero" action, but we need some artwork for this, which would somehow graphically indicate the possibility of canceling the current hero route. |
Do we need to wait for the icon design for this feature? Or is it possible to use some existing image? |
I am not aware of any existing image with the suitable style and meaning, so we need to come up with and draw it first. |
Do you have any instructions on how to do this? Or is it possible to ask for help from a certain person (interface designer)? |
I see two options here:
I think the choice of option depends on the desired button image (it will also need to be thought up). |
Hi @ihhub @zenseii @Districh-ru @Branikolog what do you think about the image? Any ideas? |
@oleg-derevenetz and @AlexSnowLeo. The concept in the picture is pretty much what I had in mind. Since we are making a pressable replacement for the disabled state of the button, in other words when there are no movement points left, maybe something like a horse that has been tethered to a short pole, to symbolize having parked for the day. |
Unfortunately, I am far from an artist. Maybe something like this? I will also attach the source code in the format paint.net. |
@AlexSnowLeo and @oleg-derevenetz , let's wait for @Districh-ru being available. He generated the original icon and he could suggest how to do it for the new one. |
We are also not talented artists, so making changes that require new art is usually problematic. But in this case, I'm afraid we can't do without it if we want everything to look nice. |
Hi, @AlexSnowLeo, I've made the hero action button image by simply taking an action cursor and placing it on the button image with a small corrections. |
Hi @Districh-ru. Yes, I have already considered this place in the code and tried to draw something suitable from the existing pictures of the game. But unfortunately there is no suitable picture (perhaps the image of the hero on a horse is suitable, but it is too colorful). Maybe there is not only another way to go? Make a mask of the image of a standing horse. And then apply it in the code along with the shadow on the button. The problem for me so far is only in shadow rendering. And let's still choose a concept, is this the one that @zenseii suggested - a horse standing on a leash? |
Hi, @AlexSnowLeo, sorry for the late reply. Hi, @Branikolog, what do you think about this image for the cancellation of move path? |
@Branikolog, @Districh-ru and @AlexSnowLeo. I have asked @FlorianBaba for his assistance/advice since he is a skilled pixel artist. @AlexSnowLeo's graphic looks like what I imagined, but I realized it looks a bit too crammed and perhaps too detailed to easily understand what it represents at a distance. I believe with some fine-tuning we can improve it though this is outside of my abilities. |
Hello there 👋 Indeed, I would love to participate. I'll keep you posted 👍 |
Hi everyone 👋 I've spent some time thinking about a suitable icon, trying to respect the original style. Just to share you my approach, I first tried @zenseii idea, which I thought was a good one. But I came to the same conclusion : i.e. that the icon is visually overloaded and not easily understandable. Then I went back to @AlexSnowLeo original idea, using the destination cross of the route taken by the heroes on the map. I tried to make the cross more prominent by using a horse's head icon. |
I'll repeat some of the comments I wrote to @FlorianBaba in DM here. I think the icon D which is a development of @AlexSnowLeo's idea is the best because it keeps the same concept as the other two states of the button, which are the prancing and walking icons, and you can see the entire horse on those too. The idea of removing the trail before the cross is better imo because it couldn't be interpreted as representing a walkable path, due to the monochrome nature of these icons which some might interpret as referring to the green arrows. It might give an impression of inconsistency if we change the icon to just show the head. As for the head being filled with one color, this might be seen as inconsistent with the general style of these icons which apart from the horse, have very thin lines outlining them and some amount of detail drawn within the object, like the knight's head having facial features etc. The exceptions are the horse and the view world button, however they are generally smaller icons than the horse head icons. Meanwhile, on @FlorianBaba's Discord server there was a discussion about this and some good ideas were given. One was to have a horse standing still and with the head bent down. In my opinion this would fit in well with the logical group: prancing; walking; standing still. |
We talked it over with @zenseii and thought it would be interesting to add a new suggestion, the one he mentioned in his previous message. It's up to you to choose now ^^ |
Hello everyone! I personally prefer D from an implementation quality state. Not perfect one, though. Logically, I expect something like @AlexSnowLeo proposed. Just a standing horse. We already have galloping and dancing icons, so new standing one feels appropriate. I'm only concerning about the horse design. Needs to be more polished and better proportioned. I also not sure about cross or ! mark. It looks too tiny. I prefer an icon being more simple: without multiple small details. I personally thought about placing just a cross, which is used for path destination, on this button : It can look simple and really easy to distinguish from other icons. But the standing horse feels kind of uniform design. 👍 It just needs to be more proportional, and players should always be able to easily distinguish it from the other two button states, especially the galloping one. |
Very nice! This is the idea I had. The reason to have the head bent is to distinguish it more from the galloping horse. I have some small comments. The horse looks a bit rounder around the stomach so maybe we could make it leaner? I believe there's something about the ears we can adjust. I'm not an equine expert but should their ears point down when they bend their head down? The tail looks like it has become a bit shorter, but I'm not sure there's enough space to make something that distinguishes it more? @Branikolog, I also considered having only the cross, but I believe it will look incoherent if the button switches states between Galloping horse, Prancing horse and a cross. This is why I'm in favor of the horse standing still. |
@zenseii I've tried to take your feedback into account for the E version. So :
Your feedback on the ears made me smile, because I had asked myself the same question as you the first time. |
I agree with this. The only thing I was concerning about is that two icons with a horse standing horizontally could be not distinguishable especially on tiny mobile screens. We should come up with a design which allow player to easily notice and understand the difference between these button states.
I have few suggestions on polishing this design even more, but I'm still hesitating about the concept. :) Since we work on hero path cancelling, just standing still horse logically fits more in my opinion. So, how do you guys think of placing just standing horse for this icon? Or we can go even further and draw horse during "braking" process :). |
Yes, I understand this and your work is very appreciated 👍It looks better now. I'm not a horse expert myself either but I try to give meaningful feedback.
You're right and I checked myself now too. To me it appears like the ears are always at a ~perpendicular angle to the head like I've shown in the images below. Could we try to make the horse icon's ear angle a bit less acute? I'm asking this because for some reason to me the head looks sharp like a wolf's head. After looking at several images of horses grazing, I noticed that they always stand with one of the front legs a bit ahead of the other like I'm showing in the bottom picture, probably to be able to bend the head lower. Another thing I think we could improve is to have a less deep curve of the horse's back as it seems a bit too much, not as much as a camel but slightly too much. I'm referring to the triangle shown in the bottom picture. This is what I'm observing from several pictures and not just that one.
I can see what you mean here. A horse breaking to me brings to mind the action you do when you interrupt a horse from moving along a path, which you do by pressing any button. I'm also sensing that we are not entirely on the same track regarding the functioning of the button. With the implementation of this PR, you will have to press this new button for 3 seconds to cancel the path. A single short press of that button will do nothing until we figure out some meaningful thing for that to do. Perhaps I misunderstood.
I had not thought about this, but I believe a sleep button would be better if it takes a completely different motive than a horse since the sleep function is something quite different from continuing a path or cancelling a path. However, your idea about sleeping and trying to differentiate this icon as much as possible made me think about having an icon of a horse laying down: This would distinguish the states even further and indicate to the player that the horse/army is now out of movement points. However, for now this is a digression and I would still much more like to see a grazing horse than this. |
Hello again 👋 @zenseii and I have had a lot of discussions on the subject, and we've come to the conclusion that since the button will have 3 versions, it's better that the three illustrations look the same and that you can see the continuity when moving from one to the other. And a gif version to show you the consistency with the other versions of the button : |
about the last concept, I would like to add that to me it seems better that a button that is supposed to reflect some sort of "inaction" and no response when the user presses the button, is done better with a passive/inactive horse, as opposed to the more active breaking horse or the icon that @LeHerosInconnu showed. In my opinion, a user would be more likely to be confused about why a button does nothing when the icon shows some sort of action, compared to a button that shows a horse standing still. |
@zenseii & @FlorianBaba |
@Districh-ru, I believe we should implement this new icon by generating it on the fly in the engine, rather than adding it to fheroes2 resources, because some parts are the same as the galloping horse. What do you think? Also feel free to give any feedback on the proposed icon. We should push the changes adding the new icon to this PR. |
I like the icon proposed by @FlorianBaba, thanks! a) We can add only the changed parts to the engine resources and make a new icon by taking galloping horse and "Blitting" the changed part (the unchanged pixels will be transparent) on it during the ICN load. The second approach needs more careful icon "drawing" in the code. ... and it will make out Hi, @ihhub, which approach do you prefer? |
Hi @zenseii and @Districh-ru , if the image difference is small and easy to generate then we can go with storing it in fheroes2 resources. If the image generation is easier to do then we can go with image generation approach. The size of the code is not an issue. |
@FlorianBaba Can you send me just a clean mask of a standing horse, without unnecessary details? I will try to encode this and apply a shadow using the game engine on the button substrate. (alex.v.timofeev@gmail.com) |
@AlexSnowLeo, @ihhub and @Districh-ru I believe the first thing to do is to check how much overlap there is between the standing and the galloping horse, perhaps make an image showing the non-common pixels on the galloping horse. This will help us understand what the best course of action is. Imo image generation shouldn't be hard since some parts like the head can just be copy-pasted, as we do on the fly. @AlexSnowLeo, the reason we are talking about just including the image differentials is due to copyright. The standing horse is derived from the galloping one and therefore we can't just take the entire standing horse and supply it with the engine. |
@zenseii It is for this reason that I initially proposed to make only a modification of the current image. For example, add only the cross symbol of the path: |
Yes, no problem, I'll send you an e-mail later today 👍 EDIT: Done 😉 |
@zenseii, @ihhub and @Districh-ru |
Close this issue: #8924
Fixed check on Hero Movement button to delete hero path always regardless of the button status.