Bug #46869
mgr/dashboard: SCSS modules are breaking our current way of branding the dashboard
% Done:
0%
Source:
Tags:
Backport:
Regression:
No
Severity:
2 - major
Reviewed:
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
History
#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
#2 Updated by Tiago Melo over 3 years ago
- Description updated (diff)
#3 Updated by Tiago Melo over 3 years ago
- Description updated (diff)
#4 Updated by Tiago Melo over 3 years ago
- Description updated (diff)
#5 Updated by Tiago Melo over 3 years ago
- Description updated (diff)
#6 Updated by Tiago Melo over 3 years ago
- Description updated (diff)
#7 Updated by Lenz Grimmer over 3 years ago
- Assignee set to Anonymous
- Target version set to v16.0.0
#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
#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
#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
#11 Updated by Lenz Grimmer over 3 years ago
- Status changed from Fix Under Review to Resolved
#12 Updated by Ernesto Puerta almost 3 years ago
- Project changed from mgr to Dashboard
- Category changed from 165 to UI