diff --git a/core/src/org/labkey/core/admin/lookAndFeelProperties.jsp b/core/src/org/labkey/core/admin/lookAndFeelProperties.jsp
index 26b83bd08aa..eb13df924b5 100644
--- a/core/src/org/labkey/core/admin/lookAndFeelProperties.jsp
+++ b/core/src/org/labkey/core/admin/lookAndFeelProperties.jsp
@@ -164,7 +164,7 @@
FolderDisplayMode currentMode = laf.getFolderDisplayMode();
inherited = isInherited(laf.getFolderDisplayModeStored());
%>
- <%=inheritCheckbox(inherited, folderDisplayMode, "folder_always", "folder_admin")%>
+ <%=inheritCheckbox(inherited, folderDisplayMode, true, "folder_always", "folder_admin")%>
@@ -180,7 +180,7 @@
FolderDisplayMode currentMenuDisplayMode = laf.getApplicationMenuDisplayMode();
inherited = isInherited(laf.getApplicationMenuDisplayModeStored());
%>
- <%=inheritCheckbox(inherited, applicationMenuDisplayMode, "menu_always", "menu_admin")%>
+ <%=inheritCheckbox(inherited, applicationMenuDisplayMode, true, "menu_always", "menu_admin")%>
|
|
<% inherited = isInherited(laf.getSystemEmailAddressStored()); %>
- <%=inheritCheckbox(inherited, systemEmailAddress)%>
- <%=disabled(inherited)%>> |
+ <%=inheritCheckbox(inherited, systemEmailAddress, hasAdminOpsPerm)%>
+ > |
| Organization name (appears in notification emails sent by system) |
<% inherited = isInherited(laf.getCompanyNameStored()); %>
<%=inheritCheckbox(inherited, companyName)%>
- |
+ > |
| |
@@ -330,7 +330,7 @@
| Default display format for date-times<%=helpPopup("Date-time format", dateTimeFormatHelp, true)%> |
<% inherited = isInherited(laf.getDefaultDateTimeFormatStored()); %>
- <%=inheritCheckbox(inherited, defaultDateTimeFormat, "dateSelect", "timeSelect")%>
+ <%=inheritCheckbox(inherited, defaultDateTimeFormat, true, "dateSelect", "timeSelect")%>
<%
String dateTimeFormat = laf.getDefaultDateTimeFormat();
DateTimeFormat td = DateUtil.splitDateTimeFormat(dateTimeFormat);
@@ -426,7 +426,7 @@
| Alternative login page<%=helpPopup("Custom Login Page", customLoginHelp, true)%> |
<% inherited = isInherited(laf.getCustomLoginStored()); %>
- <%=inheritCheckbox(inherited, customLogin)%>
+ <%=inheritCheckbox(inherited, customLogin, hasAdminOpsPerm)%>
> |
@@ -569,10 +569,15 @@
private HtmlString inheritCheckbox(boolean inherited, Enum> e)
{
- return inheritCheckbox(inherited, e, e.name());
+ return inheritCheckbox(inherited, e, true);
}
- private HtmlString inheritCheckbox(boolean inherited, Enum> e, String... ids)
+ private HtmlString inheritCheckbox(boolean inherited, Enum> e, boolean enabled)
+ {
+ return inheritCheckbox(inherited, e, enabled, e.name());
+ }
+
+ private HtmlString inheritCheckbox(boolean inherited, Enum> e, boolean enabled, String... ids)
{
if (getContainer().isRoot())
return HtmlString.EMPTY_STRING;
@@ -593,6 +598,9 @@
if (inherited)
builder.append(" checked");
+ if (!enabled)
+ builder.append(" disabled");
+
return builder.append(HtmlString.unsafe(">")).getHtmlString();
}