Skip to content
This repository was archived by the owner on Dec 20, 2025. It is now read-only.

Add helptext for checkPrecondition expression#1783

Closed
riltsken wants to merge 11 commits intospinnaker:masterfrom
riltsken:add-helptext-for-expressions
Closed

Add helptext for checkPrecondition expression#1783
riltsken wants to merge 11 commits intospinnaker:masterfrom
riltsken:add-helptext-for-expressions

Conversation

@riltsken
Copy link
Copy Markdown
Contributor

@riltsken riltsken commented Dec 7, 2015

Added helptext which points to Spring Expression Language so that users understand they can make complex evaluations here.

@riltsken riltsken force-pushed the add-helptext-for-expressions branch from 8e1bbd6 to a79c14e Compare December 7, 2015 23:29
@riltsken
Copy link
Copy Markdown
Contributor Author

riltsken commented Dec 7, 2015

By the way I wanted to add a link to the popover, but the popovers currently disappear when losing focus on the popover trigger element and the fix is some non-trivial event handling which could have an undesired effect. When angular-ui 1.0.0 rolls out it might be nice to take advantage of this fix angular-ui/bootstrap#4419

@anotherchrisberry
Copy link
Copy Markdown
Contributor

LGTM - can you rebase against master?

We will keep an eye on ui-bootstrap and upgrade to 1.0 when it is released.

@riltsken riltsken force-pushed the add-helptext-for-expressions branch from a79c14e to 61ba264 Compare December 8, 2015 16:02
@riltsken
Copy link
Copy Markdown
Contributor Author

riltsken commented Dec 8, 2015

Sorry, messed up this branch. #1784

@riltsken riltsken closed this Dec 8, 2015
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.

2 participants