From 073117aa1bf73adb7d25549412371387321c0f2a Mon Sep 17 00:00:00 2001 From: Anoop M D Date: Sat, 27 Dec 2025 09:26:07 +0530 Subject: [PATCH] chore: addressed review comments --- packages/bruno-app/src/components/Preferences/index.js | 4 +--- .../bruno-app/src/providers/ReduxStore/slices/app.js | 9 +-------- packages/bruno-app/src/themes/index.js | 4 ++-- packages/bruno-app/src/themes/light/light-monochrome.js | 2 +- packages/bruno-app/src/themes/light/light.js | 2 +- 5 files changed, 6 insertions(+), 15 deletions(-) diff --git a/packages/bruno-app/src/components/Preferences/index.js b/packages/bruno-app/src/components/Preferences/index.js index dd8df488b..b8f365f5d 100644 --- a/packages/bruno-app/src/components/Preferences/index.js +++ b/packages/bruno-app/src/components/Preferences/index.js @@ -1,7 +1,6 @@ import Modal from 'components/Modal/index'; import classnames from 'classnames'; import React, { useState } from 'react'; -import { useSelector } from 'react-redux'; import Support from './Support'; import General from './General'; @@ -14,8 +13,7 @@ import Beta from './Beta'; import StyledWrapper from './StyledWrapper'; const Preferences = ({ onClose }) => { - const preferencesTab = useSelector((state) => state.app.preferencesTab); - const [tab, setTab] = useState(preferencesTab || 'general'); + const [tab, setTab] = useState('general'); const getTabClassname = (tabName) => { return classnames(`tab select-none ${tabName}`, { diff --git a/packages/bruno-app/src/providers/ReduxStore/slices/app.js b/packages/bruno-app/src/providers/ReduxStore/slices/app.js index dbf320381..ff7115565 100644 --- a/packages/bruno-app/src/providers/ReduxStore/slices/app.js +++ b/packages/bruno-app/src/providers/ReduxStore/slices/app.js @@ -11,7 +11,6 @@ const initialState = { screenWidth: 500, showHomePage: false, showPreferences: false, - preferencesTab: null, showApiSpecPage: false, showManageWorkspacePage: false, isEnvironmentSettingsModalOpen: false, @@ -96,13 +95,7 @@ export const appSlice = createSlice({ state.showApiSpecPage = false; }, showPreferences: (state, action) => { - if (typeof action.payload === 'object') { - state.showPreferences = action.payload.show; - state.preferencesTab = action.payload.tab || null; - } else { - state.showPreferences = action.payload; - state.preferencesTab = null; - } + state.showPreferences = action.payload; }, updatePreferences: (state, action) => { state.preferences = action.payload; diff --git a/packages/bruno-app/src/themes/index.js b/packages/bruno-app/src/themes/index.js index 1f7ae1d9c..2bd122f6c 100644 --- a/packages/bruno-app/src/themes/index.js +++ b/packages/bruno-app/src/themes/index.js @@ -32,7 +32,7 @@ const themes = { export const themeRegistry = { 'light': { id: 'light', - name: 'Light', + name: 'Bruno Light', mode: 'light' }, 'light-monochrome': { @@ -42,7 +42,7 @@ export const themeRegistry = { }, 'light-pastel': { id: 'light-pastel', - name: 'Aurora', + name: 'Bruno Light Pastel', mode: 'light' }, 'catppuccin-latte': { diff --git a/packages/bruno-app/src/themes/light/light-monochrome.js b/packages/bruno-app/src/themes/light/light-monochrome.js index 167d3ef55..9dd9bb165 100644 --- a/packages/bruno-app/src/themes/light/light-monochrome.js +++ b/packages/bruno-app/src/themes/light/light-monochrome.js @@ -258,7 +258,7 @@ const lightMonochromeTheme = { hoverBorder: '#696969' }, close: { - color: '212529', + color: '#212529', bg: 'white', border: 'white', hoverBorder: '' diff --git a/packages/bruno-app/src/themes/light/light.js b/packages/bruno-app/src/themes/light/light.js index 071a484df..f57c289d7 100644 --- a/packages/bruno-app/src/themes/light/light.js +++ b/packages/bruno-app/src/themes/light/light.js @@ -286,7 +286,7 @@ const lightTheme = { hoverBorder: '#696969' }, close: { - color: '212529', + color: '#212529', bg: 'white', border: 'white', hoverBorder: ''