Can you share what you liked about it and what could be improved for next time? Your feedback will help me grow and develop my skills. Thank you so much for your time.
Alucard
@Alucard2169All comments
- @5leepySubmitted almost 2 years ago@Alucard2169Posted almost 2 years ago
hi, you did a great job in this challenge, there are some issues, one of them is the share button (when the user clicks the share button , it opens up the share menu, but if the user decides to not use it, they would have to reload the browser to close the share menu)
if added a state functionality, it will keep the current state of the share menu ( if it's open or not), i have explained it in the pull request on github.
hope it help, if you have any other issues, just ask
keep it up
Marked as helpful1 - @PepziniSubmitted almost 2 years ago
- What I found difficult is setting the text at the bottom
- The footer
- None
@Alucard2169Posted almost 2 years agohi, i fixed some of the issues, i have explained them in the pull request, hope it helps
avoid using inline css
overall you did a very good job
keep it up
0 - @msabdalaalSubmitted about 2 years ago@Alucard2169Posted about 2 years ago
Hi, Great job on the design,
next you should try to make it interactive and functional, as it will give you tons to learn on the way.
Here is how you can fix your HTML and Accessibility errors:
-
you should always change heading size by one, for example ( a page must have a H1 heading (very important), after that you should decrease the size by one like h2 > h3 > h4 ...., don't just jump to h3 or h4 );
-
Input element must have a label all times, here is how you can do that
<label for="{your input's ID}"> *some_text* </label> <input id="{your input's ID}" >
hope that helps
Marked as helpful1 -
- @yurideoliveira2712Submitted about 2 years ago
Primeiro projeto. Iniciando no mundo de programação.
@Alucard2169Posted about 2 years agohey @yurideoliveira,
Great job you design looks great,
to solve the accessibility issue, you should use at least one <h1> element in you page.
Happy coding!
Marked as helpful0 - @luisoliverosrdnSubmitted about 2 years ago
Really proud of this one! I learned that sometimes less is more, I tried to add complexity to the code as needed, not the other way around.
Any feedback or suggestion of how to improve the code is more than welcome!
Thanks!
@Alucard2169Posted about 2 years agohi @luisoliverosrdn,
You did a awesome job.
- here are some points to help you:-
-- use
display: flex
and a flex-direction of column with height 100vh on body, to center the card perfectly.-- you don't have to use
alt
attribute on anchor tag.happy coding!
Marked as helpful0 - @Halix7Submitted over 2 years ago
That was eazy .
@Alucard2169Posted over 2 years agoHi,
-
Design look good, but a bit too small, so you should work on the dimensions to make it more visible and stand out on the screen.
-
The formula you used to calculate the tip and total amount aren't working properly, you can look up online to see the correct formula for tip-calculator.
-
The reset button is suppose to clear every input and amount display, and bring everything back to square one. You should work on that.
-
People input isn't suppose to take negative or fraction values.
-
Bill input and custom tip input isn't suppose to take negative values.
-
Using attributes like
hiddenheading
andzero-people
on html element is not valid, if you want to set something to hidden you can use CSS or JS for it.
Best of Luck
0 -
- @NazemrapSubmitted over 2 years ago
Got some difficulties to align properly the white box to the border: div { display: block; margin-left: auto; margin-right: auto; margin-top: 15%; } I did that way but it might not be the most efficiant.
@Alucard2169Posted over 2 years agohi @Nazemrap great work.
For centering the card property with the body
you can put your card in a container and then use
display: flex
on the body, remember to set a height on your body of 100vh and then usejustify-content
andalign-items
property.Flexbox and grid are very important to make responsive designs. and they also make the work a lot easier.
here this video gives a great understanding of flexbox
Marked as helpful0 - @ojitxslmlSubmitted over 2 years ago
Hello!
Question
-How can I complete the action in:
<form action=``>
?Any feedback about how to improve my code is welcome! 🙌
@Alucard2169Posted over 2 years agoHi @ojitxslml nice work, one thing you can add is a error when you type. Fraction in people count input cause people can't be in fraction as you know, all in all its a nice project.
Also you can work on the sizing cause if the user types a big enough number the while calculator exeeds it's size limit in mobile version.
Keep up the good work
1 - @FelipeFamaSubmitted almost 3 years ago@Alucard2169Posted almost 3 years ago
Hi there @FelipeFama Great job on this project 🎉🎉🎉
Here are a few things you can improve:-
-
Your card is a bit too big in height, you can work on that.
-
That will solve the scrolling issue, if it doesn't just use
overflow:hidden
on the body, but make sure that your card is fully visible. -
To center your card you can use
display:flex
on body -
Also keep in mind that you can't actually write your content directly inside the
<div>
tag, div tag is use to divide our content into parts, you have to write your content inside tags like<p>,<h1>,<article>,<aside>
which are made to store content.
Nice project, keep it up 😊😊😊
Marked as helpful1 -
- @agaushaSubmitted almost 3 years ago
Any feedback will be appreciated, thank you.
@Alucard2169Posted almost 3 years agoHi @agausha great job finishing this challenge 🎉🎉🎉🎉
There are a few things you can improve:-
-
when you open your site, the card ( the whole ) isn't centered completely. You can use flex on your body and then center your card using that.
-
To fix your accessibility issues just do
<main></main>
instead of<div class="main">
and remember to create a new report 😊😊.
You did Great, Keep it up 😊😊😊
Marked as helpful1 -
- @azenetescSubmitted almost 3 years ago@Alucard2169Posted almost 3 years ago
@azenetesc really awesome work 🥳🥳🥳
You can make it better by adding a little shadow to your card
Keep up the good work 😊😊😊
Marked as helpful0 - @Rashmi-WijesekaraSubmitted almost 3 years ago@Alucard2169Posted almost 3 years ago
Hi there @Rashmi-Wijesekara Nice work on this project 🎉🎉🎉
It looks good so far,
-
Next thing you can do is, add a hover effect on the image.
-
The mobile version is not completely centered (Vertically), so you can do something about that.
-
The Desktop version is too close to the top, you can either use flex on body or just add some margin-top to the card, but i would suggest using flex.
-
there are some issues with the accessibility and that's cause
-
in your HTML file, a image tap must have an
alt
attribute. It will describe the image to the client in case the image is not loading.
I hope this helps you.
Nice work, Keep It Up 🥳🥳🥳
Marked as helpful1 -