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

fix sidebar English language #286

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

noams24
Copy link
Contributor

@noams24 noams24 commented May 19, 2024

Issue #271

Copy link

vercel bot commented May 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
maakaf-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 8, 2024 1:29pm

const { theme } = useTheme();
const t = useTranslations('Components.sideBar');
const navigationItems: NavigationItems[] = [
Copy link
Collaborator

Choose a reason for hiding this comment

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

in this scope this array will be re declared on every render

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I know, but how can I use the "useTranslations" hook outside this scope?

`fixed inset-y-0 z-50 bg-lightBg dark:bg-darkBg border-l
border-l-blue-400 w-[75%] shadow-lg transform transition-transform
ease-in-out duration-300`,
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we not use dir here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It didn't worked so well using dir

Copy link
Collaborator

@Darkmift Darkmift left a comment

Choose a reason for hiding this comment

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

please address review.

@Darkmift
Copy link
Collaborator

@noams24 please resolve conflicts

@Darkmift
Copy link
Collaborator

@noams24 please update

@noams24
Copy link
Contributor Author

noams24 commented Jun 1, 2024

I still couldn't figured how to fix this error:
image

Maybe @Tamir198 knows how solve it?

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.

2 participants