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

Added scaling to mobile and fixed a few build errors #176

Open
wants to merge 37 commits into
base: master
Choose a base branch
from

Conversation

Finkoy
Copy link

@Finkoy Finkoy commented May 6, 2018

There are a few known bugs that haven't been fixed since the original version. The code editor still does not work and will probably have to be rewritten in order to get the Android keyboard to pop up. The return key on mobile phones seem to be malfunctioning (not create an extra space sometimes and deletes characters). The design is also not yet perfect and a bit unintuitive, but is usable. Upon pressing the text editor, other divs will close. However, the user must touch away (lose focus from the editor) in order to make the divs reappear. The text editor still looks small due to the popup keyboard on phones, so more work will be done to make it look better. Users can compile their code and view the results in the terminal. Tabs allow users to go through the text prompt, terminal, and text editor. The navigation bar now collapses for mobile devices and text editor settings collapse into a button. Additionally, the project should have fewer build errors. Ignore the various delete commits and possibly redundant comments; most of the work was done in one big commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant