Skip to content
This repository was archived by the owner on Mar 21, 2026. It is now read-only.

Qr code component#165

Open
mbashia wants to merge 4 commits intooptimumBA:qr_code_componentfrom
mbashia:qr_code_component
Open

Qr code component#165
mbashia wants to merge 4 commits intooptimumBA:qr_code_componentfrom
mbashia:qr_code_component

Conversation

@mbashia
Copy link
Copy Markdown

@mbashia mbashia commented Aug 12, 2024

No description provided.

Copy link
Copy Markdown
Member

@almirsarajcic almirsarajcic left a comment

Choose a reason for hiding this comment

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

The tests are failing.
Also, you've committed and pushed some unnecessary files like READMEs, extra images, etc.

Comment thread challenges/index.html Outdated
Comment thread challenges/qr_code_component_main/README-template.md Outdated
Comment thread challenges/qr_code_component_main/README.md Outdated
Comment thread challenges/qr_code_component_main/style-guide.md Outdated
Comment thread tailwind.config.js
Comment thread src/styles.css Outdated
Comment thread package.json Outdated
@mbashia mbashia requested a review from almirsarajcic August 13, 2024 12:47
Copy link
Copy Markdown
Member

@almirsarajcic almirsarajcic left a comment

Choose a reason for hiding this comment

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

The tests are still failing (https://github.com/optimumBA/frontend_mentor_challenges/actions/runs/10369974217/job/28717872421?pr=165). Execute npm run test locally to run the tests.

Files are still in the wrong location, and also there are still some extra files.

Comment thread challenges/index.html Outdated
@mbashia
Copy link
Copy Markdown
Author

mbashia commented Aug 13, 2024

The tests are still failing (https://github.com/optimumBA/frontend_mentor_challenges/actions/runs/10369974217/job/28717872421?pr=165). Execute npm run test locally to run the tests.

Files are still in the wrong location, and also there are still some extra files.

My test are passing locally

@mbashia
Copy link
Copy Markdown
Author

mbashia commented Aug 13, 2024

Let me go through it again, and correct where necessary

@mbashia mbashia force-pushed the qr_code_component branch from 64bcbe1 to 71e3581 Compare August 14, 2024 11:48
@mbashia mbashia requested a review from almirsarajcic August 14, 2024 12:06
Copy link
Copy Markdown
Member

@almirsarajcic almirsarajcic left a comment

Choose a reason for hiding this comment

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

Much better, but it doesn't match the design exactly (paddings, margins, line height etc.).
https://www.frontendmentor.io/challenges/qr-code-component-iux_sIO_H
Screenshot 2024-08-15 at 12 18 13

@mbashia
Copy link
Copy Markdown
Author

mbashia commented Aug 15, 2024

Much better, but it doesn't match the design exactly (paddings, margins, line height etc.). https://www.frontendmentor.io/challenges/qr-code-component-iux_sIO_H Screenshot 2024-08-15 at 12 18 13

Improving this on the next four exercises

@mbashia mbashia requested a review from almirsarajcic August 20, 2024 03:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants