-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Added load, delete and export features in the Types Widget #1158
Conversation
7b471ea
to
71f7295
Compare
What about adding syntax highlight in the text input? http://doc.qt.io/qt-5/qtwidgets-richtext-syntaxhighlighter-example.html http://doc.qt.io/qt-5/qtwidgets-widgets-codeeditor-example.html |
@XVilka |
@XVilka good point - I have added the issue here https://github.com/radareorg/cutter/issues/1159, Cutter already have a syntax highlighter 👍 |
I'll make the changes by weekend since I'll finish my exams till then. Thank you all for the reviews! |
71f7295
to
5d8c8ac
Compare
5d8c8ac
to
a2fd167
Compare
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.
Looks great to me. Still there are some things to improve as discussed privately, but this for the next time :)
Thanks! |
Closes #1071 , closes #1074 , closes #1076 .
Preview: