From d0f8643f012e1b53266692d4043a65040e68e345 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=BAlia=20Jaeger=20Foresti?=
<60678893+juliajforesti@users.noreply.github.com>
Date: Thu, 23 Jun 2022 21:31:41 -0300
Subject: [PATCH] [FIX] sidebar colors (#25987)
<!-- This is a pull request template, you do not need to uncomment or remove the comments, they won't show up in the PR text. -->
<!-- Your Pull Request name should start with one of the following tags
[NEW] For new features
[IMPROVE] For an improvement (performance or little improvements) in existing features
[FIX] For bug fixes that affect the end-user
[BREAK] For pull requests including breaking changes
Chore: For small tasks
Doc: For documentation
-->
<!-- Checklist!!! If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.
- I have read the Contributing Guide - https://github.com/RocketChat/Rocket.Chat/blob/develop/.github/CONTRIBUTING.md#contributing-to-rocketchat doc
- I have signed the CLA - https://cla-assistant.io/RocketChat/Rocket.Chat
- Lint and unit tests pass locally with my changes
- I have added tests that prove my fix is effective or that my feature works (if applicable)
- I have added necessary documentation (if applicable)
- Any dependent changes have been merged and published in downstream modules
-->
## Proposed changes (including videos or screenshots)
<!-- CHANGELOG -->
<!--
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to that issue below.
This description will appear in the release notes if we accept the contribution.
-->
<!-- END CHANGELOG -->
## Issue(s)
<!-- Link the issues being closed by or related to this PR. For example, you can use #594 if this PR closes issue number 594 -->
## Steps to test or reproduce
<!-- Mention how you would reproduce the bug if not mentioned on the issue page already. Also mention which screens are going to have the changes if applicable -->
## Further comments
<!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
Co-authored-by: Guilherme Gazzo <5263975+ggazzo@users.noreply.github.com>
---
apps/meteor/client/sidebar/hooks/useSidebarPaletteColor.js | 3 +++
turbo.json | 3 +--
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/apps/meteor/client/sidebar/hooks/useSidebarPaletteColor.js b/apps/meteor/client/sidebar/hooks/useSidebarPaletteColor.js
index 4c34c8cd8d2..8396281fb43 100644
--- a/apps/meteor/client/sidebar/hooks/useSidebarPaletteColor.js
+++ b/apps/meteor/client/sidebar/hooks/useSidebarPaletteColor.js
@@ -167,6 +167,9 @@ const getStyle = (
--rcx-button-colors-icon-background-color: ${toVar(colors.sibebarSurface)};
--rcx-button-colors-icon-hover-background-color: ${toVar(colors.n900)};
--rcx-button-colors-icon-hover-border-color: ${toVar(colors.n900)};
+ --rcx-button-colors-icon-focus-background-color: var(--rcx-color-neutral-300);
+ --rcx-button-colors-icon-focus-border-color: var(--rcx-color-neutral-500);
+ --rcx-button-colors-icon-focus-shadow-color: none;
--rcx-button-colors-icon-success-active-border-color: ${toVar(colors.n900)};
--rcx-button-colors-icon-success-active-background-color: ${toVar(colors.n800)};
diff --git a/turbo.json b/turbo.json
index 47b65ad6023..c12232840f7 100644
--- a/turbo.json
+++ b/turbo.json
@@ -7,8 +7,7 @@
},
"build": {
"dependsOn": ["^build"],
- "outputs": ["dist/**"],
- "inputs": ["!tests/**"]
+ "outputs": ["dist/**"]
},
"testunit": {
"dependsOn": ["build"],
--
GitLab