Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Page Up, Backspace, and other keys show up as special characters #8

Closed
ctrlcctrlv opened this issue Oct 20, 2017 · 3 comments
Closed

Page Up, Backspace, and other keys show up as special characters #8

ctrlcctrlv opened this issue Oct 20, 2017 · 3 comments
Labels
bug

Comments

@ctrlcctrlv
Copy link
Contributor

Backspace shows up as ć
Home shows up as Ć
Page up shows up as œ
Page down shows up as Œ
End shows up as Ũ

Weirdly, arrow keys and delete work.

2017-10-20-173225_swaygrab
@FedeDP FedeDP added the bug label Oct 20, 2017
@FedeDP
Copy link
Collaborator

FedeDP commented Oct 20, 2017 •
edited
Loading

SUPPORT:

  • PG_UP/DOWN in roster (go to first/last)
  • PG_UP/DOWN in Chat (scroll an entire page)
  • support home and end in Input

EDIT: btw backspace is working for me.

@ctrlcctrlv
Copy link
Contributor Author

ctrlcctrlv commented Oct 22, 2017 •
edited
Loading

Commit 9777707 strangely did not seem to work for me, I still get œ/Œ.

I know we're not using Python, but I feel this is relevant: https://stackoverflow.com/questions/22157426/getch-returns-wrong-characters

Nevermind, looking at the commit again, I see that it is for the roster window, not chat window :)

ctrlcctrlv added a commit to ctrlcctrlv/slack that referenced this issue Oct 22, 2017
@ctrlcctrlv
Fix most annoying part of ga2arch#8
41e640e 
Backspace works now on my system. I can't think of a good reason for us
to be defining our own KEY_BS rather than using ncurses' KEY_BACKSPACE -
so let's stop doing that!
@FedeDP FedeDP mentioned this issue Oct 22, 2017
Open
@FedeDP
Copy link
Collaborator

FedeDP commented Oct 22, 2017 •
edited
Loading

Waiting for your PR then closing this issue :)

EDIT: obviously let me know if you encounter any issue!

FedeDP added a commit that referenced this issue Oct 22, 2017
@FedeDP
Merge pull request #17 from ctrlcctrlv/master
8dcdef0 
Fix most annoying part of #8
@FedeDP FedeDP closed this as completed Oct 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ctrlcctrlv @FedeDP