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

Type and docs highlighting #9

Merged
merged 3 commits into from
May 23, 2015
Merged

Type and docs highlighting #9

merged 3 commits into from
May 23, 2015

Conversation

archaeron
Copy link
Member

Adds highligting to the type and documentation views

features:

  • highlighting comes from the idris compiler
  • easy to add to a new view
  • testable

Before:
without-highlighting

After:
with-highlighting

@cnd
Copy link
Member

cnd commented May 23, 2015

when I open file from context menu with atom it doesn't load and first time I used to get:

Uncaught TypeError: Cannot read property 'stdin' of undefined
/C:/Users/Rainbow/.atom/packages/language-idris/lib/idris-model.coffee:136

but mostly just blank file, if I reopen editor file loads fine and says that there is outdated .ibc format
but if I close and reopen idris file I see blank pane, it doesn't load.

so when there is outdated .ibc file near it doesn't load idr file, seems like

@archaeron
Copy link
Member Author

could it be that the idris code you where working with had errors in it? I think there is an issue when you work on invalid code. The ide-mode seems to stop. We should start a new one when this happens. I think it's unrelated to this pull request though.

cnd pushed a commit that referenced this pull request May 23, 2015
@cnd cnd merged commit db7f254 into master May 23, 2015
@cnd
Copy link
Member

cnd commented May 23, 2015

@archaeron it's not related error but outdated .ibc but yes, that could be different issue

@archaeron
Copy link
Member Author

I've created an inssue here: #13

@archaeron archaeron deleted the type-and-docs-highlighting branch May 28, 2015 17:46
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