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

Improvements #1

Merged

Conversation

RunDevelopment
Copy link

@atomize I made a few improvements to your branch. Please look over them and tell me if you found any mistakes.

A few of the changes:

  • Error messages are no longer highlighted.
  • The Loading... text is displayed again.
  • Re-added the data-src-loaded.

We generally don't commit package-lock.json, so I deleted it.

Copy link
Owner

@atomize atomize left a comment

Choose a reason for hiding this comment

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

@RunDevelopment hey this looks good, all the important parts are there! Thanks for cleaning up this stuff to fit with core better/sorry if it was a bit of a style mess (ternary operators, i know, i know, i overuse'em =) Your changes are much more stylistically in line with the prism core upon further study, and I see what you are going for now, so, noted! I suppose I could've studied you're preferred style a bit before just sticking my plugin code in there hehe. I've never peeked in to the Prism API/core build before this last week so this was fun and I am grateful for that -- so thanks 🙏

I'm stoked for this feature, thanks for all the back and forth making it happen. I wanted to implement it because it seems like one of those things that should've always been there. I hope if this is baked in to a release people use it!

@atomize atomize merged commit c759e49 into atomize:file-highlight+data-range Mar 16, 2019
@RunDevelopment RunDevelopment deleted the 1813-improvements branch March 16, 2019 22:47
atomize added a commit that referenced this pull request Mar 22, 2019
* add data-range to file-highlight

* Add data-range to file-highlight

* gulp?

* gulp?

* Improvements (#1)

* Removed package-lock

* Improved File highlight

* Tutorials from comments - first commit

* style.css messing up nested lists

* un-es6 Snarkdown

* more un-es6 snarkdown

* gulped
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