fieat(openalex): Simpify UX

dataesr · Jan 8, 2025 · b8941b3 · b8941b3
1 parent b41084c
commit b8941b3
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 3 deletions.
diff --git a/client/src/pages/openalex-affiliations/components/send-feedback-button.jsx b/client/src/pages/openalex-affiliations/components/send-feedback-button.jsx
@@ -93,6 +93,11 @@ export default function SendFeedbackButton({ className, corrections, resetCorrec
           <TextInput
             label="Please indicate your email. Only an encrypted version of your email will be public."
             onChange={(e) => setUserEmail(e.target.value)}
+            onKeyDown={(e) => {
+              if (e.key === 'Enter' && corrections.length > 0 && validEmail) {
+                sendFeedback();
+              }
+            }}
             required
             type="email"
           />

diff --git a/client/src/pages/openalex-affiliations/results/index.jsx b/client/src/pages/openalex-affiliations/results/index.jsx
@@ -612,12 +612,17 @@ export default function Affiliations() {
                             <Row verticalAlign="bottom">
                               <Col>
                                 <TextInput
-                                  messageType={rorMessageType}
+                                  hint='Enter a valid ROR id and "check" it with ROR API'
+                                  label="ROR"
                                   message={rorMessage}
+                                  messageType={rorMessageType}
                                   onChange={(e) => setRor(e.target.value)}
+                                  onKeyDown={(e) => {
+                                    if (e.key === 'Enter' && ror.length > 0) {
+                                      getCleanRor();
+                                    }
+                                  }}
                                   value={ror}
-                                  label="ROR"
-                                  hint='Enter a valid ROR id and "check" it with ROR API'
                                 />
                               </Col>
                               <Col md="3">

diff --git a/client/src/pages/openalex-affiliations/search.jsx b/client/src/pages/openalex-affiliations/search.jsx
@@ -26,6 +26,7 @@ const { VITE_APP_TAG_LIMIT } = import.meta.env;
 const START_YEAR = 2010;
 // Generate an array of objects with all years from START_YEAR
 const years = [...Array(new Date().getFullYear() - START_YEAR + 1).keys()]
+  .sort((a, b) => b - a)
   .map((year) => (year + START_YEAR).toString())
   .map((year) => ({ label: year, value: year }));