Fix ambiguity in spacetime edit modal form as to what fields were ena…

…bled for a given choice of permanent or temporary change
csmberkeley · Feb 5, 2020 · 0708e49 · 0708e49
1 parent be61572
commit 0708e49
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 4 deletions.
diff --git a/csm_web/frontend/src/components/MentorSection.js b/csm_web/frontend/src/components/MentorSection.js
@@ -331,6 +331,7 @@ class SpacetimeEditModal extends React.Component {
  type="text"
  name="location"
  value={location}
+ autoFocus
  />
  </label>
  {/* Would use a fieldset to be semantic, but Chrome has a bug where flexbox doesn't work for fieldset */}

diff --git a/csm_web/frontend/static/frontend/css/style.css b/csm_web/frontend/static/frontend/css/style.css
@@ -771,12 +771,21 @@ table tbody tr:last-child > td {
 }
 
 #spacetime-edit-form #date-of-change-fields input[type="radio"] {
- margin-right: 10px;
+ margin: 0 10px 0 0;
  -webkit-appearance: none;
  -moz-appearance: none;
- padding: 2px 8px;
+ padding: 0;
+ width: 16px;
+ height: 16px;
+ display: inline-block;
  border: 2px solid var(--csm-green);
- border-radius: 13px;
+ border-radius: 8px;
+ vertical-align: middle;
+}
+
+#spacetime-edit-form #date-of-change-fields input[type="radio"][value="true"] {
+ position: relative;
+ bottom: 1px;
 }
 
 #spacetime-edit-form #date-of-change-fields input[type="radio"]:checked {
@@ -823,7 +832,10 @@ table tbody tr:last-child > td {
  font-size: small;
 }
 
-
+#spacetime-edit-form select:disabled, #spacetime-edit-form input:disabled {
+ cursor: not-allowed;
+ background-color: #fafafa;
+}
 
 #spacetime-edit-form input[type="submit"] {
  margin: auto;