Skip to content

Commit b42bdf6

Browse files
authored
Merge pull request #291 from wpengine/chore-fix-mobile-ux-docs-page
Chore: Various UX fixes
2 parents 5679d01 + 6dff879 commit b42bdf6

File tree

2 files changed

+10
-7
lines changed

2 files changed

+10
-7
lines changed

src/components/docs-previous-next-link.jsx

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,32 +19,35 @@ export default function DocsPreviousNextLinks({ routes }) {
1919
const nextPage = routes[currentIndex + 1];
2020

2121
return (
22-
<nav aria-label="pagination" className="mt-8 mb-4 flex justify-between">
22+
<nav
23+
aria-label="pagination"
24+
className="mt-8 mb-4 flex justify-between space-x-2"
25+
>
2326
{previousPage && (
2427
<Link
25-
className="max-w-[200px] font-normal text-gray-400 no-underline transition duration-200 ease-in hover:text-white focus:text-white"
28+
className="max-w-[50%] font-normal text-gray-400 no-underline transition duration-200 ease-in hover:text-white focus:text-white"
2629
href={previousPage.route}
2730
noDefaultStyles
2831
aria-label={`Go to previous page: ${previousPage.title}`}
2932
>
30-
<span className="mb-1 ml-10 block text-sm">Previous</span>
33+
<span className="mb-1 block text-sm sm:ml-10">Previous</span>
3134
<span>
32-
<ChevronLeftIcon className="inline h-5 w-10 pr-4 align-text-top" />
35+
<ChevronLeftIcon className="hidden h-5 w-10 pr-4 align-text-top sm:inline" />
3336
</span>
3437
<span className="text-white">{previousPage.title}</span>
3538
</Link>
3639
)}
3740
{nextPage && (
3841
<Link
39-
className="max-w-[200px] font-normal text-gray-400 no-underline transition duration-200 ease-in hover:text-white focus:text-white"
42+
className="max-w-[50%] font-normal text-gray-400 no-underline transition duration-200 ease-in hover:text-white focus:text-white"
4043
href={nextPage.route}
4144
noDefaultStyles
4245
aria-label={`Go to next page: ${nextPage.title}`}
4346
>
4447
<span className="mb-1 block text-sm">Next</span>
4548
<span className="text-white">{nextPage.title}</span>
4649
<span>
47-
<ChevronRightIcon className="inline h-5 w-10 pl-4 align-text-top" />
50+
<ChevronRightIcon className="hidden h-5 w-10 pl-4 align-text-top sm:inline" />
4851
</span>
4952
</Link>
5053
)}

src/components/heading.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ export default function Heading({ level, children, id, ...props }) {
1212
>
1313
<Link
1414
href={`#${id}`}
15-
className="no-underline transition-colors group-hover:text-blue-500"
15+
className="break-all no-underline transition-colors group-hover:text-blue-500"
1616
noDefaultStyles
1717
>
1818
{children}

0 commit comments

Comments
 (0)