Skip to content

The merge layer button looks too much like a download graphic. #1178

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.

Sign up for GitHub

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

Jump to bottom
Open
JasonHoku opened this issue May 26, 2025 · 0 comments
Open

The merge layer button looks too much like a download graphic. #1178

JasonHoku opened this issue May 26, 2025 · 0 comments

Comments

@JasonHoku
Copy link

JasonHoku commented May 26, 2025 •
edited
Loading

I and another user both didn't recognize it as merge.

It could be updated to be more intuitive and less confusing by using a more clear merge icon instead.

Image

Image

I tried for a bit to understand the code base deeply enough to submit a PR for this issue but I don't think I understand the frameworks well enough.

I found the classname for the icon here in \piskel\src\css\font-icon.css at line 62

.piskel-icon-merge:before {
  content: "\e609";
}

And found mention of the unicode symbol as a glyph in this file: \piskel\src\css\fonts\icomoon.svg

But that seems to be where my mission comes to an end.

Would you all accept a simple svg and html replacement PR?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@JasonHoku