fheroes2 icon indicating copy to clipboard operation
fheroes2 copied to clipboard

Creature separation window, add more vertical spacing between the small number buttons and the "OKAY" and "CANCEL" buttons

Open Cheerfulbull opened this issue 1 year ago • 8 comments

Made to address issue #7925. This increases the spacing between the Okay / Cancel buttons and the small numbers. Screenshot from 2024-04-11 22-33-30 I understand that the editor is currently priority, so feel free to disregard this, but I thought this was a fairly easy issue, so might as well fix it.

Cheerfulbull avatar Apr 11 '24 10:04 Cheerfulbull

Hi, @Cheerfulbull ! I've checked the PR and have a few comments. First of all, fast split icons highlighting is grey now. Is this change intentional? Previously, it had a red glowing effect.

I also feel, that the space between fast split icons and buttons line is a bit too large. I suggest making it the same, as between those split icons and the text above them.

Not sure, if it's related to the PR, but frame decoration looks not well in this area. image

Branikolog avatar Apr 14 '24 10:04 Branikolog

If the window would be improved, then some more space between the arrows (up/down) and the Min/Max buttonw would be welcome because it's rather tight on a touch device.

Mr-Bajs avatar Apr 14 '24 11:04 Mr-Bajs

Hi, @Mr-Bajs

If the window would be improved, then some more space between the arrows (up/down) and the Min/Max buttonw would be welcome because it's rather tight on a touch device.

Indeed. But I believe it's out of the scope of this PR. I believe there could be created a separate set of issues, dedicated increasing the convenience using touch screens. We have dozens of places, where the original UI appears to be too small for touch devices. Some windows require even a complete rework in my opinion.

Branikolog avatar Apr 14 '24 12:04 Branikolog

Hello @Branikolog , I have fixed the padding to make it more even. I was unable to replicate the visual glitch you sent. As for they grey highlighting (It's gold now for some reason?) I'm not sure what's causing that.

Cheerfulbull avatar Apr 15 '24 06:04 Cheerfulbull

Also I don’t know why the MacOS check failed, seems to be a bug

Cheerfulbull avatar Apr 20 '24 22:04 Cheerfulbull

Also I don’t know why the MacOS check failed, seems to be a bug

It is fixed.

ihhub avatar Apr 21 '24 10:04 ihhub

Hi @Branikolog , please check these changes.

ihhub avatar Apr 30 '24 03:04 ihhub

Hi, @ihhub Unfortunately, I cannot download compiled build for this PR now. The list does not appear on the page. I made some testing 1-2 weeks ago and highlight colour was still an issue unless something was changed in the last couple of days. I believe red colour is more perceptible for this UI element.

Branikolog avatar Apr 30 '24 12:04 Branikolog

@Branikolog , now you will be able to download a build.

ihhub avatar May 05 '24 13:05 ihhub

Hi, @ihhub !

Yellow selection frame is still the case. It is supposed to be red.

Also, there're still tiny defects on a frame which is automatically stretched according to the required size. image

I suppose it is a flaw of our frame-generating algorithm which cuts decoration on the frame.

Branikolog avatar May 06 '24 12:05 Branikolog

Hi @Branikolog , I reverted back color changes.

@Cheerfulbull , can you please explain why you changed colors?

ihhub avatar May 08 '24 14:05 ihhub

That was a mistake, I thought that was one of the positioning / window size functions.

Cheerfulbull avatar May 09 '24 23:05 Cheerfulbull

@Cheerfulbull , thank you very much for the fix!

ihhub avatar May 10 '24 11:05 ihhub