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

Y position of toolbar to render it, now we consider if the content has overflow customization to calculate the proper Y value #595

Merged
merged 1 commit into from
Oct 7, 2016

Conversation

antoniopol06
Copy link
Contributor

No description provided.

…s overflow customization to calculate the proper Y value

Fixes liferay#549
@ipeychev
Copy link
Contributor

ipeychev commented Oct 4, 2016

Just started reviewing :)

:octocat: Sent from GH.

@jbalsas
Copy link
Contributor

jbalsas commented Oct 5, 2016

Hey @ipeychev, could we release a new version after this gets merged?

Thanks!

@ipeychev
Copy link
Contributor

ipeychev commented Oct 6, 2016

Sure, probably this weekend.

Thanks,

@jbalsas
Copy link
Contributor

jbalsas commented Oct 6, 2016

Awesome, thanks!

We'll try to squeeze one more fix in, then 😉

@ipeychev ipeychev merged commit 8c77054 into liferay:master Oct 7, 2016
@ipeychev
Copy link
Contributor

ipeychev commented Oct 7, 2016

Thank you, pull request merged! See changes here.

:octocat: Sent from GH.

antoniopol06 pushed a commit to antoniopol06/alloy-editor that referenced this pull request Nov 14, 2016
ipeychev pushed a commit that referenced this pull request Nov 17, 2016
ipeychev pushed a commit that referenced this pull request Nov 17, 2016
ipeychev pushed a commit that referenced this pull request Nov 17, 2016
azotova pushed a commit to azotova/alloy-editor that referenced this pull request Jan 7, 2017
azotova pushed a commit to azotova/alloy-editor that referenced this pull request Jan 7, 2017
azotova pushed a commit to azotova/alloy-editor that referenced this pull request Jan 7, 2017
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.

3 participants