feat(time selector): Add option to make time selector required or not…

…, default to not
esarder · Feb 13, 2019 · d4dcef0 · d4dcef0
1 parent 821c409
commit d4dcef0
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/src/os/ui/data/descriptorprovider.js b/src/os/ui/data/descriptorprovider.js
@@ -76,7 +76,6 @@ os.ui.data.DescriptorProvider.prototype.removeDescriptor = function(descriptor,
 /**
  * Get the descriptors registered to this provider.
  * @return {!Array.<T>} The descriptors
- * @protected
  */
 os.ui.data.DescriptorProvider.prototype.getDescriptors = function() {
   var dm = os.dataManager;

diff --git a/src/os/ui/time/time.js b/src/os/ui/time/time.js
@@ -14,7 +14,8 @@ os.ui.time.timeDirective = function() {
     scope: {
       'hours': '=',
       'mins': '=',
-      'secs': '='
+      'secs': '=',
+      'isRequired': '=?'
     },
     templateUrl: os.ROOT + 'views/time/time.html',
     controllerAs: 'time'

diff --git a/views/time/time.html b/views/time/time.html
@@ -1,5 +1,5 @@
 <div class="form-inline">
-  <input class="form-control c-date-time__time-input" type="number" ng-model="hours" min="0" max="23" placeholder="h"> :
-  <input class="form-control c-date-time__time-input" type="number" ng-model="mins" min="0" max="59" placeholder="m"> :
-  <input class="form-control c-date-time__time-input" type="number" ng-model="secs" min="0" max="59" placeholder="s">
+  <input class="form-control c-date-time__time-input" type="number" ng-model="hours" ng-required="isRequired" min="0" max="23" placeholder="h"> :
+  <input class="form-control c-date-time__time-input" type="number" ng-model="mins" ng-required="isRequired" min="0" max="59" placeholder="m"> :
+  <input class="form-control c-date-time__time-input" type="number" ng-model="secs" ng-required="isRequired" min="0" max="59" placeholder="s">
 </div>