-
Notifications
You must be signed in to change notification settings - Fork 172
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
feat(connection actions): add context menu for connection actions #74
feat(connection actions): add context menu for connection actions #74
Conversation
Add the options menu for when the user clicks with the right mouse button in a connection. re diego3g#37
Can you apply the i18n messages as we merged #57? |
I'm on it! |
@natanaelsirqueira I'm fixing some problems with the i18n, so you can wait until i merge #73 so we'll have this part fixed. |
I was just about to comment that I've experienced some weird behavior with the translations. 😄 Thanks for letting me know. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job @natanaelsirqueira, i just left some suggestions to keep the code cleaner but feel free to discuss with me the points you think that are not necessary right now.
Also, if you could fix the conflicts after this, it would be great!
Add the options menu for when the user clicks with the right mouse button in a connection.
Connect/Disconnect
Refresh
Edit
Delete
fix #37