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

React UI update (#103, #104, #106, #107) #105

Merged
merged 4 commits into from
Feb 28, 2015
Merged

React UI update (#103, #104, #106, #107) #105

merged 4 commits into from
Feb 28, 2015

Conversation

jbalsas
Copy link
Contributor

@jbalsas jbalsas commented Feb 23, 2015

No description provided.

@ipeychev
Copy link
Contributor

Just started reviewing :)

:octocat: Sent from GH.

@jbalsas
Copy link
Contributor Author

jbalsas commented Feb 26, 2015

Hey Iliyan, I pushed the new commit with the positioning fixes. I've done some refactors to the mixins to reuse as much code as possible.

Let me know what you think!

@jbalsas
Copy link
Contributor Author

jbalsas commented Feb 27, 2015

Hey Iliyan, I've pushed one more commit with the implementation of most of the buttons. After this, we're just missing the TwitterQuote (#109) and the behaviour change for the Link button (#108).

I'll start working on those on monday, but I wanted to know if you like what's in here or if I should do any changes before going any further.

Thanks!

@jbalsas jbalsas changed the title Remove Bootstrap dependency React UI update (#103, #104, #106, #107) Feb 27, 2015
@ipeychev
Copy link
Contributor

Okay, Chema, great! I will process these for sure till Monday. Thanks 👍

@ipeychev
Copy link
Contributor

Just started reviewing :)

:octocat: Sent from GH.

@ipeychev ipeychev merged commit bfa60af into liferay:master Feb 28, 2015
@ipeychev
Copy link
Contributor

Thank you, pull request merged!

:octocat: Sent from GH.

@jbalsas jbalsas deleted the update_skin_react branch March 6, 2015 10:50
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