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.

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

Fix #1227 Styler - Set default position for components #1228

Merged
merged 5 commits into from
Oct 28, 2016
Merged

Fix #1227 Styler - Set default position for components #1228

merged 5 commits into from
Oct 28, 2016

Conversation

offtherailz
Copy link
Member

This prevent the close event to be called.

@offtherailz
Fix geosolutions-it#1227 . Set default position for components
db7345e 
This prevent the close event to be called.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.765% when pulling db7345e on offtherailz:i1227 into 27a45b0 on geosolutions-it:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 79.77% when pulling 4f15935 on offtherailz:i1227 into 27a45b0 on geosolutions-it:master.

@offtherailz
Copy link
Member Author

It still has some problems after a combination of clicks. Let me check better.

@offtherailz offtherailz changed the title Fix #1227 . Set default position for components Fix #1227 . Styler - Set default position for components Oct 28, 2016
@offtherailz offtherailz changed the title Fix #1227 . Styler - Set default position for components Fix #1227 Styler - Set default position for components Oct 28, 2016
@offtherailz
Use version of react-color
4722fd6 
- fixes geosolutions-it#1227
- reduces react color size (geosolutions-it#1202)
- minor fixes to equal interval tool in styler
@offtherailz
Copy link
Member Author

Ok, now everything works in local, with the latest version of react-color

@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 79.77% when pulling 67b017d on offtherailz:i1227 into 27a45b0 on geosolutions-it:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 79.628% when pulling 8631f9f on offtherailz:i1227 into 27a45b0 on geosolutions-it:master.

@mbarto mbarto merged commit 593a5ef into geosolutions-it:master Oct 28, 2016
@offtherailz offtherailz deleted the i1227 branch March 16, 2017 16:34
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@offtherailz @coveralls @mbarto