Skip to content
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

Change icon used for add slider #167

Open
KohmeiK opened this issue Jul 27, 2023 · 3 comments
Open

Change icon used for add slider #167

KohmeiK opened this issue Jul 27, 2023 · 3 comments
Assignees
@jp3wil

Comments

@KohmeiK
Copy link
Member

KohmeiK commented Jul 27, 2023

Image

Discuss options from here
@KohmeiK KohmeiK added enhancement New feature or request frontend and removed enhancement New feature or request frontend labels Jul 27, 2023
@rkprad
Copy link
Collaborator

rkprad commented Jul 27, 2023

The top two options are the correct ones. SmartSelect_20230727_140858_GitHub.jpg

Given that the rest icons are angled, the first option makes sense.

@rkprad
Copy link
Collaborator

rkprad commented Jul 27, 2023

The last three are okay if pin-in-slot is available but PMKS+ does not analyze pin-in-slot.

@rkprad
Copy link
Collaborator

rkprad commented Jul 27, 2023

I felt that the remove ground and remove input (don't remember this now or not sure if this exists) can also have a negative sign similar to other +/- signs being used.
Similarly for delete joints and delete links too.

jp3wil added a commit that referenced this issue Nov 10, 2023
@jp3wil
Made changes to decimal places (#151) and updated slider icons (#167)
2490d15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jp3wil jp3wil

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rkprad @KohmeiK @jp3wil