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.

Sign up for GitHub

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

Jump to bottom

Sofia´s Recipe library #18

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

Sofia´s Recipe library #18

wants to merge 29 commits into from

Conversation

solen80a
Copy link

No description provided.

@solen80a
Created the basics files and HTML code
683d523
@solen80a
Updates to basic code to match the design. Added all the basic HTML a…
6b1e6a7 
…nd started with CSS.
@solen80a
Added javascript to change color styles on buttons. Added messages wh…
b81b22d 
…en clicking the sorting buttons.
@solen80a
Changes buttons to form. Updated javascript to diactivate forms reloa…
bf8694a 
…d to get the click event to stick. Also made som style to articles.
@solen80a
Fixed some style. Updated javascrip.
96fba67
@solen80a
changed fron textcontent to innerHTML to be able to use paragraphs fo…
aa5a07b 
…r the message.
@solen80a
Added CSS Grid to recipe cards, with the clamp adjustment. Also added…
ede0766 
… mediaqury for filter-sort grid to change between mobile/tablet and desktop/widescreen.
@solen80a
Published to netlify and added the link to the read me file.
438331c
@solen80a
Added mockup recipe array and added a forEach loop to place the recip…
52068d8 
…es in every card.
@solen80a
Combined my filter and sort change of color. Fixed some styling of my…
879ebbc 
… recipe cards.
@solen80a
Filter for diets are working and the button change color.
cc31f3e
@solen80a
Whn you click filter All button the whole array och recipes are shown.
f212204
@solen80a
Added a radom button, that show a random recipe. Added a button to di…
5b17aef 
…splay diet filter if no match and an if else statment if that happens.
@solen80a
Connected api.spoonacular.com and made the recipes from the API dipla…
862e7a8 
…y in my recipe cards.
@solen80a
Fixed cards to be responsive with the API recipes.
27e33bb
@solen80a
Radom button is now picking a radom recipe from the fetched API data.
91eb3a0
@solen80a
Filter on vegetarian = true from the fetched API data.
fe74e4c
@solen80a
Fixed the toggle/active state on clicked sort and filter buttons.
170b43a
@solen80a
Fixed the styling of my recipe cards.
f61204a
@solen80a
Fixed so all my filters work.
b7a5b84
@solen80a
Sorting on popularity, using spoonacularscore.
71200f4
@solen80a
Added some error handling for the API request.
3197d89
@solen80a
Fixed the daily API quata error.
9e17d84
@solen80a
Cleaned som unused code.
f93b1cf
oskarnordin
oskarnordin reviewed Mar 16, 2025
View reviewed changes
Copy link

@oskarnordin oskarnordin left a comment

Choose a reason for hiding this comment

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

Well done, I think there are some smaller changes that could take place to make it even more readable and understandable. But overall, very good job and smart solutions for the problems you have to solve.

@solen80a
Copy link
Author

Thank you Oskar and great suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oskarnordin oskarnordin oskarnordin left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@solen80a @oskarnordin