Skip to content

Styleguide Third Phase #2227

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 64 commits into
base: master
Choose a base branch
from
Open

Styleguide Third Phase #2227

wants to merge 64 commits into from

Conversation

Diokuz
Copy link
Contributor

@Diokuz Diokuz commented Jun 16, 2017 •
edited
Loading

Third phase

What was done

  1. Typos Typos #2191
  2. Currency [Ashes] Move Currency to utils #2206
  3. DateTime [Ashes] Move DateTime to utils #2228

Bugs

  1. Address mutation Fix prop mutation #2323

Diokuz and others added 25 commits June 16, 2017 11:43
@Diokuz
Add fonts and rework bell button
6f9114f
@Diokuz
Handle core fonts
0dce966
@Diokuz
rename colors.css to variables.css
059b651
@Diokuz
Add docs
53c4e6e
@Diokuz
replace postcss-assets with postcss-url
714cd8c
@Diokuz
Remove less from styleguide
ba00425
@Diokuz
cr fix
3af14de
@Bad-Company
feat: add datetime.jsx to Styleguide
fa1831d
@tonypizzicato
fix variables.css usage
7ba48b5
@tonypizzicato
Merge pull request #2191 from FoxComm/typos
0f936c9 
Typos
@Bad-Company @tonypizzicato
refactor: move Currency component to /utils
f6ec175
@Bad-Company @tonypizzicato
fix: change missed path's
a46d37b
@Bad-Company @tonypizzicato
refactor: add flow & replace less with css-modules
bcd40d4
@Bad-Company @tonypizzicato
refactor: remove _currency.less
d355e76
@Bad-Company @tonypizzicato
fix: flow
b64d29c
@Bad-Company @tonypizzicato
refactor: use Change component for transaction mode.
6e5faaa
@Bad-Company @tonypizzicato
chore: change prop description
8849ce1
@Bad-Company @tonypizzicato
feat: add currency to styleguide
40386c2
@Bad-Company @tonypizzicato
feat: add mocha-tests
acf85f8 
& remove old tests
@tonypizzicato
formatting and import fixes
ff77401
@Bad-Company
chore: remove duplicated test
26673da 
minor correction to test description
@tonypizzicato
Merge pull request #2206 from FoxComm/refactor/currency
8b47e33 
[Ashes] Move Currency to utils
@Bad-Company
feat: add mocha-test for datetime
04acd11
@Bad-Company
Merge branch 'feature/ashes-styleguide-phase-3'
a892c37 
Conflicts:
	ashes/src/components/gift-cards/gift-card.jsx
	ashes/src/components/rmas/helpers.jsx
	ashes/src/components/table/cell.jsx
	ashes/styleguide/config.styleguide.js
@Bad-Company
fix: paths after merge feature branch
2dfb084
tonypizzicato and others added 24 commits June 26, 2017 14:19
@tonypizzicato
Merge pull request #2228 from FoxComm/refactor/move-datetime-to-utils
844498e 
[Ashes] Move DateTime to utils
@Diokuz
Merge remote-tracking branch 'origin/master' into feature/ashes-style…
0ca11fb 
…guide-phase-3
@Diokuz
fix datetime path
51950cc
@Diokuz
Fix prop mutation
7b1dc0c
@tonypizzicato
Merge pull request #2323 from FoxComm/fix/1899-address-bug
94201f8 
Fix prop mutation
@tonypizzicato
styleguidist styles update
9f0c678
@tonypizzicato
update examples styles
1bbe7f2
@tonypizzicato
components styles fixes
4f0df20
@tonypizzicato
get rid of dev docs in styleguide
8cc530d
@tonypizzicato
cleanup fonts/colors examples
6f2addc
@tonypizzicato
experiments
f63c39a
@tonypizzicato
cleanup sidebar
6f1336e
@tonypizzicato
update sidebar styles
732bbd9
@tonypizzicato
expand/collapse sidebar
1ca7ef6
@tonypizzicato
sidebar animation
a32cdec
@tonypizzicato
fix heading styles
a65446d
@tonypizzicato
fix example code style
83467f5
@tonypizzicato
move scroll/expand logic to TableOfContents
23ad334
@tonypizzicato
cleanup
dc79d21
@tonypizzicato
update sg design
8790564
@tonypizzicato
add smoothscroll-polyfill
6955354
@tonypizzicato
Merge pull request #2290 from FoxComm/feature/styleguidist-ui
9e3218b 
[Ashes] Styleguidist UI updates
@tonypizzicato
update docs
140f390
@tonypizzicato
fix actions dd
142b1fd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Team: Admin [Tony] WIP

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Diokuz @Bad-Company @tonypizzicato