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

results panel layout refactor #1361

Merged
merged 1 commit into from
Dec 22, 2016

Conversation

Gnafu
Copy link
Contributor

@Gnafu Gnafu commented Dec 20, 2016

Changed the results panel style

@Gnafu Gnafu requested a review from mbarto December 20, 2016 11:46
@Gnafu Gnafu added this to the R52016 milestone Dec 20, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 80.205% when pulling 9a2403c on Gnafu:wip_resultspanel into 0113d24 on geosolutions-it:master.

/>
</div>
<div style={{
display: "flex",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this working on IE9?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we keep that for now?

@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 80.205% when pulling a782bc4 on Gnafu:wip_resultspanel into 0113d24 on geosolutions-it:master.

@offtherailz
Copy link
Member

Should we solve codacy issues before to merge?

@Gnafu
Copy link
Contributor Author

Gnafu commented Dec 21, 2016

@offtherailz yes, I'm working on it

@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.205% when pulling 0c0389d on Gnafu:wip_resultspanel into 9b926c1 on geosolutions-it:master.

@offtherailz offtherailz merged commit 03eb0c5 into geosolutions-it:master Dec 22, 2016
@Gnafu Gnafu deleted the wip_resultspanel branch January 9, 2017 09:45
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.

4 participants