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 #1

Updated by Tiago Melo over 3 years ago

  • Subject changed from SCSS modules are breaking our current way of branding the dashboard to mgr/dashboard: SCSS modules are breaking our current way of branding the dashboard
Actions #2

Updated by Tiago Melo over 3 years ago

  • Description updated (diff)
Actions #3

Updated by Tiago Melo over 3 years ago

  • Description updated (diff)
Actions #4

Updated by Tiago Melo over 3 years ago

  • Description updated (diff)
Actions #5

Updated by Tiago Melo over 3 years ago

  • Description updated (diff)
Actions #6

Updated by Tiago Melo over 3 years ago

  • Description updated (diff)
Actions #7

Updated by Lenz Grimmer over 3 years ago

  • Assignee set to Anonymous
  • Target version set to v16.0.0
Actions #8

Updated by Alfonso Martínez over 3 years ago

  • Related to Feature #45934: mgr/dashboard: Use SASS's new Modules instead of the old @import added
Actions #9

Updated by Alfonso Martínez over 3 years ago

  • Status changed from New to In Progress
  • Assignee changed from Anonymous to Alfonso Martínez
Actions #10

Updated by Alfonso Martínez over 3 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 37617
Actions #11

Updated by Lenz Grimmer over 3 years ago

  • Status changed from Fix Under Review to Resolved
Actions #12

Updated by Ernesto Puerta about 3 years ago

  • Project changed from mgr to Dashboard
  • Category changed from 165 to UI
Actions

Also available in: Atom PDF