Closed Mohamilr closed 3 years ago
✔️ Deploy Preview for grand-resume canceled.
🔨 Explore the source changes: deaf545f789800f662ef6e8460e9345327dec938
🔍 Inspect the deploy log: https://app.netlify.com/sites/grand-resume/deploys/61718b87c6abc20007764056
Good, this works functionally well
I'm thinking we could change the behaviour of the delete button appearing on all first sections i.e we should have at least one form section of the different parts of the CV to be visible.
an additional check like this could do.
setRecord({
...record,
education: record.education.length > 1 && record.education.filter( // d
(education, idx) => idx !== id
),
});
Changes made
What do you think about the layout on the languages when the button is added
Changes made
What do you think about the layout on the languages when the button is added
Looks a bit odd, but since the language can be an optional field I guess we can add the delete button to the first item
Changes made What do you think about the layout on the languages when the button is added
Looks a bit odd, but since the language can be an optional field I guess we can add the delete button to the first item
Does it mean the language section will no longer appear when the inputs are deleted or will it still remain.
Changes made What do you think about the layout on the languages when the button is added
Looks a bit odd, but since the language can be an optional field I guess we can add the delete button to the first item
Does it mean the language section will no longer appear when the inputs are deleted or will it still remain.
It can still appear would just show a placeholder e.g "No Available Languages"
Done @adebayoileri
You can delete entries now