From b25b6f36bb03fb5ba4f4b5f4527b67bd312ef072 Mon Sep 17 00:00:00 2001 From: Pragadesh-45 <54320162+Pragadesh-45@users.noreply.github.com> Date: Thu, 12 Mar 2026 21:41:11 +0530 Subject: [PATCH] refactor: simplify environment list actions and improve styling (#7459) --- .../EnvironmentDetails/StyledWrapper.js | 2 +- .../EnvironmentList/StyledWrapper.js | 14 +++- .../EnvironmentList/index.js | 81 ++++++++++-------- .../EnvironmentDetails/StyledWrapper.js | 2 +- .../EnvironmentList/StyledWrapper.js | 14 +++- .../EnvironmentList/index.js | 82 +++++++++++-------- 6 files changed, 119 insertions(+), 76 deletions(-) diff --git a/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/EnvironmentDetails/StyledWrapper.js b/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/EnvironmentDetails/StyledWrapper.js index 8c6e532ba..80414babd 100644 --- a/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/EnvironmentDetails/StyledWrapper.js +++ b/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/EnvironmentDetails/StyledWrapper.js @@ -12,7 +12,7 @@ const StyledWrapper = styled.div` display: flex; align-items: center; justify-content: space-between; - padding: 16px 20px 8px 20px; + padding: 9px 20px 8px 20px; flex-shrink: 0; .title { diff --git a/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/StyledWrapper.js b/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/StyledWrapper.js index 87ba75d63..b2f6aad5c 100644 --- a/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/StyledWrapper.js +++ b/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/StyledWrapper.js @@ -72,7 +72,7 @@ const StyledWrapper = styled.div` font-size: 12px; background: transparent; border: 1px solid ${(props) => props.theme.border.border1}; - border-radius: 5px; + border-radius: 6px; color: ${(props) => props.theme.text}; transition: border-color 0.15s ease; @@ -110,7 +110,15 @@ const StyledWrapper = styled.div` display: flex; flex-direction: column; overflow: hidden; - padding: 0 8px; + padding: 8px; + } + + .section-header { + margin-inline: 4px !important; + padding-left: 6px !important; + border-radius: 6px ; + padding-right: 3px !important; + padding-block: 4px !important; } .environments-list { @@ -153,7 +161,7 @@ const StyledWrapper = styled.div` font-size: 13px; color: ${(props) => props.theme.text}; cursor: pointer; - border-radius: 5px; + border-radius: 6px; transition: background 0.15s ease; .environment-name { diff --git a/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/index.js b/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/index.js index 231ceee25..29c36f3d4 100644 --- a/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/index.js +++ b/packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/index.js @@ -49,7 +49,6 @@ const EnvironmentList = ({ const [openImportModal, setOpenImportModal] = useState(false); const [searchText, setSearchText] = useState(''); - const [isEnvListSearchExpanded, setIsEnvListSearchExpanded] = useState(false); const envListSearchInputRef = useRef(null); const [isCreatingInline, setIsCreatingInline] = useState(false); const [renamingEnvUid, setRenamingEnvUid] = useState(null); @@ -558,51 +557,65 @@ const EnvironmentList = ({ <> - - - )} > - {isEnvListSearchExpanded && ( -
- - setSearchText(e.target.value)} - className="env-list-search-input" - autoComplete="off" - autoCorrect="off" - autoCapitalize="off" - spellCheck="false" - /> - {searchText && ( - - )} -
- )} +
+ + setSearchText(e.target.value)} + className="env-list-search-input" + autoComplete="off" + autoCorrect="off" + autoCapitalize="off" + spellCheck="false" + /> + {searchText && ( + + )} +
{filteredEnvironments.map((env) => (
props.theme.border.border1}; - border-radius: 5px; + border-radius: 6px; color: ${(props) => props.theme.text}; transition: border-color 0.15s ease; @@ -111,7 +111,15 @@ const StyledWrapper = styled.div` display: flex; flex-direction: column; overflow: hidden; - padding: 0 8px; + padding: 8px; + } + + .section-header { + margin-inline: 4px; + padding-left: 6px; + border-radius: 6px; + padding-right: 3px; + padding-block: 4px; } .environments-list { @@ -154,7 +162,7 @@ const StyledWrapper = styled.div` font-size: 13px; color: ${(props) => props.theme.text}; cursor: pointer; - border-radius: 5px; + border-radius: 6px; transition: background 0.15s ease; .environment-name { diff --git a/packages/bruno-app/src/components/WorkspaceHome/WorkspaceEnvironments/EnvironmentList/index.js b/packages/bruno-app/src/components/WorkspaceHome/WorkspaceEnvironments/EnvironmentList/index.js index 47db76f4e..b3f72a244 100644 --- a/packages/bruno-app/src/components/WorkspaceHome/WorkspaceEnvironments/EnvironmentList/index.js +++ b/packages/bruno-app/src/components/WorkspaceHome/WorkspaceEnvironments/EnvironmentList/index.js @@ -48,7 +48,6 @@ const EnvironmentList = ({ const [openImportModal, setOpenImportModal] = useState(false); const [searchText, setSearchText] = useState(''); - const [isEnvListSearchExpanded, setIsEnvListSearchExpanded] = useState(false); const envListSearchInputRef = useRef(null); const [isCreatingInline, setIsCreatingInline] = useState(false); const [renamingEnvUid, setRenamingEnvUid] = useState(null); @@ -552,51 +551,66 @@ const EnvironmentList = ({ <> - - - )} > - {isEnvListSearchExpanded && ( -
- - setSearchText(e.target.value)} - className="env-list-search-input" - autoComplete="off" - autoCorrect="off" - autoCapitalize="off" - spellCheck="false" - /> - {searchText && ( - - )} -
- )} +
+ + setSearchText(e.target.value)} + className="env-list-search-input" + autoComplete="off" + autoCorrect="off" + autoCapitalize="off" + spellCheck="false" + /> + {searchText && ( + + )} +
{filteredEnvironments.map((env) => (