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
HexEditor+LibGUI: Initial version of HexEditor, LibGUI bug fix #649
Conversation
a42a90e
to
3f1f18e
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.
Looking very cool, lemme just nitpick a little bit.
First off, please wrap commit messages at 72 characters.
@@ -113,6 +113,11 @@ int GMenu::realize_menu() | |||
request.menu.enabled = true; | |||
request.menu.checkable = false; | |||
request.menu.checked = false; | |||
|
|||
// no shortcut on submenu, make sure this is cleared out |
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.
This looks like a useful fix, but please make it in a separate commit.
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.
Got this broken out now.
Libraries/LibGUI/GFilePicker.cpp
Outdated
@@ -30,10 +30,10 @@ Optional<String> GFilePicker::get_open_filepath() | |||
|
|||
Optional<String> GFilePicker::get_save_filepath(const String& title, const String& extension) | |||
{ | |||
GFilePicker picker(Mode::Save, String::format("%s.%s", title.characters(), extension.characters())); | |||
auto picker = GFilePicker::construct(Mode::Save, String::format("%s.%s", title.characters(), extension.characters())); |
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.
This is already fixed in LibGUI, please rebase on top of master. :)
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.
Whoops, fixed now.
Applications/HexEditor/GHexEditor.h
Outdated
#include <LibDraw/TextAlignment.h> | ||
#include <LibGUI/GScrollableWidget.h> | ||
|
||
class GHexEditor : public GScrollableWidget { |
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.
The G
prefix is generally used for classes in LibGUI. Since this class is outside LibGUI, you can just call it HexEditor
or something like that instead.
I was encountering an entire system crash when the window server attempted to do something with the shortcut text on a submenu. This bug only seemed to appear when I had a lone submenu inside of a menu.
The very first release of the Hex Editor for Serenity.
3f1f18e
to
e07e446
Compare
Delightful 😄 |
Initial version of Hex Editor
I was encountering an entire system crash when the window server attempted to do something with the shortcut text on a submenu. This bug only seemed to appear when I had a lone submenu inside of a menu.
Also fixed a bug with the save dialog involving references.