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.

Sign up for GitHub

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

Jump to bottom

Alternative dark: improve border color of notification #5206

Merged
merged 1 commit into from Mar 15, 2023
Merged

Alternative dark: improve border color of notification #5206

merged 1 commit into from Mar 15, 2023

Conversation

math-GH
Copy link
Contributor

@math-GH math-GH commented Mar 14, 2023

See #5205 (comment)

Changes proposed in this pull request:

  • use the blue color as it is already used for other variables

How to test the feature manually:

  1. use Alternative dark theme
  2. do something, that opens a notification, f.e. switch on/off a extension
  3. see the border color of the notification

Pull request checklist:

  • clear commit messages
  • code manually tested

@math-GH math-GH added the Theme label Mar 14, 2023
@math-GH math-GH added this to the 1.22.0 milestone Mar 14, 2023
@math-GH math-GH changed the title Alternative dark: improve order color of notification Alternative dark: improve border color of notification Mar 14, 2023
@Alkarex Alkarex merged commit afccff4 into FreshRSS:edge Mar 15, 2023
1 check passed
@math-GH math-GH deleted the alternative-dark-border-color-notification branch March 15, 2023 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Frenzie Frenzie Frenzie approved these changes

@ColonelMoutarde ColonelMoutarde ColonelMoutarde approved these changes

Assignees
No one assigned
Labels
Theme
Projects
None yet
Milestone
1.22.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@math-GH @Frenzie @ColonelMoutarde @Alkarex