-
Notifications
You must be signed in to change notification settings - Fork 16
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
Added chose category dropdown for new quiz #1389
Added chose category dropdown for new quiz #1389
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a picture or video of how it works
src/containers/my-quizzes/edit-quiz-container/EditQuizContainer.styles.ts
Outdated
Show resolved
Hide resolved
src/containers/my-quizzes/edit-quiz-container/EditQuizContainer.styles.ts
Outdated
Show resolved
Hide resolved
src/containers/my-quizzes/edit-quiz-container/EditQuizContainer.tsx
Outdated
Show resolved
Hide resolved
src/containers/my-quizzes/edit-quiz-container/EditQuizContainer.tsx
Outdated
Show resolved
Hide resolved
src/containers/my-quizzes/edit-quiz-container/EditQuizContainer.tsx
Outdated
Show resolved
Hide resolved
Fix lint please |
src/containers/my-quizzes/edit-quiz-container/EditQuizContainer.tsx
Outdated
Show resolved
Hide resolved
"successAddedOuiz": "Ouiz was successtully created" | ||
"successAddedOuiz": "Ouiz was successtully created", | ||
"categoryDropdown": "Category...", | ||
"categoryDropdownHelper": "Choose the category:" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe better to move these translations to "common.json"? Because we need this for other pages.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
position: 'absolute', | ||
top: '-33px', | ||
left: '-14px' | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are you using this variable? You use this style only once, so you can just write this in the 84-line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -17,6 +17,8 @@ export interface QuizContentProps { | |||
setDescription: Dispatch<SetStateAction<CreateQuizParams['description']>> | |||
questions: Question[] | |||
setQuestions: Dispatch<SetStateAction<Question[]>> | |||
category: string | null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe it's better to use category: Category | null?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
c6b3e82
to
000a245
Compare
new_commit
dbb9326
to
c38ad42
Compare
@@ -37,6 +37,8 @@ | |||
"fileSizeError": "Size for one file cannot be more than 10 MB.", | |||
"noItems": "Елементів не знайдено", | |||
"view": "Переглянути", | |||
"categoryDropdown": "Категорія...", | |||
"categoryDropdownHelper": "Оберіть категорію:", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't need this anymore. Delete this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
SonarCloud Quality Gate failed. 0 Bugs 75.0% Coverage Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Added chose category dropdown for new quiz
2023-11-22.14.35.50.mov