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.

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

graph editor: limit port label size and added on hover styles #186

Merged
merged 4 commits into from
Dec 11, 2024

Conversation

fde31
Copy link
Member

@fde31 fde31 commented Dec 11, 2024

Added styles and code to deal with long port names by clipping the label with an ellipsis and adding hover highlighting styles
See example screenshots attached

closes #173

Screenshot 2024-12-11 at 15 39 00 ee ex Screenshot 2024-12-11 at 15 39 04 Screenshot 2024-12-11 at 15 38 33

@fde31 fde31 requested a review from x37v December 11, 2024 15:39
@fde31 fde31 changed the title graph editor: limit port label size and added on hover styless graph editor: limit port label size and added on hover styles Dec 11, 2024
@fde31 fde31 force-pushed the fde/port_overflow branch from 3859054 to 5543e6c Compare December 11, 2024 15:46
Copy link
Contributor

@x37v x37v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great!

@fde31 fde31 merged commit 3fe91c4 into develop Dec 11, 2024
1 check passed
@fde31 fde31 deleted the fde/port_overflow branch December 11, 2024 19:53
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.

long port names overlap each other in graph
2 participants