Skip to content

Commit

Permalink
[wicket] don't use all-caps for key shortcuts (oxidecomputer#3693)
Browse files Browse the repository at this point in the history
We're somewhat inconsistent on casing for key shortcuts, e.g. "UP/DOWN"
in some places and "Up/Down" in others. Settle on a consistent case
style for key shortcuts.

I think all-upper-case like "CTRL-R CTRL-R" looks too much like
shouting, so choose title case instead.
  • Loading branch information
sunshowers authored Jul 18, 2023
1 parent 553846b commit f824d33
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions wicket/src/ui/panes/overview.rs
Original file line number Diff line number Diff line change
Expand Up @@ -228,9 +228,9 @@ impl InventoryView {
pub fn new() -> InventoryView {
InventoryView {
help: vec![
("Rack View", "<ESC>"),
("Switch Component", "<LEFT/RIGHT>"),
("Scroll", "<UP/DOWN>"),
("Rack View", "<Esc>"),
("Switch Component", "<Left/Right>"),
("Scroll", "<Up/Down>"),
("Ignition", "<I>"),
],
scroll_offsets: ALL_COMPONENT_IDS
Expand Down
6 changes: 3 additions & 3 deletions wicket/src/ui/panes/rack_setup.rs
Original file line number Diff line number Diff line change
Expand Up @@ -114,16 +114,16 @@ impl Default for RackSetupPane {
fn default() -> Self {
Self {
help: vec![
("Scroll", "<UP/DOWN>"),
("Scroll", "<Up/Down>"),
("Current Status Details", "<D>"),
],
rack_uninitialized_help: vec![
("Scroll", "<UP/DOWN>"),
("Scroll", "<Up/Down>"),
("Current Status Details", "<D>"),
("Start Rack Setup", "<Ctrl-K>"),
],
rack_initialized_help: vec![
("Scroll", "<UP/DOWN>"),
("Scroll", "<Up/Down>"),
("Current Status Details", "<D>"),
("Start Rack Reset", "<Ctrl-R Ctrl-R>"),
],
Expand Down

0 comments on commit f824d33

Please sign in to comment.