Skip to content

Commit eb7a089

Browse files
fix: address PR review feedback on header CSS
- #header-status-host: flex: 0 0 0% -> flex: 0 1 auto so the teleported HeaderStatus (chapter title + % complete) is visible on all viewports, not just >=992px. Auto basis sizes to content; shrink=1 tolerates tight layouts; grow=0 still prevents gap when element has no content. - .banner ul.nav: overflow: hidden -> overflow: clip. Clips paint without creating a scroll container, preserving focus rings on child elements (overflow:hidden clips child outlines; overflow:clip does not). - .sidebar-tooltip-text: add margin-left: 0 to reset the -60px inherited from .has-tooltip .tooltip-text. Without this, left: calc(100% + 4px) was offset 60px left, placing the tooltip over the trigger element.
1 parent 9f87ee2 commit eb7a089

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

EssentialCSharp.Web/wwwroot/css/styles.css

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ a:hover {
217217
}
218218

219219
#header-status-host {
220-
flex: 0 0 0%;
220+
flex: 0 1 auto;
221221
min-width: 0;
222222
}
223223

@@ -230,7 +230,7 @@ a:hover {
230230
.banner ul.nav {
231231
flex-wrap: nowrap;
232232
max-height: var(--toolbar-height);
233-
overflow: hidden;
233+
overflow: clip;
234234
align-items: center;
235235
}
236236

@@ -325,6 +325,7 @@ a:hover {
325325
position: absolute;
326326
top: 50%;
327327
left: calc(100% + 4px);
328+
margin-left: 0;
328329
transform: translateY(-50%);
329330
border-radius: 10px;
330331
padding: 10px;

0 commit comments

Comments
 (0)