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

Add star-rating.js #974

Merged
merged 2 commits into from
Aug 14, 2018
Merged

Add star-rating.js #974

merged 2 commits into from
Aug 14, 2018

Conversation

pryley
Copy link
Contributor

@pryley pryley commented Aug 9, 2018

No description provided.

Copy link
Contributor

@pineapplemachine pineapplemachine left a comment

Choose a reason for hiding this comment

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

The source attribute for star-rating should be an array containing paths to both the JavaScript and the needed css. (ctrl+f "source": [ for prior examples in the data.json file.) Can you make this change? If you do that, it looks like this should be good to go.

data.js Outdated
tags: ["rating", "stars", "star-rating"],
description: "Transforms a SELECT element into a dynamic star rating control",
url: "https://geminilabs.github.io/star-rating.js/",
source: "https://raw.githubusercontent.com/geminilabs/star-rating.js/master/src/star-rating.js"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be an array of paths for both the needed JS and CSS.

Copy link
Contributor Author

@pryley pryley Aug 13, 2018

Choose a reason for hiding this comment

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

Done b893ac7

@pineapplemachine
Copy link
Contributor

The data.json entry looks good now. Thank you for contributing!

@pineapplemachine pineapplemachine merged commit 895562d into microjs:master Aug 14, 2018
pineapplemachine added a commit to pineapplemachine/microjs.com that referenced this pull request Feb 7, 2019
* make-7-feb-2019:
  Rebuild atom.xml and data-min.js
  Fix the build script progress bar
  Add gradstop
  Add easyrouter
  Add ShadowQuery
  Added Wirup, WirupExpress to the list.
  Updating info on lcavadas/Storage.js
  Change spck-embed source.
  Added tutorial.js library.
  add spck-embed.js
  Added vanilla-picker-mini.js
  Add Beedle to the list
  sanitize-xml.js
  js2txt.js
  format-xml.js
  Fix source location of DOMtastic (microjs#976)
  Add star-rating.js (microjs#974)
  Add almy.js (microjs#975)
  Add Excellent.js
  adding Holen js

# Conflicts:
#	lib/logger.js
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