fix top bar bubbles
This commit is contained in:
@@ -33,6 +33,7 @@ Follow the current NodeBB theme model:
|
|||||||
- Template overrides belong under `templates/` and must preserve the same relative path as the Harmony template they replace.
|
- Template overrides belong under `templates/` and must preserve the same relative path as the Harmony template they replace.
|
||||||
- Do not copy or redefine Harmony templates unless an actual override is needed.
|
- Do not copy or redefine Harmony templates unless an actual override is needed.
|
||||||
- If a template does not exist here, NodeBB inherits it from the configured base theme.
|
- If a template does not exist here, NodeBB inherits it from the configured base theme.
|
||||||
|
- In the Westgate topbar, notification, chat, draft, and navigation count bubbles must honor NodeBB/Harmony's `hidden` class and only display when there is an actual count to check.
|
||||||
|
|
||||||
NodeBB theme documentation: https://docs.nodebb.org/development/themes/
|
NodeBB theme documentation: https://docs.nodebb.org/development/themes/
|
||||||
|
|
||||||
|
|||||||
@@ -149,7 +149,7 @@ $wg-topbar-panel-bg: linear-gradient(
|
|||||||
color: #fff0dc;
|
color: #fff0dc;
|
||||||
}
|
}
|
||||||
|
|
||||||
.wg-topbar__utility-list .visible-closed {
|
.wg-topbar__utility-list .visible-closed:not(.hidden) {
|
||||||
display: inline-block !important;
|
display: inline-block !important;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -74,6 +74,11 @@ assertIncludes(
|
|||||||
'.wg-topbar',
|
'.wg-topbar',
|
||||||
'Topbar styles should be scoped to .wg-topbar'
|
'Topbar styles should be scoped to .wg-topbar'
|
||||||
);
|
);
|
||||||
|
assertIncludes(
|
||||||
|
stylesheet,
|
||||||
|
'.wg-topbar__utility-list .visible-closed:not(.hidden)',
|
||||||
|
'Topbar collapsed utility labels should not override hidden count badges'
|
||||||
|
);
|
||||||
assertIncludes(
|
assertIncludes(
|
||||||
client,
|
client,
|
||||||
'initWestgateTopbar',
|
'initWestgateTopbar',
|
||||||
|
|||||||
Reference in New Issue
Block a user