Skip to content

Commit

Permalink
Fix project switcher for
Browse files Browse the repository at this point in the history
  • Loading branch information
benjackwhite committed Nov 7, 2023
1 parent 3dfc715 commit 9a3dfde
Show file tree
Hide file tree
Showing 6 changed files with 60 additions and 39 deletions.
48 changes: 48 additions & 0 deletions frontend/src/layout/GlobalModals.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
import { kea, path, actions, reducers, useActions, useValues } from 'kea'
import { CreateOrganizationModal } from 'scenes/organization/CreateOrganizationModal'
import { InviteModal } from 'scenes/organization/Settings/InviteModal'
import { CreateProjectModal } from 'scenes/project/CreateProjectModal'
import { inviteLogic } from 'scenes/organization/Settings/inviteLogic'

import type { globalModalsLogicType } from './GlobalModalsType'

export const globalModalsLogic = kea<globalModalsLogicType>([
path(['layout', 'navigation', 'globalModalsLogic']),
actions({
showCreateOrganizationModal: true,
hideCreateOrganizationModal: true,
showCreateProjectModal: true,
hideCreateProjectModal: true,
}),
reducers({
isCreateOrganizationModalShown: [
false,
{
showCreateOrganizationModal: () => true,
hideCreateOrganizationModal: () => false,
},
],
isCreateProjectModalShown: [
false,
{
showCreateProjectModal: () => true,
hideCreateProjectModal: () => false,
},
],
}),
])

export function GlobalModals(): JSX.Element {
const { isCreateOrganizationModalShown, isCreateProjectModalShown } = useValues(globalModalsLogic)
const { hideCreateOrganizationModal, hideCreateProjectModal } = useActions(globalModalsLogic)
const { isInviteModalShown } = useValues(inviteLogic)
const { hideInviteModal } = useActions(inviteLogic)

return (
<>
<InviteModal isOpen={isInviteModalShown} onClose={hideInviteModal} />
<CreateOrganizationModal isVisible={isCreateOrganizationModalShown} onClose={hideCreateOrganizationModal} />
<CreateProjectModal isVisible={isCreateProjectModalShown} onClose={hideCreateProjectModal} />
</>
)
}
2 changes: 2 additions & 0 deletions frontend/src/layout/navigation-3000/Navigation.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import { Scene, SceneConfig } from 'scenes/sceneTypes'
import { FlaggedFeature } from 'lib/components/FlaggedFeature'
import { FEATURE_FLAGS } from 'lib/constants'
import { SidePanel } from './sidepanel/SidePanel'
import { GlobalModals } from '../GlobalModals'

