Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI ux update #152

Merged
merged 7 commits into from
Nov 16, 2022
Merged

UI ux update #152

merged 7 commits into from
Nov 16, 2022

Conversation

mouafus
Copy link
Collaborator

@mouafus mouafus commented Nov 7, 2022

What ?

  • Fix 404 redirecting when attempt to access some route usign drawer component

How ?

  • Remove the drawer component
  • Adding a layout for each page of the app, the layout contain header navigation

Why

  • POV of UX/UI, drawer component is not realy necessary in the app. the navigation menu length is just 2, drawer can be ignore.

@Asam237
Copy link
Member

Asam237 commented Nov 7, 2022

@mouafus

simplescreenrecorder-2022-11-07_14.58.20.mp4


export const NotFound = () => {
const {t} = useTranslation();
console.log("ici")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove console log

sizeLarge: {
height: 42,
},
// contained
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove comment

@Asam237
Copy link
Member

Asam237 commented Nov 7, 2022

@mouafus

Looks great

I left you some comments and it is necessary to fix the bug concerning the route on the details of an alert on a region and change the port in config of the frontend 5000 instead of 8000

simplescreenrecorder-2022-11-07_18.43.32.mp4

@Asam237 Asam237 self-requested a review November 7, 2022 18:25
@Asam237 Asam237 linked an issue Nov 7, 2022 that may be closed by this pull request
@Zaker237 Zaker237 linked an issue Nov 12, 2022 that may be closed by this pull request
6 tasks
@mouafus mouafus mentioned this pull request Nov 16, 2022
…to ui_ux_update

� Conflicts:
�	frontend/src/config.ts
�	frontend/src/pages/404.tsx
�	frontend/src/pages/advices/Advices.tsx
�	frontend/src/pages/detail/Detail.page.tsx
�	frontend/src/translations/fr.ts
@Asam237 Asam237 merged commit d19f1c9 into osscameroon:main Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UI/UI improvments, code optimisation
2 participants