ion icon indicating copy to clipboard operation
ion copied to clipboard

Disable checkboxes in eighth admin activity edit page

Open Laur04 opened this issue 4 years ago • 3 comments
trafficstars

Steps to reproduce

  1. Navigate to https://localhost:8080/eighth/admin/activities/edit/1 (after adding an activity in your dev environment)
  2. Uncheck the "restricted" box
  3. You are able to check the Freshman Allowed, Sophomores Allowed, Juniors Allowed, Seniors Allowed boxes

Expected behavior

When the "restricted" box is unchecked, the above boxes should be both grayed out (readonly) and have disabled=true

Actual behavior

The Freshman, Sophomore and Junior boxed are only grayed out; the Senior box doesn't change at all.

Laur04 avatar Dec 26 '20 03:12 Laur04

I will (attempt) to fix this issue

krishnans2006 avatar Jun 01 '21 23:06 krishnans2006

When I try to reproduce the issue, none of the boxes are grayed out. Do I need any users/prior restrictions before unchecking?

krishnans2006 avatar Jun 03 '21 00:06 krishnans2006

You shouldn't need anything like that. If you try clicking the checkboxes, you should see that the Freshmen, Sophomore and Junior boxes have a gray check, while the Senior checkbox turns orange. If you look at the source, you will see that the first three have disabled=True while the Senior box doesn't.

Laur04 avatar Jun 03 '21 01:06 Laur04