[whimsy] branch master updated: fix role checkboxes and enable/disable for vote and discuss

[cl...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

clr pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 6e56d4c  fix role checkboxes and enable/disable for vote and discuss
6e56d4c is described below

commit 6e56d4c85aea1d308501b0760984b6caf27202e2
Author: Craig L Russell <cr...@oracle.com>
AuthorDate: Thu Feb 1 11:51:08 2018 -0800

    fix role checkboxes and enable/disable for vote and discuss
---
 www/project/icla/views/pages/invite.js.rb | 29 ++++++++++++++++++++---------
 1 file changed, 20 insertions(+), 9 deletions(-)

diff --git a/www/project/icla/views/pages/invite.js.rb b/www/project/icla/views/pages/invite.js.rb
index 7818e5f..49d3c46 100644
--- a/www/project/icla/views/pages/invite.js.rb
+++ b/www/project/icla/views/pages/invite.js.rb
@@ -177,21 +177,23 @@ class Invite < Vue
     if @showRoleFrame
       _div.form_check do
         _label do
-          _input type: :radio, name: :role, value: :committer,
+          _input type: :radio, name: :role, value: :committer, id: 'role_committer',
           onClick: -> {@role = :committer;
+            @disabled = false
             @subject = @subjectPhase + ' Invite ' + @iclaname +
               ' to become a committer for ' + @pmc
             @proposalText = 'I propose to invite ' + @iclaname +
               ' to become a committer.'
-            @voteProposalText = @proposalText + ' Here is my +1.'
+            @voteProposalText = @proposalText + "\nHere is my +1."
           }
           _span @phasePrefix +
             ' invite to become a committer'
         end
         _br
         _label do
-          _input type: :radio, name: :role, value: :pmc,
-          onClick: -> {@role = :pmc;
+          _input type: :radio, name: :role, value: :pmc, id: 'role_pmc',
+          onClick: -> {@role = :pmc
+            @disabled = false
             @subject = @subjectPhase + ' Invite ' + @iclaname +
               ' to become committer and ' + @pmcOrPPMC + ' member for ' + @pmc
             @proposalText = 'I propose to invite ' + @iclaname +
@@ -204,8 +206,9 @@ class Invite < Vue
         if @showDiscussFrame
           _br
           _label do
-            _input type: :radio, name: :role, value: :invite,
-            onClick: -> {@role = :invite;
+            _input type: :radio, name: :role, value: :invite, id: 'role_invite',
+            onClick: -> {@role = :invite
+              @disabled = false
               @subject = @subjectPhase + ' Invite ' + @iclaname +
               ' to submit an ICLA for ' + @pmc
               @proposalText = 'I propose to invite ' + @iclaname +
@@ -373,7 +376,13 @@ class Invite < Vue
   # field setters
   #
 
-  def setIclaName(event)
+  def resetCheckBoxes()
+    document.getElementById('role_pmc').checked = false if document.getElementById('role_pmc')
+    document.getElementById('role_committer').checked = false if document.getElementById('role_committer')
+    document.getElementById('role_invite').checked = false if document.getElementById('role_invite')
+  end
+
+def setIclaName(event)
     @iclaname = event.target.value
     self.checkValidity()
   end
@@ -410,7 +419,8 @@ class Invite < Vue
     @showVoteErrorMessage = false;
     @showNoticeErrorMessage = false;
     self.checkValidity()
-    @disabled = false;
+    @disabled = true;
+    self.resetCheckBoxes()
   end
 
   def setdiscussComment(event)
@@ -433,7 +443,8 @@ class Invite < Vue
     @showVoteErrorMessage = false;
     @showNoticeErrorMessage = false;
     self.checkValidity()
-    @disabled = false;
+    @disabled = true;
+    self.resetCheckBoxes()
   end
 
   def setvoteComment(event)

-- 
To stop receiving notification emails like this one, please contact
clr@apache.org.