Skip to content

Commit aa59e7a

Browse files
committed
Separate Question ID vs Question Number
Separate to allow for Unique IDs and displaying of question numbers in ascending order
1 parent 5b4f5b3 commit aa59e7a

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

peer-prep/src/Components/QuestionList/Question.jsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ export const Question = ({question, i, deleteQuestion, updateQuestion}) => {
3535
<div className={isEdit ? "container": "question"}>
3636
<div className="q-header" onClick = {() => toggle(i)}>
3737
<div className="q-name">
38-
<div className={isEdit ? "q-form-id": "q-id"}>#{question.id}</div>
38+
<div className={isEdit ? "q-form-id": "q-id"}>#{i + 1}</div>
3939
{isEdit === true? <input type="text" className="q-form-title q-form-input" defaultValue = {question.title}
4040
onChange = {(e) => {setTitle(e.target.value)}}/> :
4141
<div className="q-title">{question.title}</div>}
@@ -94,7 +94,7 @@ export const Question = ({question, i, deleteQuestion, updateQuestion}) => {
9494
}}>Submit</button> </div> : null}
9595

9696
</div>
97-
<div className="delete-btn" onClick = {(e) => deleteQuestion(i)}>
97+
<div className="delete-btn" onClick = {(e) => deleteQuestion(question.id)}>
9898
<img src= {delete_icon} alt="" />
9999
</div>
100100
</div>

peer-prep/src/Components/QuestionList/QuestionForm.jsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import React, { useState } from 'react'
22
import './QuestionForm.css'
33
import delete_icon from '../Assets/bin.png'
44

5-
export const QuestionForm = ({qId, addQuestion, setAddQ, setQId}) => {
5+
export const QuestionForm = ({qId, addQuestion, setAddQ, setQId, questionNumber}) => {
66
const [title, setTitle] = useState("");
77
const [difficulty, setDifficulty] = useState("");
88
const [topic, setTopic] = useState("");
@@ -17,7 +17,7 @@ export const QuestionForm = ({qId, addQuestion, setAddQ, setQId}) => {
1717
<div className="form-container">
1818
<form className = "container" onSubmit={handleSubmit(setAddQ, addQuestion)}>
1919
<div className="q-form-header">
20-
<div className="q-form-id">#{qId}</div>
20+
<div className="q-form-id">#{questionNumber}</div>
2121
<input type="text" className="q-form-title q-form-input"
2222
placeholder='Question Title' onChange = {
2323
(e) => setTitle(e.target.value)

peer-prep/src/Components/QuestionList/QuestionList.jsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,13 +75,13 @@ export const Questions = () => {
7575

7676
<div className="q-wrapper">
7777
<div className="accordion">
78-
{qs.map((q, i) => (
79-
<Question key = {q[0]} question = {q[1]} i = {q[0]} deleteQuestion = {deleteQuestion} updateQuestion = {updateQuestion}/>
78+
{qs.map((q, index) => (
79+
<Question key = {index} question = {q[1]} i = {index} deleteQuestion = {deleteQuestion} updateQuestion = {updateQuestion}/>
8080
))}
8181
</div>
8282

8383
{isAddQ === false ? <div></div> :
84-
<QuestionForm qId = {qId} addQuestion = {addQuestion} setAddQ = {setAddQ} setQId = {setQId}/>}
84+
<QuestionForm questionNumber = {Array.from(database.database).length + 1} qId = {qId} addQuestion = {addQuestion} setAddQ = {setAddQ} setQId = {setQId}/>}
8585
<div className="add-q-btn" onClick = {() => {
8686
if (!isAddQ) {
8787
setQId(qId + 1);

0 commit comments

Comments
 (0)