Skip to content

Commit

Permalink
Merge pull request #364 from ably/meganav-fix-and-spacing-utilities
Browse files Browse the repository at this point in the history
Add basic spacing utilities and fix meganav/slider alignments
  • Loading branch information
jamiehenson authored Jun 10, 2024
2 parents 745e302 + b9e64f8 commit ae71438
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 6 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@ably/ui",
"version": "14.0.3",
"version": "14.0.3-dev.7b5c832",
"description": "Home of the Ably design system library ([design.ably.com](https://design.ably.com)). It provides a showcase, development/test environment and a publishing pipeline for different distributables.",
"repository": {
"type": "git",
Expand Down
2 changes: 1 addition & 1 deletion src/core/MeganavItemsMobile.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ const MeganavItemsMobile = ({
const classNames = `ui-meganav-link ${theme.textColor}`;

return (
<ul className="flex md:hidden" data-id="meganav-items-mobile">
<ul className="flex md:hidden items-center" data-id="meganav-items-mobile">
<li>
{sessionState?.signedIn && sessionState?.logOut ? (
<SignOutLink absUrl={absUrl} {...sessionState.logOut}>
Expand Down
4 changes: 2 additions & 2 deletions src/core/Slider.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ const Slider = ({ children, options }: SliderProps) => {
} pointer-events-auto rounded border border-mid-grey hover:border-active-orange flex justify-center items-center ui-icon-cta ui-icon-cta-left`}
onClick={prev}
>
<div className="ui-icon-cta-holder flex gap-4">
<div className="ui-icon-cta-holder flex w-48">
<div className="w-full h-full flex-shrink-0 flex items-center justify-center">
<Icon name="icon-gui-arrow-left" size="1.5rem" />
</div>
Expand All @@ -207,7 +207,7 @@ const Slider = ({ children, options }: SliderProps) => {
} pointer-events-auto rounded border border-mid-grey hover:border-active-orange justify-center items-center ui-icon-cta ui-icon-cta-right`}
onClick={next}
>
<div className="ui-icon-cta-holder flex gap-4">
<div className="ui-icon-cta-holder flex w-48">
<div className="w-full h-full flex-shrink-0 flex items-center justify-center">
<Icon name="icon-gui-arrow-right" size="1.5rem" />
</div>
Expand Down
4 changes: 2 additions & 2 deletions src/core/Slider/component.css
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@

@screen sm {
.ui-icon-cta-left:hover .ui-icon-cta-holder {
transform: translateX(-120%);
transform: translateX(-100%);
}
.ui-icon-cta-right .ui-icon-cta-holder {
transform: translateX(-120%);
transform: translateX(-100%);
}
.ui-icon-cta-right:hover .ui-icon-cta-holder {
transform: translateX(0%);
Expand Down
8 changes: 8 additions & 0 deletions src/core/styles.components.css
Original file line number Diff line number Diff line change
Expand Up @@ -32,4 +32,12 @@
margin-left: 3px;
font-size: 8px;
}

.ui-section-spacing {
@apply py-80 sm:py-96 md:py-128;
}

.ui-divider-spacing {
@apply my-20 sm:my-24 md:my-32;
}
}

0 comments on commit ae71438

Please sign in to comment.