RDN-1128
1

Calendar Checkboxes

Scott on 8/5/2015 8:26 PM

Scott on 9/13/2015 7:26 PM

%
(hrs)
Logged

0 (hrs)


 Description

Go to League > Calendar > New

For each checkbox on this page, there should be a <label></label> that surrounds them.  So the user can click on the entire next to the checkbox, instead of just the checkbox.

Skip Navigation Links
 

Scott Saturday, August 08, 2015 10:52 PM #

Please also do this for Edit Recurring Event as well.

 

Dharmesh Saturday, August 08, 2015 3:43 AM #

Sorry! I didn't get this issue correctly. 

yes I know reason of why we put labels around checkboxes and the text next to them. The reson for that is to make checkbox with label clickable. It means clicking on the label, we can turns the checkbox on/off. 

I changed and fixed it.

 

Scott Friday, August 07, 2015 8:10 PM #

This isn't corrent.  I guess I didn't explain it good enough. Figured you would understand.  Review below. The label needs to surround both the checkbox and the text next to the checkbox.  Do you know why we put labels around checkboxes and the text next to them? 

This

<label> + @Html.CheckBoxFor(x => x.IsPublicEvent, new { @class = "alignLeft", @onchange = "PublicEventChangeCalendar(this)" }) </label> + <small class="i"> + Shows event on the public @RDN.Library.Classes.Config.LibraryConfig.WebsiteShortName calendar. <span class="b"> + For + Bouts, Fundraisers etc... + </span> + </small>

 

Needs to be

 

<label>

+ @Html.CheckBoxFor(x => x.IsPublicEvent, new { @class = "alignLeft", @onchange = "PublicEventChangeCalendar(this)" })  + <small class="i"> + Shows event on the public @RDN.Library.Classes.Config.LibraryConfig.WebsiteShortName calendar. <span class="b"> + For + Bouts, Fundraisers etc... + </span> + </small>

</label>

 


Powered by BugNET 1.6.339.0