Feat(heuristics): Add optional weights feature toggle in settings by rakshityadav1868 · Pull Request #1833 · ruxailab/RUXAILAB

rakshityadav1868 · 2026-03-07T19:20:06Z

Overview

Add an optional "Enable Weights Feature" toggle in the Settings tab of the Edit Test page, allowing admins to control whether the Weights tab is visible and accessible for heuristic studies.

Changes Made

1. Data Model Update

Modified HeuristicStudy.js to include useWeights boolean field (default: false)
Added useWeights to toFirestore() method for persistence

2. Settings UI Enhancement

Added toggle switch in HeuristicsSettings.vue to enable/disable weights feature

3. Frontend Logic

Updated EditTest.vue to conditionally show/hide Weights tab based on useWeights flag
Added watcher to automatically adjust active tab index when Weights tab visibility changes

4. Internationalization

Added translations in all supported languages:
- English, Spanish, French, German, Portuguese, Russian, Hindi, Arabic, Chinese, Japanese
New i18n keys:
- HeuristicsSettings.titles.enableWeights: Label for toggle
- HeuristicsSettings.messages.enableWeightsDescription: Description text

Screen.Recording.2026-03-08.at.12.46.23.AM.mp4

Signed-off-by: Rakshit Yadav <yadavrakshit60@gmail.com>

sonarqubecloud · 2026-03-07T19:21:03Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Copilot

Pull request overview

Adds a per-heuristic-study useWeights flag and exposes it in the Edit Test “Settings” tab so admins can toggle visibility/access to the Weights tab.

Changes:

Extend HeuristicStudy to persist a new boolean useWeights field to Firestore.
Add a new “Enable Weights Feature” toggle to HeuristicsSettings.vue and conditionally render the Weights tab/content in EditTest.vue.
Add i18n strings for the new toggle across supported locales.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
src/ux/Heuristic/views/EditTest.vue	Conditionally renders the Weights tab and adjusts tab indices when visibility changes.
src/ux/Heuristic/models/HeuristicStudy.js	Adds `useWeights` to the heuristic study model and persistence.
src/ux/Heuristic/components/HeuristicsSettings.vue	Adds settings UI toggle and commits `useWeights` into Vuex.
src/app/plugins/locales/en.json	Adds i18n keys for the toggle label/description.
src/app/plugins/locales/es.json	Adds i18n keys for the toggle label/description.
src/app/plugins/locales/fr.json	Adds i18n keys for the toggle label/description.
src/app/plugins/locales/de.json	Adds i18n keys for the toggle label/description.
src/app/plugins/locales/pt_br.json	Adds i18n keys for the toggle label/description.
src/app/plugins/locales/ru.json	Adds i18n keys for the toggle label/description.
src/app/plugins/locales/hi.json	Adds i18n keys for the toggle label/description.
src/app/plugins/locales/ar.json	Adds i18n keys for the toggle label/description.
src/app/plugins/locales/zh.json	Adds i18n keys for the toggle label/description.
src/app/plugins/locales/ja.json	Adds i18n keys for the toggle label/description.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-07T19:24:49Z

src/ux/Heuristic/views/EditTest.vue

+// Watch for changes in showWeightsTab to fix index
+watch(showWeightsTab, (newVal, oldVal) => {
+  // If weights tab visibility changed and we're currently on or past the weights tab
+  if (newVal !== oldVal) {
+    if (!newVal && index.value > 1) {
+      // Weights tab is now hidden and index is on Settings (was 3, should be 2)
+      index.value = 2
+    }
+  }
+})


The watcher only corrects the active tab when weights are disabled. If weights are enabled while the user is on Settings (index 2 when weights are hidden), the current index will now point to the Weights tab and the UI will unexpectedly jump away from Settings. Consider updating the watcher to also handle the false→true transition (and generally clamp index to the highest available tab based on showWeightsTab/showSettingsTab).

Suggested change

// Watch for changes in showWeightsTab to fix index

watch(showWeightsTab, (newVal, oldVal) => {

// If weights tab visibility changed and we're currently on or past the weights tab

if (newVal !== oldVal) {

if (!newVal && index.value > 1) {

// Weights tab is now hidden and index is on Settings (was 3, should be 2)

index.value = 2

}

}

})

// Watch for changes in tab visibility to keep the active index valid

watch(

[showWeightsTab, showSettingsTab],

([weightsVisible, settingsVisible], [oldWeightsVisible]) => {

// Handle the case where weights are enabled while the user is on Settings.

// When weights are hidden, SETTINGS has index 2; when weights are shown,

// WEIGHTS becomes 2 and SETTINGS moves to 3. If we don't adjust here,

// the user would unexpectedly jump from SETTINGS to WEIGHTS.

if (!oldWeightsVisible && weightsVisible && settingsVisible && index.value === 2) {

index.value = 3

return

}

// In all cases, clamp the index to the highest available tab based on visibility.

let maxIndex = 1 // HEURISTICS (0) and OPTIONS (1) are always present

if (weightsVisible) {

maxIndex = 2

}

if (settingsVisible) {

maxIndex = weightsVisible ? 3 : 2

}

if (index.value > maxIndex) {

index.value = maxIndex

}

}

)

Copilot · 2026-03-07T19:24:50Z

