Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 5.3.9104] Notices: Add customization for background color, text color, and text position. #61915

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

RXminuS
Copy link
Contributor

@RXminuS RXminuS commented Apr 16, 2024

Closes #59751

  • Added variant option for Notices. Defaults to default option for position.
  • Added styleOverrides option to Notices with three fields:
    • backgroundColor
    • textColor
    • textCentered

This means in the site-config a user can now specify notices as:

{
"notices": [
    {
      "message": "THIS IS AN IMPORTANT NOTICE",
      "location": "top",
      "variant": "danger",
      "styleOverrides": {
        "backgroundColor": "#7f1d1d",
        "textColor": "#fecaca",
        "textCentered": true
      }
    }
  ]
}

resulting in the following notice
CleanShot 2024-03-26 at 00 03 52@2x

Test plan

  • Updated existing regression tests
  • Manually tested with various site-config settings.

Backport ccba7d4 from #61338

… position. (#61338)

Closes #59751

Co-authored-by: Rik Nauta <rik.nauta@sourcegraph.com>
@sourcegraph-bot
Copy link
Contributor

📖 Storybook live preview

@@ -31,6 +31,7 @@ All notable changes to Sourcegraph are documented in this file.
- GitHub apps installation records will only be deleted from the database if the GitHub App has been uninstalled or if the GitHub app has been deleted. [#60460](https://github.com/sourcegraph/sourcegraph/pull/60460)
- The Anthropic provider for Cody has been updated to use the messages API which includes support for Claude 3 models. This is applicable to both BYOK and Cody Gateway users. The messages API does not support model identifiers which only set a major model version such as: `claude-2`, `claude-instant-v1` and `claude-instant-1`. Default values have been updated to `claude-2.0` and `claude-instant-1.2`, any legacy models identifiers in the site config will be set to the corresponding default previously mentioned. [#60953](https://github.com/sourcegraph/sourcegraph/pull/60953) [#61324](https://github.com/sourcegraph/sourcegraph/pull/61324)
- The AWS Bedrock provider for Cody has been updated to use Anthropic's Messages API, bringing support for Claude 3 models. [#61347](https://github.com/sourcegraph/sourcegraph/pull/61347)
- Notices configured in the site config now allow for specifying a style or color. [#61338](https://github.com/sourcegraph/sourcegraph/pull/61338)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't need to be here. It only need to exist on main.
I've added it here for you: #61918

@Chickensoupwithrice Chickensoupwithrice merged commit 6af7844 into 5.3.9104 Apr 18, 2024
17 of 19 checks passed
@Chickensoupwithrice Chickensoupwithrice deleted the sg/backport-61338-to-5.3.9104 branch April 18, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants