Skip to content

Commit

Permalink
Include colour names in highlight menu (#153).
Browse files Browse the repository at this point in the history
  • Loading branch information
solemnwarning committed Apr 9, 2022
1 parent 155477f commit 7b901b4
Showing 1 changed file with 15 additions and 1 deletion.
16 changes: 15 additions & 1 deletion src/Tab.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -993,7 +993,21 @@ void REHex::Tab::OnDataRightClick(wxCommandEvent &event)

for(int i = 0; i < Palette::NUM_HIGHLIGHT_COLOURS; ++i)
{
wxMenuItem *itm = new wxMenuItem(hlmenu, wxID_ANY, " ");
/* Hardcoded list of names for the highlight colours.
* This will need to be done better soon... but for now all the highlight
* colours used in each pallette are the same and we don't have any more
* specific names for them (#60).
*/
static const char *highlight_strings[] = {
"Red",
"Orange",
"Yellow",
"Green",
"Violet",
"Grey",
};

wxMenuItem *itm = new wxMenuItem(hlmenu, wxID_ANY, highlight_strings[i]);

wxColour bg_colour = active_palette->get_highlight_bg(i);

Expand Down

0 comments on commit 7b901b4

Please sign in to comment.