src/ux/Heuristic/views/EditTest.vue

 const windowWidth = ref(window.innerWidth)
 const showSettingsTab = computed(() => !props.isTemplate)
+const test = computed(() => store.getters.test)
+const showWeightsTab = computed(() => test.value.useWeights ?? false)


In template preview mode, showWeightsTab is derived solely from test.useWeights. The template preview payload currently provides testWeights but not useWeights, so weights will never be shown in the preview even if the template contains weights. Either include useWeights in the template payload or derive the preview visibility from existing data (e.g., non-empty testWeights) / default to showing weights for templates.

Suggested change

const showWeightsTab = computed(() => test.value.useWeights ?? false)

const showWeightsTab = computed(() => {

if (props.isTemplate) {

const weights = props.templateTest?.testWeights

if (Array.isArray(weights)) {

return weights.length > 0

}

return !!weights

}

return test.value?.useWeights ?? false

})

Copilot · 2026-03-07T19:24:50Z

src/ux/Heuristic/views/EditTest.vue

          <HeuristicsSettings
-            v-if="showSettingsTab && index == 3"
+            v-if="showSettingsTab && (showWeightsTab ? index == 3 : index == 2)"
            :is-template="isTemplate"


Changes made inside HeuristicsSettings (like toggling useWeights) currently won’t make the Save button appear because this component doesn’t emit a change event and the parent only sets change = true for other tabs. Consider emitting a change event from HeuristicsSettings when useWeights changes and wiring it up here so the user can persist the toggle.

Suggested change

:is-template="isTemplate"

:is-template="isTemplate"

@change="change = true"

Copilot · 2026-03-07T19:24:50Z

src/ux/Heuristic/components/HeuristicsSettings.vue

+  set: (value) => {
+    store.commit('SET_TEST', {
+      ...test.value,
+      useWeights: value,
+    })


Toggling useWeights commits to Vuex but doesn’t notify the parent view that there are unsaved changes, so the toggle can’t be persisted unless the user edits something else first. Consider emitting a change event (or otherwise setting the parent’s change flag) from this setter / switch update handler.

feat add new weight option toggle feature

bf02c55

Signed-off-by: Rakshit Yadav <yadavrakshit60@gmail.com>

Copilot AI review requested due to automatic review settings March 7, 2026 19:20

github-actions bot added heuristic high-complexity new-feature New feature or request size/M ui/ux labels Mar 7, 2026

Copilot started reviewing on behalf of rakshityadav1868 March 7, 2026 19:20 View session

Copilot AI reviewed Mar 7, 2026

View reviewed changes

github-actions bot added stale and removed stale labels Mar 15, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat(heuristics): Add optional weights feature toggle in settings#1833

Feat(heuristics): Add optional weights feature toggle in settings#1833
rakshityadav1868 wants to merge 1 commit intoruxailab:developfrom
rakshityadav1868:adding-weight-toggle-btn

rakshityadav1868 commented Mar 7, 2026

Uh oh!

sonarqubecloud bot commented Mar 7, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 7, 2026

Uh oh!

Copilot AI Mar 7, 2026

Uh oh!

Copilot AI Mar 7, 2026

Uh oh!

Copilot AI Mar 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

-// Watch for changes in showWeightsTab to fix index
-watch(showWeightsTab, (newVal, oldVal) => {
-  // If weights tab visibility changed and we're currently on or past the weights tab
-  if (newVal !== oldVal) {
-    if (!newVal && index.value > 1) {
-      // Weights tab is now hidden and index is on Settings (was 3, should be 2)
-      index.value = 2
-    }
-  }
-})
+// Watch for changes in tab visibility to keep the active index valid
+watch(
+  [showWeightsTab, showSettingsTab],
+  ([weightsVisible, settingsVisible], [oldWeightsVisible]) => {
+    // Handle the case where weights are enabled while the user is on Settings.
+    // When weights are hidden, SETTINGS has index 2; when weights are shown,
+    // WEIGHTS becomes 2 and SETTINGS moves to 3. If we don't adjust here,
+    // the user would unexpectedly jump from SETTINGS to WEIGHTS.
+    if (!oldWeightsVisible && weightsVisible && settingsVisible && index.value === 2) {
+      index.value = 3
+      return
+    }
+    // In all cases, clamp the index to the highest available tab based on visibility.
+    let maxIndex = 1 // HEURISTICS (0) and OPTIONS (1) are always present
+    if (weightsVisible) {
+      maxIndex = 2
+    }
+    if (settingsVisible) {
+      maxIndex = weightsVisible ? 3 : 2
+    }
+    if (index.value > maxIndex) {
+      index.value = maxIndex
+    }
+  }
+)

-const showWeightsTab = computed(() => test.value.useWeights ?? false)
+const showWeightsTab = computed(() => {
+  if (props.isTemplate) {
+    const weights = props.templateTest?.testWeights
+    if (Array.isArray(weights)) {
+      return weights.length > 0
+    }
+    return !!weights
+  }
+  return test.value?.useWeights ?? false
+})

	:is-template="isTemplate"
	:is-template="isTemplate"
	@change="change = true"

Conversation

rakshityadav1868 commented Mar 7, 2026

Overview

Changes Made

1. Data Model Update

2. Settings UI Enhancement

3. Frontend Logic

4. Internationalization

Uh oh!

sonarqubecloud bot commented Mar 7, 2026

Quality Gate passed

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants