Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • P Plasma Workspace
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 21
    • Issues 21
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 109
    • Merge requests 109
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • PlasmaPlasma
  • Plasma Workspace
  • Merge requests
  • !1098

applets/notifications: Tint header for critical notifications

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Nate Graham requested to merge ngraham/plasma-workspace:tint-critical-notification-headers into master Oct 07, 2021
  • Overview 23
  • Commits 1
  • Pipelines 0
  • Changes 2

Requires frameworks/plasma-framework!348 (closed).

We get intermittent user complaints and bug reports about notifications not being visible enough, especially on large or cluttered screens.

This commit attempts to remedy the situation by tinting the header for critical notifications to use the color-scheme-specified "warning" color. Only critical notifications receive this treatment since they're the only ones that the user really does need to see.

BUG: 420541 FIXED-IN: 5.24

Screenshot_20211012_201152

cc @teams/vdg

Edited Oct 13, 2021 by Nate Graham
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: tint-critical-notification-headers