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

Syntax Highlighting #39

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

Conversation

142ayushkumar
Copy link
Collaborator

codemirror configured

@142ayushkumar 142ayushkumar changed the title added codemirror in .gitignore & codemirror configured Syntax Highlighting Dec 19, 2017
@kaustubhhiware
Copy link
Owner

kaustubhhiware commented Dec 20, 2017

@142ayushkumar can you just pull from the master once?
That way, a review app will be auto-deployed.
This branch seems to be 4 commits behind master.

@142ayushkumar
Copy link
Collaborator Author

142ayushkumar commented Dec 20, 2017

@kaustubhhiware Done!

@kaustubhhiware
Copy link
Owner

kaustubhhiware commented Dec 20, 2017

@142ayushkumar You have made your master branch level with this repo's master.
What you need to do is make syntax-highlighting branch level with this repo's master.

To give a preliminary idea, heroku needs an app.json file (added yesterday) to auto-deploy PR's. That is present in your master, but not in your syntax-highlighting

@142ayushkumar
Copy link
Collaborator Author

@kaustubhhiware , i have merged my branched with master now!

@kaustubhhiware kaustubhhiware temporarily deployed to c0derunr-pr-39 December 20, 2017 13:46 Inactive
@kaustubhhiware
Copy link
Owner

@142ayushkumar Please refer to http://c0derunr-pr-39.herokuapp.com/

Seems code highlighting is not working.

@@ -91,6 +91,9 @@
}

</style>
<script src="/static/codemirror/lib/codemirror.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.

You have mentioned static/codemirror folder here. But it hasn't been added in your PR.

Copy link
Owner

Choose a reason for hiding this comment

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

image

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

But it is there locally 😕

Copy link
Owner

Choose a reason for hiding this comment

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

You have mentioned codemirror in .gitignore. Seems it's ignoring all codemirror folders.

I think you wanted to ignore ./codemirror. but gitignore makes it ignore ./static/codemirror as well. Check how you can fix this. Looks like an easy fix.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

if i remove codemirror from .gitignore there will be a lot of files then.

Copy link
Owner

Choose a reason for hiding this comment

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

You need to see how to add ./codemirror to .gitignore, but not ignore ./static/codemirror

It is possible to NOT ignore certain files / folders in .gitignore. What you've done is not wrong, it's just incomplete.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

./codemirror
!./static/codemirror
will do the work i guess

@kaustubhhiware kaustubhhiware temporarily deployed to c0derunr-pr-39 December 20, 2017 15:28 Inactive
@kaustubhhiware
Copy link
Owner

Redeployed app.
Highllighting works, but language switcher stopped working. 😕

@kaustubhhiware
Copy link
Owner

Using static codemirror seems to overload files.
Can you first use some online alternative?

Then we can try to debug where the switcher is failing.

@142ayushkumar
Copy link
Collaborator Author

I didn't noticed

@142ayushkumar
Copy link
Collaborator Author

Yes, language switcher stopped working, to change language now we have to refresh the page.
I will try some other alternatives.

@142ayushkumar
Copy link
Collaborator Author

@kaustubhhiware ,I configured ace for syntax highlighting but the same problem is still there, after configuring language switcher doesn't work.

@142ayushkumar
Copy link
Collaborator Author

@kaustubhhiware , i think something is wrong with language selector anyway, by chosing some language the template doesn't change like Java.

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.

2 participants