-
Notifications
You must be signed in to change notification settings - Fork 829
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
React Octicons! #222
Merged
Merged
React Octicons! #222
Changes from 1 commit
Commits
Show all changes
99 commits
Select commit
Hold shift + click to select a range
542d2f6
First commit on a react-octicons component
jonrohan 2cb57f7
Renaming for npmjs
jonrohan b5a9afd
Updating sheild location
jonrohan 0cccc4f
remove clasee
jonrohan a0d68d5
building babel dist
jonrohan d7728f2
Outputting to build instead
jonrohan 36e3e4b
Setting up linting
jonrohan 581a4f2
github-lint?
jonrohan 60d21a7
1.1.0
jonrohan 9d94633
1.1.0
jonrohan 8f120ac
Be explicit so that it's not mixed up with index.scss
jonrohan 20170b6
Making the sizing stricter
jonrohan 11933d8
Large
jonrohan b41121c
Adding alignment options
jonrohan 0071790
Merge branch 'master' into react_octicons
jonrohan ba46dc4
Removing lint from test
jonrohan c72f20b
Renaming
jonrohan a8a7a07
Updating react package to build components
jonrohan 571e219
Revert package.json
jonrohan 1b30538
Cleaning lint
jonrohan d2e5c24
Don't need this in files
jonrohan f962884
Importing separately for all vs specific octicons
jonrohan f0b872f
Adding named.js to the package files
jonrohan cfdb19f
slimmer icons, bulkier <Octicon>
shawnbot 88e952f
UpperCase all the things
shawnbot 661d320
export all the icons???
shawnbot eb64b31
Merge remote-tracking branch 'origin/master' into reocticons
shawnbot 3a8157f
icon output fix, viewBox
shawnbot 6f811ae
add parcel for testing
shawnbot 98826f5
ignore stuff
shawnbot 6a3c8ec
clarify
shawnbot 4ecbf0e
use next.js
shawnbot 635b444
make a table
shawnbot 32a0078
add pages to files list
shawnbot 5c07b32
reasonable FIGMA_DOMAIN default; bail on missing FIGMA_TOKEN
shawnbot e030c2c
remove unneeded rubocop:disable
shawnbot 2cf602c
tidy
shawnbot 85a6fe2
add icons/all endpoint for named icons
shawnbot f587796
clean up deps, add jest
shawnbot 62487aa
add basic tests
shawnbot e7bdde9
add .babelrc that works with jest and next
shawnbot 8404d60
add getIcon(name) example to demo app
shawnbot 4d6dc42
start fleshing out new docs
shawnbot c86a792
add "small" size
shawnbot 5adf596
add iconsByName object export to icons/all
shawnbot 59e2c22
moar docs
shawnbot 7df502a
add tree-shaking link
shawnbot 7ab2d07
tidy up icon docs
shawnbot 39df26a
updating props, examples
shawnbot aa3e559
refactor all icons exports
shawnbot 0386dc2
update docs for new all icons exports
shawnbot 5173a54
nix "ES6"
shawnbot 5f23980
update the "about" bit
shawnbot 91e7dfe
"Octicons for React" sounds better
shawnbot c996c55
linty
shawnbot 4666318
add bit on custom icons, tweak Octicon language
shawnbot de30402
better explain the icon usage options
shawnbot ff8c997
v0.1.0
shawnbot fae9547
move script/build.js => script/build
shawnbot 959f0b7
add comma in "GitHub, Inc."
shawnbot c1db936
update build script name; add preversion run-script
shawnbot 131bb28
move react into peer and dev dependencies
shawnbot 3e8f487
suggest node 8+
shawnbot 9267ee9
move script/build => script/build.js
shawnbot 4f4a9df
add eslint, tidy package.json, build scripts
shawnbot 44ee3c6
add eslint configs
shawnbot babe5c4
tidy up icon builds
shawnbot c3e4956
remove un-applied quotes rule
shawnbot 35029db
don't lint generated .js files
shawnbot e3c85df
eslint --fix
shawnbot 6b9c1f9
add lint script
shawnbot 0bec8ae
lint before testing
shawnbot 970a58c
lint tests
shawnbot b57bf2d
doh
shawnbot f78cf21
add snapshot to Octicon test, hush console.error()
shawnbot e494f3d
add aria-hidden="true", user-select: none
shawnbot ea74679
try rollup?
shawnbot 7115814
update version to match
shawnbot 8351762
tidy up, improve Octicon tests
shawnbot 02a007a
don't set 'module' in package.json
shawnbot 4fce348
do sizing right, moar tests
shawnbot 10a3731
move everything into src/
shawnbot 5be4bd2
update rollup config
shawnbot c068058
write icons to src/icons
shawnbot e8e8929
remove lib/octicons_react/named.js from /.gitignore
shawnbot 63ffd9d
wrapping
shawnbot fc73084
React (not React.js)
shawnbot d80832b
add stage-0, rename next env to "production"
shawnbot d23a00c
use .npmignore instead of files; downgrade next, install stage-0
shawnbot 9d8aca0
refactor rollup config, output ESM + UMD
shawnbot ebbe36b
ignore dist/
shawnbot 8269535
show all the octicons!
shawnbot db11ac7
tidy up rollup config, use env preset
shawnbot 7c9d61d
nix es2015 preset (always use env!)
shawnbot b8f4a03
oops
shawnbot 34a532e
simplify rollup config
shawnbot b014412
nix prepare run-script
shawnbot a008e25
use some primer components
shawnbot b61e03c
clean up build run-script logic, remove pages/ from .npmignore
shawnbot File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,9 +5,11 @@ | |
"homepage": "https://octicons.github.com", | ||
"author": "GitHub, Inc.", | ||
"license": "MIT", | ||
"main": "index.js", | ||
"main": "build/index.js", | ||
"module": "index.js", | ||
"repository": "primer/octicons", | ||
"files": [ | ||
"build", | ||
"icons", | ||
"index.js", | ||
"pages" | ||
|
@@ -20,7 +22,8 @@ | |
"test": "jest", | ||
"start": "next", | ||
"lint": "eslint index.js __tests__ script", | ||
"build": "script/build.js", | ||
"prebuild": "script/build.js", | ||
"build": "rollup -c rollup.config.js", | ||
"preversion": "npm run build -s", | ||
"publish": "../../script/notify success", | ||
"prepublishOnly": "../../script/notify pending", | ||
|
@@ -49,7 +52,10 @@ | |
"octicons": "7.2.0", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note: |
||
"react": "^16.4.0", | ||
"react-dom": "^16.4.1", | ||
"react-test-renderer": "^16.4.1" | ||
"react-test-renderer": "^16.4.1", | ||
"rollup": "^0.62.0", | ||
"rollup-plugin-babel": "^3.0.5", | ||
"rollup-plugin-commonjs": "^9.1.3" | ||
}, | ||
"peerDependencies": { | ||
"react": ">=15" | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
import babel from 'rollup-plugin-babel' | ||
import commonjs from 'rollup-plugin-commonjs' | ||
|
||
export default { | ||
input: 'index.js', | ||
plugins: [ | ||
babel({ | ||
babelrc: false, | ||
exclude: 'node_modules/**', | ||
presets: [['env', {modules: false}], 'react'] | ||
}), | ||
commonjs() | ||
], | ||
output: { | ||
file: 'build/index.js', | ||
format: 'esm' | ||
} | ||
} |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was wrong for me to include. I removed it in 02a007a.