improved date and radio selection logic

Signed-off-by: Vedansh Saini <[email protected]>

Author: vedansh-5

src/popup.html

@@ -64,11 +64,11 @@ <h4>Your Github Username</h4>
 						<p class="text-sm">Fetch your contributions between:</p>
 						<div class="flex justify-between items-center my-2 mx-4">
 							<div class="flex items-center gap-2">
-								<input type="radio" id="lastWeekContribution" name="timeframe" class="form-radio" onclick="toggleRadio(this)">
+								<input type="radio" id="lastWeekContribution" name="timeframe" class="form-radio">
 								<label for="lastWeekContribution">Last 7 days</label>
 							</div>
 							<div class="flex items-center gap-2">
-								<input type="radio" id="yesterdayContribution" name="timeframe" class="form-radio" onclick="toggleRadio(this)">
+								<input type="radio" id="yesterdayContribution" name="timeframe" class="form-radio">
 								<label for="yesterdayContribution">Last 1 day</label>
 							</div>
 						</div>

src/scripts/main.js

@@ -124,17 +124,17 @@ function handleLastWeekContributionChange() {
 	let value = lastWeekContributionElement.checked;
 	let labelElement = document.querySelector("label[for='lastWeekContribution']");
 	if (value) {
-			startingDateElement.disabled = true;
-			endingDateElement.disabled = true;
+			startingDateElement.readOnly = true;
+			endingDateElement.readOnly = true;
 			endingDateElement.value = getToday();
 			startingDateElement.value = getLastWeek();
 		        handleEndingDateChange();
 		        handleStartingDateChange();
 			labelElement.classList.add("selectedLabel");
 			labelElement.classList.remove("unselectedLabel");
 	} else {
-			startingDateElement.disabled = false;
-			endingDateElement.disabled = false;
+			startingDateElement.readOnly = false;
+			endingDateElement.readOnly = false;
 			labelElement.classList.add("unselectedLabel");
 			labelElement.classList.remove("selectedLabel");
 	}
@@ -147,17 +147,17 @@ function handleYesterdayContributionChange() {
 	let labelElement = document.querySelector("label[for='yesterdayContribution']");
 
 	if (value) {
-		startingDateElement.disabled = true;
-		endingDateElement.disabled = true;
+		startingDateElement.readOnly = true;
+		endingDateElement.readOnly = true;
 		endingDateElement.value = getToday();
 		startingDateElement.value = getYesterday();
 			handleEndingDateChange();
 			handleStartingDateChange();
 		labelElement.classList.add("selectedLabel");
 		labelElement.classList.remove("unselectedLabel");
 	} else {
-		startingDateElement.disabled = false;
-		endingDateElement.disabled = false;
+		startingDateElement.readOnly = false;
+		endingDateElement.readOnly = false;
 		labelElement.classList.add("unselectedLabel");
 		labelElement.classList.remove("selectedLabel");
 	}

src/scripts/popup.js

@@ -253,8 +253,8 @@ document.addEventListener('DOMContentLoaded', function() {
 
             const startDateInput = document.getElementById('startingDate');
             const endDateInput = document.getElementById('endingDate');
-            startDateInput.disabled = false;
-            endDateInput.disabled = false;
+            startDateInput.readOnly = false;
+            endDateInput.readOnly = false;
 
             chrome.storage.local.set({
                 lastWeekContribution: false,
@@ -292,7 +292,7 @@ document.addEventListener('DOMContentLoaded', function() {
                     endDateInput.value = getToday();
                 }
 
-                startDateInput.disabled = endDateInput.disabled = true;
+                startDateInput.readOnly = endDateInput.readOnly = true;
 
                 chrome.storage.local.set({
                     startingDate: startDateInput.value,
@@ -391,8 +391,8 @@ document.querySelectorAll('input[name="timeframe"]').forEach(radio => {
 
             const startDateInput = document.getElementById('startingDate');
             const endDateInput = document.getElementById('endingDate');
-            startDateInput.disabled = false;
-            endDateInput.disabled = false;
+            startDateInput.readOnly = false;
+            endDateInput.readOnly = false;
 
             chrome.storage.local.set({
                 lastWeekContribution: false,
@@ -464,7 +464,7 @@ function toggleRadio(radio) {
         endDateInput.value = getToday();
     }
 
-    startDateInput.disabled = endDateInput.disabled = true;
+    startDateInput.readOnly = endDateInput.readOnly = true;
 
     chrome.storage.local.set({
         startingDate: startDateInput.value,