export function Navigation({
children,
Expand Down Expand Up @@ -48,6 +49,7 @@ export function Navigation({
</main>
<SidePanel />
<CommandPalette />
<GlobalModals />
</div>
)
}
4 changes: 3 additions & 1 deletion frontend/src/layout/navigation/OrganizationSwitcher.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ import { sceneLogic } from 'scenes/sceneLogic'
import { userLogic } from 'scenes/userLogic'
import { AvailableFeature, OrganizationBasicType } from '~/types'
import { navigationLogic } from './navigationLogic'
import { globalModalsLogic } from '../GlobalModals'

export function AccessLevelIndicator({ organization }: { organization: OrganizationBasicType }): JSX.Element {
return (
Expand Down Expand Up @@ -44,7 +45,8 @@ export function OtherOrganizationButton({
}

export function NewOrganizationButton(): JSX.Element {
const { closeSitePopover, showCreateOrganizationModal } = useActions(navigationLogic)
const { closeSitePopover } = useActions(navigationLogic)
const { showCreateOrganizationModal } = useActions(globalModalsLogic)
const { guardAvailableFeature } = useActions(sceneLogic)

return (
Expand Down
4 changes: 3 additions & 1 deletion frontend/src/layout/navigation/ProjectSwitcher.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ import { urls } from 'scenes/urls'
import { userLogic } from 'scenes/userLogic'
import { AvailableFeature, TeamBasicType } from '~/types'
import { navigationLogic } from './navigationLogic'
import { globalModalsLogic } from '../GlobalModals'

export function ProjectName({ team }: { team: TeamBasicType }): JSX.Element {
return (
Expand All @@ -25,7 +26,8 @@ export function ProjectSwitcherOverlay(): JSX.Element {
const { currentOrganization, projectCreationForbiddenReason } = useValues(organizationLogic)
const { currentTeam } = useValues(teamLogic)
const { guardAvailableFeature } = useActions(sceneLogic)
const { showCreateProjectModal, hideProjectSwitcher } = useActions(navigationLogic)
const { hideProjectSwitcher } = useActions(navigationLogic)
const { showCreateProjectModal } = useActions(globalModalsLogic)

return (
<div className="project-switcher-container">
Expand Down
23 changes: 4 additions & 19 deletions frontend/src/layout/navigation/TopBar/TopBar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,9 @@ import { Announcement } from './Announcement'
import { navigationLogic } from '../navigationLogic'
import { HelpButton } from 'lib/components/HelpButton/HelpButton'
import { CommandPalette } from 'lib/components/CommandPalette'
import { CreateOrganizationModal } from 'scenes/organization/CreateOrganizationModal'
import { InviteModal } from 'scenes/organization/Settings/InviteModal'
import { Link } from 'lib/lemon-ui/Link'
import { IconMenu, IconMenuOpen } from 'lib/lemon-ui/icons'
import { CreateProjectModal } from 'scenes/project/CreateProjectModal'
import './TopBar.scss'
import { inviteLogic } from 'scenes/organization/Settings/inviteLogic'
import { UniversalSearchPopover } from 'lib/components/UniversalSearch/UniversalSearchPopover'
import { TaxonomicFilterGroupType } from 'lib/components/TaxonomicFilter/types'
import { groupsModel } from '~/models/groupsModel'
Expand All @@ -20,20 +16,11 @@ import { featureFlagLogic } from 'lib/logic/featureFlagLogic'
import { FEATURE_FLAGS } from 'lib/constants'
import ActivationSidebarToggle from 'lib/components/ActivationSidebar/ActivationSidebarToggle'
import { NotebookButton } from '~/layout/navigation/TopBar/NotebookButton'
import { GlobalModals } from '~/layout/GlobalModals'

export function TopBar(): JSX.Element {
const {
isSideBarShown,
noSidebar,
minimalTopBar,
mobileLayout,
isCreateOrganizationModalShown,
isCreateProjectModalShown,
} = useValues(navigationLogic)
const { toggleSideBarBase, toggleSideBarMobile, hideCreateOrganizationModal, hideCreateProjectModal } =
useActions(navigationLogic)
const { isInviteModalShown } = useValues(inviteLogic)
const { hideInviteModal } = useActions(inviteLogic)
const { isSideBarShown, noSidebar, minimalTopBar, mobileLayout } = useValues(navigationLogic)
const { toggleSideBarBase, toggleSideBarMobile } = useActions(navigationLogic)
const { groupNamesTaxonomicTypes } = useValues(groupsModel)
const { featureFlags } = useValues(featureFlagLogic)

Expand Down Expand Up @@ -96,9 +83,7 @@ export function TopBar(): JSX.Element {
</div>
</header>
<CommandPalette />
<InviteModal isOpen={isInviteModalShown} onClose={hideInviteModal} />
<CreateOrganizationModal isVisible={isCreateOrganizationModalShown} onClose={hideCreateOrganizationModal} />
<CreateProjectModal isVisible={isCreateProjectModalShown} onClose={hideCreateProjectModal} />
<GlobalModals />
</>
)
}
18 changes: 0 additions & 18 deletions frontend/src/layout/navigation/navigationLogic.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,6 @@ export const navigationLogic = kea<navigationLogicType>([
openSitePopover: true,
closeSitePopover: true,
toggleSitePopover: true,
showCreateOrganizationModal: true,
hideCreateOrganizationModal: true,
showCreateProjectModal: true,
hideCreateProjectModal: true,
toggleProjectSwitcher: true,
hideProjectSwitcher: true,
openAppSourceEditor: (id: number, pluginId: number) => ({ id, pluginId }),
Expand Down Expand Up @@ -95,20 +91,6 @@ export const navigationLogic = kea<navigationLogicType>([
toggleSitePopover: (state) => !state,
},
],
isCreateOrganizationModalShown: [
false,
{
showCreateOrganizationModal: () => true,
hideCreateOrganizationModal: () => false,
},
],
isCreateProjectModalShown: [
false,
{
showCreateProjectModal: () => true,
hideCreateProjectModal: () => false,
},
],
isProjectSwitcherShown: [
false,
{
Expand Down

0 comments on commit 9a3dfde

Please sign in to comment.