@olwiba/ui 0.2.16 → 0.2.17
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/index.js +3 -3
- package/dist/index.js.map +1 -1
- package/package.json +1 -1
- package/src/marketing/Footer.tsx +5 -1
- package/src/marketing/Navbar.tsx +22 -2
package/package.json
CHANGED
package/src/marketing/Footer.tsx
CHANGED
|
@@ -39,7 +39,11 @@ export function Footer({
|
|
|
39
39
|
children: (
|
|
40
40
|
<>
|
|
41
41
|
{brand.logo}
|
|
42
|
-
|
|
42
|
+
{/* Same trim as Navbar's lockup, and more visible here: text-lg
|
|
43
|
+
scales the unused ascender space up with the font size. */}
|
|
44
|
+
<span className="text-lg font-semibold leading-none text-foreground [text-box-edge:ex_alphabetic] [text-box-trim:trim-both]">
|
|
45
|
+
{brand.name}
|
|
46
|
+
</span>
|
|
43
47
|
</>
|
|
44
48
|
),
|
|
45
49
|
})}
|
package/src/marketing/Navbar.tsx
CHANGED
|
@@ -42,7 +42,24 @@ export function Navbar({
|
|
|
42
42
|
children: (
|
|
43
43
|
<span className="flex items-center gap-2 font-semibold">
|
|
44
44
|
{brand.logo}
|
|
45
|
-
|
|
45
|
+
{/* `items-center` centres the text's *box*, not its glyphs, and
|
|
46
|
+
the box reserves room for ascenders and descenders the name may
|
|
47
|
+
not use — so a lowercase wordmark sits low with dead air above.
|
|
48
|
+
`leading-none` only removes half-leading; the ascender space is
|
|
49
|
+
still inside the box, which is why it wasn't enough on its own.
|
|
50
|
+
|
|
51
|
+
text-box-trim/-edge trims the box to the glyphs themselves.
|
|
52
|
+
`ex alphabetic` = x-height to baseline, chosen over `cap
|
|
53
|
+
alphabetic` because these wordmarks are lowercase (nestrrr).
|
|
54
|
+
A product with a capitalised name wants `cap` instead —
|
|
55
|
+
otherwise its capitals overflow the trimmed box and read high.
|
|
56
|
+
|
|
57
|
+
leading-none stays as the fallback: text-box-trim is not
|
|
58
|
+
Baseline yet (no Firefox), and there it degrades to the old
|
|
59
|
+
behaviour rather than breaking. */}
|
|
60
|
+
<span className="leading-none [text-box-edge:ex_alphabetic] [text-box-trim:trim-both]">
|
|
61
|
+
{brand.name}
|
|
62
|
+
</span>
|
|
46
63
|
</span>
|
|
47
64
|
),
|
|
48
65
|
})}
|
|
@@ -104,7 +121,10 @@ export function Navbar({
|
|
|
104
121
|
children: (
|
|
105
122
|
<span className="flex items-center gap-2 font-semibold">
|
|
106
123
|
{brand.logo}
|
|
107
|
-
|
|
124
|
+
{/* Same fix as the desktop lockup above. */}
|
|
125
|
+
<span className="leading-none [text-box-edge:ex_alphabetic] [text-box-trim:trim-both]">
|
|
126
|
+
{brand.name}
|
|
127
|
+
</span>
|
|
108
128
|
</span>
|
|
109
129
|
),
|
|
110
130
|
})}
|