Skip to content
This repository has been archived by the owner on Nov 27, 2017. It is now read-only.

Fix #313 just check if there's middle components, if not then we show the popover #323

Merged
merged 2 commits into from Mar 27, 2017
Merged

Conversation

gashcrumb
Copy link
Contributor

This also reduces the height of steps so there's less vertical used by a flow, also tweaks some highlight behavior.

@kahboom
Copy link
Contributor

kahboom commented Mar 27, 2017

@gashcrumb - Are your tests passing locally? They seem to have failed on CircleCI. :(

@gashcrumb
Copy link
Contributor Author

@kahboom lint once again :-)

Copy link
Contributor

@kahboom kahboom left a comment

Choose a reason for hiding this comment

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

👍

@@ -91,6 +91,7 @@
//----- Sidebar: Progress Line: Complete State ----------------->>

&.complete {
border: 3px solid $pf-blue-400;
Copy link
Contributor

Choose a reason for hiding this comment

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

Woo!

flex-wrap: nowrap;
min-height: 250px;
min-height: 150px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry about that, just wanted it to be spaced out more on a larger screen.. maybe we can address that later on.

@kahboom kahboom merged commit 1ad353b into syndesisio:master Mar 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants