Skip to content

Commit

Permalink
fix: update UI based on the docs reviews (#47)
Browse files Browse the repository at this point in the history
* fix: 🐛 remove integrations

* fix: 🐛 remove global search bar

* fix: 🐛 remove changelog

* fix: 🐛 dataset overview text
  • Loading branch information
Aydawka authored Jan 17, 2025
1 parent e213ea9 commit 23c6ce8
Show file tree
Hide file tree
Showing 9 changed files with 5 additions and 101 deletions.
2 changes: 1 addition & 1 deletion mermaid/svg/dataset-metadata-er.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion mermaid/svg/er.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion mermaid/svg/study-metadata-er.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
20 changes: 0 additions & 20 deletions src/components/header/AppHeader.vue
Original file line number Diff line number Diff line change
Expand Up @@ -26,20 +26,6 @@ const profileOptions = [
{ default: () => "View Profile" }
),
},
{
icon: renderIcon("carbon:chart-network"),
key: "integrations",
label: () =>
h(
RouterLink,
{
to: {
path: "/integrations",
},
},
{ default: () => "Integrations" }
),
},
{
key: "d1",
type: "divider",
Expand Down Expand Up @@ -74,12 +60,6 @@ const handleSelect = (key: string | number) => {
</RouterLink>

<n-space justify="space-between" align="center">
<n-input size="large" placeholder="Search">
<template #suffix>
<Icon icon="ion:search-sharp" />
</template>
</n-input>

<div class="flex items-center justify-center space-x-3">
<RouterLink to="/auth/login" v-if="!authStore.isAuthenticated">
<n-button type="primary">
Expand Down
14 changes: 0 additions & 14 deletions src/components/sidebar/AppSidebar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -242,20 +242,6 @@ const lowerMenuOptions: MenuOption[] = [
{ default: () => "Documentation" }
),
},
{
icon: renderIcon("tabler:history"),
key: "changelog",
label: () =>
h(
RouterLink,
{
to: {
path: "/help/changelog",
},
},
{ default: () => "Changelog" }
),
},
],
icon: renderIcon("tabler:help"),
key: "help",
Expand Down
12 changes: 0 additions & 12 deletions src/router/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,10 @@ import UserProfile from "@/views/account/UserProfile.vue";
import LoginPage from "@/views/auth/LoginPage.vue";
import LogOut from "@/views/auth/LogOut.vue";
import SignUp from "@/views/auth/SignUp.vue";
import Changelog from "@/views/help/ChangeLog.vue";
import Documentation from "@/views/help/DocumentationPage.vue";
import HelpPage from "@/views/help/HelpPage.vue";
import ReportIssue from "@/views/help/ReportIssue.vue";
import HomePage from "@/views/home/HomePage.vue";
import Integrations from "@/views/integrations/IntegrationPage.vue";
import AllSettings from "@/views/settings/AllSettings.vue";
import AllStudies from "@/views/studies/AllStudies.vue";
import StudyContributors from "@/views/study/contributors/StudyContributors.vue";
Expand Down Expand Up @@ -523,11 +521,6 @@ const router = createRouter({
path: "documentation",
component: Documentation,
},
{
name: "changelog",
path: "changelog",
component: Changelog,
},
{
name: "report-issue",
path: "report-issue",
Expand All @@ -541,11 +534,6 @@ const router = createRouter({
path: "/profile",
component: UserProfile,
},
{
name: "integrations",
path: "/integrations",
component: Integrations,
},
{ name: "not-found", path: "/:pathMatch(.*)", component: NotFound },
],
});
Expand Down
20 changes: 0 additions & 20 deletions src/views/help/ChangeLog.vue

This file was deleted.

30 changes: 0 additions & 30 deletions src/views/integrations/IntegrationPage.vue

This file was deleted.

4 changes: 2 additions & 2 deletions src/views/study/datasets/AllDatasets.vue
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ const navigateToDataset = (datasetId: string) => {
<template>
<main class="flex h-full w-full flex-col pr-6">
<n-space justify="space-between">
<h2>All Datasets</h2>
<h2>Datasets</h2>

<RouterLink
:to="{
Expand Down Expand Up @@ -90,7 +90,7 @@ const navigateToDataset = (datasetId: string) => {
:width="150"
/>

<p class="text-center">We could not find a previously published dataset.</p>
<p class="text-center">We could not find any datasets for this study.</p>
</div>

<div v-else>
Expand Down

0 comments on commit 23c6ce8

Please sign in to comment.