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.

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

notebook -- it is now broken on the iphone #3223

Closed
williamstein opened this issue May 16, 2008 · 5 comments
Closed

notebook -- it is now broken on the iphone #3223

williamstein opened this issue May 16, 2008 · 5 comments

Comments

@williamstein
Copy link
Contributor

Now I think left or right parenthesis sends carriage return. I think this likely has something to do with updating the keyboard support for the new safari browser.

Component: notebook

Issue created by migration from https://trac.sagemath.org/ticket/3223

@williamstein
Copy link
Contributor Author

Attachment: sage-3223.patch.gz

@williamstein
Copy link
Contributor Author

comment:1

I fixed this by disabling all keyboard shortcut handling on the iphone. This is a good idea, since the iphone does not have any of the keys needed to send any of the keyboard shortcuts, and it gets around the problem.

To referee this 3-line patch, just verify that clearly I didn't break anything, since all I did was add a special case to check for the iphone string in the browser UA tag, and only then disable keyboard shortcut handling.

@williamstein
Copy link
Contributor Author

comment:2

I'm moving this back to 3.2. It's harmless (famous last words), and I really think having iphone support working again is an extremely important bug fix.

@williamstein williamstein modified the milestones: sage-3.2.1, sage-3.2 Nov 17, 2008
@boothby
Copy link

boothby commented Nov 17, 2008

comment:3

+1

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Nov 18, 2008

comment:5

Merged in Sage 3.2.rc2

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Nov 18, 2008
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants