Project

General

Profile

Actions

Bug #46869

closed

mgr/dashboard: SCSS modules are breaking our current way of branding the dashboard

Added by Enno Gotthold almost 4 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
High
Category:
UI
Target version:
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
2 - major
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

As of PR https://github.com/ceph/ceph/pull/35501 the dashboard branding doesn't work like before.

This means that we need to adjust some code and of course also the HACKING.rst (see: [[https://github.com/ceph/ceph/pull/35954#discussion_r466950052]])

Current issues:
  • We have direct import of variables from '_bootstrap-defaults.scss'. Changing vars in vendor files will not update this.
  • We need to change the order of imports of '_bootstrap-defaults.scss' and 'vendor/variables'. ATM updating vendor variables is not updating the theme.
  • the @import of ceph custom styles can significantly increased the size of the build, depending on the content of 'vendor/variables'

Related issues 1 (0 open1 closed)

Related to Dashboard - Feature #45934: mgr/dashboard: Use SASS's new Modules instead of the old @importResolvedEnno Gotthold

Actions
Actions

Also available in: Atom PDF