Skip to content

Conversation

@BeckerWdf
Copy link
Member

Fixes: #135

@BeckerWdf
Copy link
Member Author

This is the current version:
image

@jasmin261098: Would it be better to have the horizontal lines aligned to the pixel grid?

@Michael5601
Copy link
Contributor

Thank you for the new icon. I think it looks great already.

We will always have some lines that are not aligned to the pixel grid. I always try to reduce the amount of these as much a possible but we can't eliminate it every time.
In this case I think it is more important that the icon is centered in the view area as it is now, than to align it pixel perfect.
The only think you could do is move the lower part of the arrow one pixel down without changing the rest:

image

Other than that I approve the proposed version.

@BeckerWdf
Copy link
Member Author

We will always have some lines that are not aligned to the pixel grid. I always try to reduce the amount of these as much a possible but we can't eliminate it every time.

We did change the icon once again so that both lines are aligned to the pixel grid. See:
image

Can you pls. re-review?

@BeckerWdf BeckerWdf changed the title New Icon for "Undo" New Icon for "Undo" and "Redo" Nov 26, 2025
@Michael5601
Copy link
Contributor

Michael5601 commented Nov 26, 2025

Can you pls. re-review?

Very nice. I approve the icons.

@Michael5601
Copy link
Contributor

Also @DanaStg seems to not have signed the eca yet.

@BeckerWdf
Copy link
Member Author

Very nice. I approve the icons.

Yes I know. We are working on it...

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.

Dual-Tone-Icon-Pack: Platform Icon "undo_edit"

3 participants