Skip to content

Commit

Permalink
Resolve overlap issue of the last sidebar element with the footer me…
Browse files Browse the repository at this point in the history
…nu across all devices (#1608)

* Resolve overlap issue of the last sidebar element with the footer menu across all devices

* Ensure sidebar displays on large screens starting from 1024px screen-width

* removed mistakenly added vs-code-settings.json from the previous commit
  • Loading branch information
rajveeerr authored Dec 5, 2024
1 parent 330879e commit 99917c5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions src/components/Appbar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ export const Appbar = () => {
const toggleCollapse = () => setIsCollapsed(!isCollapsed);

const sidebarVariants = {
expanded: { width: '12vw' },
expanded: { width: '20vw' },
collapsed: { width: '4vw' },
};

Expand All @@ -66,7 +66,7 @@ export const Appbar = () => {
stiffness: 200,
damping: 20,
}}
className="fixed left-0 top-0 z-[999] hidden h-full flex-col border-r border-primary/10 bg-background dark:bg-background 2xl:flex"
className="fixed left-0 top-0 z-[999] hidden h-full flex-col border-r border-primary/10 bg-background dark:bg-background lg:flex"
>
<div className="flex h-full flex-col gap-4">
<div className="flex w-full items-center border-b border-primary/10 px-2 py-4">
Expand Down Expand Up @@ -97,7 +97,7 @@ export const Appbar = () => {
initial={{ y: 100 }}
animate={{ y: 0 }}
transition={{ duration: 0.3, ease: 'easeInOut' }}
className="fixed bottom-0 left-0 right-0 z-[999] 2xl:hidden"
className="fixed bottom-0 left-0 right-0 z-[999] lg:hidden"
>
<div className="flex items-center justify-around border-t border-primary/10 bg-background p-4 shadow-xl">
<SidebarItems items={menuOptions} isCollapsed={!isMediumToXL} />
Expand Down
2 changes: 1 addition & 1 deletion src/components/Sidebar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,7 @@ export function Sidebar({
<Accordion
type="multiple"
defaultValue={currentActiveContentIds.map((num) => `item-${num}`)}
className="w-full px-4 capitalize"
className="w-full px-4 capitalize pb-24"
>
{memoizedContent}
</Accordion>
Expand Down

0 comments on commit 99917c5

Please sign in to comment.