Skip to content

Added key mappings for [, ], , and \#3373

Merged
emilk merged 1 commit intoemilk:masterfrom
mkrueger:dev/mkrueger/key_codes
Jan 8, 2024
Merged

Added key mappings for [, ], , and \#3373
emilk merged 1 commit intoemilk:masterfrom
mkrueger:dev/mkrueger/key_codes

Conversation

@mkrueger
Copy link
Copy Markdown
Contributor

@mkrueger mkrueger commented Sep 23, 2023 •
edited
Loading

Added key mappings for '[', ']', ',' and ''.

Added keys

  • egui::Key::Backslash
  • egui::Key::OpenBracket
  • egui::Key::CloseBracket
  • egui::Key::Comma

@mkrueger mkrueger mentioned this pull request Sep 23, 2023
Closed
@mkrueger mkrueger mentioned this pull request Oct 9, 2023
Closed
emilk
emilk approved these changes Nov 10, 2023
View reviewed changes
@emilk
Copy link
Copy Markdown
Owner

emilk commented Nov 10, 2023

Have you tested this on web?

@mkrueger
Copy link
Copy Markdown
Contributor Author

mkrueger commented Nov 30, 2023

works on web for me.

@mkrueger
Copy link
Copy Markdown
Contributor Author

mkrueger commented Dec 1, 2023

rebased the PR on current HEAD - no conflicts.

@emilk
Copy link
Copy Markdown
Owner

emilk commented Jan 6, 2024

Thanks for the PR! Unfortunately this requires another rebase 😬

@mkrueger
Copy link
Copy Markdown
Contributor Author

mkrueger commented Jan 7, 2024 •
edited
Loading

there were many changes in that area :/ - comma got already added now - others are missing. There are surely plenty of keys that are currently not supported. I only added the ones I need for my terminal program.

emilk
emilk reviewed Jan 7, 2024
View reviewed changes
crates/egui/src/data/input.rs Outdated
CloseBracket,

/// '`'
Backquote,
Copy link
Copy Markdown
Owner

@emilk emilk Jan 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has many different names, but I suggest we follow wikipedia (https://en.wikipedia.org/wiki/Backtick) and call it Backtick, but add "backquote" and "grave" as pseudonyms in the docstring.

Copy link
Copy Markdown
Contributor Author

@mkrueger mkrueger Jan 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed - I don't mind how it's called doesn't matter if it's grave, backtick or backquote - I just need the key :).

@emilk emilk added eframe Relates to epi and eframe egui labels Jan 8, 2024
@emilk emilk changed the title Added key mappings for '[', ']', ',' and '\'. Added key mappings for [, ], , and \. Jan 8, 2024
@emilk
Copy link
Copy Markdown
Owner

emilk commented Jan 8, 2024 •
edited
Loading

cargo check --locked --all-features --all-targets fails

@mkrueger
Copy link
Copy Markdown
Contributor Author

mkrueger commented Jan 8, 2024

ok fixed - some more renamings. Assumed that a VirtualKeyCode -> KeyCode but they renamed some keys.

LBracket -> BracketLeft
RBracket -> BracketRight
Grave -> Backquote :)

@mkrueger
Added key mappings for '[', ']', '`' and '\'.
15db853 
* Closes #3372
* Added Key::Backslash, Key::OpenBracket, Key::CloseBracket, Key::Backquote
@emilk emilk merged commit 8b4c9fd into emilk:master Jan 8, 2024
@emilk emilk changed the title Added key mappings for [, ], , and \. Added key mappings for [, ], , and \ Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@emilk emilk emilk approved these changes

Assignees

No one assigned

Labels

eframe Relates to epi and eframe egui

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Need more key mappings

2 participants

@mkrueger @emilk