Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

feat(timepicker): add seconds support #3022

Closed
wants to merge 2 commits into from
Closed

feat(timepicker): add seconds support #3022

wants to merge 2 commits into from

Conversation

rdinicut
Copy link

No description provided.

@Zielun
Copy link

Zielun commented Feb 5, 2015

What's the status of this feature change? Any plans to release it?

@siebeneicher
Copy link

+1

@karianna karianna added this to the Backlog milestone Mar 3, 2015
@@ -27,7 +27,7 @@ exclude = [
// - Safari
// - PhantomJS
browsers = [
'Chrome'
Copy link
Contributor

Choose a reason for hiding this comment

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

Undo this change.

@wesleycho
Copy link
Contributor

The directive should default to the current behavior, but allow for optional overriding via a config object at the directive level.

In the directive's isolate scope, we could have

scope: {
  config: '=?'
}

And if it is present, use it to override the default options.

cgat added a commit to cgat/bootstrap that referenced this pull request Aug 25, 2015
rdinicut made a PR (angular-ui#3022)
for this, which was never merged. I've changed showSeconds to default to
false and merged with current master.

Conflicts:
	karma.conf.js
	src/timepicker/test/timepicker.spec.js
	src/timepicker/timepicker.js
	template/timepicker/timepicker.html
@Foxandxss
Copy link
Contributor

Let's continue in #4272

@Foxandxss Foxandxss closed this Oct 16, 2015
cgat added a commit to cgat/bootstrap that referenced this pull request Oct 25, 2015
rdinicut made a PR (angular-ui#3022)
for this, which was never merged. I've changed showSeconds to default to
false and merged with current master.
cgat added a commit to cgat/bootstrap that referenced this pull request Oct 25, 2015
rdinicut made a PR (angular-ui#3022)
for this, which was never merged. I've changed showSeconds to default to
false and merged with current master.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants