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

Keyboard Shortcuts:Tab button in text area #34

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

Conversation

142ayushkumar
Copy link
Collaborator

@142ayushkumar 142ayushkumar commented Dec 14, 2017

Closes #15

@Demfier
Copy link
Collaborator

Demfier commented Dec 14, 2017

My my, what did you do change 336 files 😅

Try putting that js in init.html itself and see if it works

@Demfier
Copy link
Collaborator

Demfier commented Dec 14, 2017

Also, you still haven't given us the browser console error

@142ayushkumar
Copy link
Collaborator Author

@Demfier I didn't even opened 336 files i guess , i don't know how come i have made changes in so many files, but i tried too many times to make things work.
I have tried putting that in init.html too, it doesn't work.

@Demfier
Copy link
Collaborator

Demfier commented Dec 14, 2017

browser console error please!

@142ayushkumar
Copy link
Collaborator Author

image

@Demfier
Copy link
Collaborator

Demfier commented Dec 14, 2017

the $ you are using in the js isn't defined. Is jquery loading properly?

@142ayushkumar
Copy link
Collaborator Author

How can i check if jquery is loading properly or not?

@Demfier
Copy link
Collaborator

Demfier commented Dec 14, 2017 via email

@Demfier
Copy link
Collaborator

Demfier commented Dec 14, 2017 via email

@kaustubhhiware
Copy link
Owner

You need to add node_modules in .gitignore, that is what's causing the large number of file changes.

Unless you do that, it will be really difficult for us to review this.

@142ayushkumar
Copy link
Collaborator Author

image
I have loaded jQuery before js code

@142ayushkumar
Copy link
Collaborator Author

Need some help here.

@kaustubhhiware
Copy link
Owner

So as per what I spoke with @142ayushkumar , changes in existing files have been made in init.html and my.js


<head>
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.3/jquery.min.js"></script>
<script src="static/js/my.js"></script>
Copy link
Owner

Choose a reason for hiding this comment

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

Need to write /static/js/my.js

See this: Another django project

@kaustubhhiware
Copy link
Owner

@142ayushkumar Could you please explicitly write what you are trying to solve with this PR?

Also, did you test it locally? Do not submit PR's which do not work locally / before testing.

Please mention which all shortcuts you have thought of, and which are implemented in this PR.

I tried using it locally, does not seem to work.

@kaustubhhiware
Copy link
Owner

It will help if you can mention all possible shortcuts you can think of. That way, you'll stay motivated if and when you work on them.

@142ayushkumar
Copy link
Collaborator Author

142ayushkumar commented Dec 18, 2017

As the name suggests "Tab button in text area", i expected that when in text area tab button will be used it will give '\t'. Yes, i tested this locally and it didn't seemed to work in this project but when i tested the same code in some html and js files it works, see => http://jsfiddle.net/jz6J5/.
See #15 , demifier suggested to make a PR.

@142ayushkumar
Copy link
Collaborator Author

@Demfier and @kaustubhhiware , #39 takes care of this pr, by configuring codemirror tab button works as desired in this pr.

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

Successfully merging this pull request may close these issues.

3 participants