Skip to content
OverviewSolutions
6
Comments
8
Tamana
@Tamana123-2

All comments

  • Freshta Nazari•80
    @FreshtanazariSubmitted over 1 year ago

    any feedbacks would be appreciated, this is my second solution though i just edited some parts.

    CSS, HTML

    1
    Tamana•210
    @Tamana123-2Posted over 1 year ago

    Hi! Your shadow is alittle too much. hhhh Nice time friend.

    Marked as helpful
    0
  • Freshta Nazari•80
    @FreshtanazariSubmitted over 1 year ago

    hi everyone! would appreciate any constructive feedback.

    HTML, CSS

    3
    Tamana•210
    @Tamana123-2Posted over 1 year ago

    Hi Freshta Nazari! Great job for completing the project. The height of container is larger than the exact design, plus it is not centered. your background-color has not been set. Not perfect, but you tried hard enough, see your project better next time. Nice time.

    Marked as helpful
    2
  • Michael•240
    @mksoofianSubmitted over 1 year ago

    I was overthinking about the layering of objects in this project.

    I was trying initially to break the image, header and paragraph into three boxes and set them to fill the card in a ratio proportion but wasnt having luck with that. Especially because I thought this would be responsive design and I would be able to just include a max-width on the card to keep it from getting too large. However, I wasn't having any luck with that.

    Would someone be willing to share how I could have accomplished that?

    Basic HTML - QR Code

    2
    Tamana•210
    @Tamana123-2Posted over 1 year ago

    Hi mksoofian! Great job for completing the project. Although I wanted to check you code, but I could not find the CSS file. For the whole design you can get a container, so that you can set the padding, background-color margin ..., then one div for the image, one div for the first text and finally one div for the second context. As your first project, it is a great attempt. Keep moving forward. Nice time!

    Marked as helpful
    1
  • Tamana•210
    @Tamana123-2Posted over 1 year ago

    Hi! Good job for completing the challenge, but remember to start the project from mobile design. First you could get display grid for being more flexible on desktop version as well; you can write all the code needed for mobile version, when it comes for desktop design you could use media quarries and set the (grid-template-columns: 1 fr fr) to make the lay out side by side. I also did the project if you want you can have a look at. Nice time.

    Marked as helpful
    0
  • bOREDpANDA69•70
    @bOREDpANDA69Submitted over 1 year ago

    Hey everyone,

    This is my first ever project, I would like to know what I could do better, any response will be of great help.

    Thanks you

    Responsive Product Preview Page

    1
    Tamana•210
    @Tamana123-2Posted over 1 year ago

    Hi bOREDpANDA69! Hope you are doing great. Before designing the layout you could have a look at design guide, there is every color used in the project. plus there is opacity or the font-color for the content. you can compare your design with the exact project here. Nice time.

    1
  • Tamana•210
    @Tamana123-2Posted over 1 year ago

    Hi pineapplegoose! Hope you are doing great. Everything is okay in the project you've done, but as I believe your font had not been set, since you got to import the font with it's weight from google plus it need single quotation mark. like this one @import url('https://fonts.googleapis.com/css2?family=Hanken+Grotesk:wght@500;700;800&display=swap'); and in the CSS file you can write (font-family: 'Hanken Grotesk'), plus you used box-shadow for three side of summary part although it is for two part. Keep going not for becoming perfect but brave enough to try every day for what you want. Hope it was helpful. Nice time.

    Marked as helpful
    1
  • Tamana•210
    @Tamana123-2Posted over 1 year ago

    Hi generieyyc ! Hope you are doing great. You did well in this project, although there are few flaws. the whole content and button does not have border you can set (border: none) font-size of order summary and paragraph is smaller. the line under change is in hover mode, plus you can set div for (icon-plan-change) to flex. and each of them again in separate dives with display block so you could edit the space between them. Button have shadow and cancel order got to be gray with weight. I am not perfect either, but lets keep trying more and more to update our skill and improve day by day, Don't give up. Nice time.

    Marked as helpful
    0
  • Christopher Portorreal•70
    @crixferSubmitted over 1 year ago

    Centering and putting the image aside of the white box, then centering the text in the right-sided white box.

    I tried adding Bootstrap coding, but it broke my codes every time. I need more practice.

    What do you recommend to do these kinds of positioning and sizing without using so many codes? I really appreciate any help you can provide.

    Display Flex centering Perfume challenge

    4
    Tamana•210
    @Tamana123-2Posted over 1 year ago

    Hello Mr. Christopher Portorreal! Great job for completing the challenge. In your mobile design you could use grid, for being more flexible with desktop version instead of flex. You can add the image of desktop design in HTML file with a media of (min-width: 600px) and in you CSS file you could go with (grid-template-columns: 1fr 1fr) for making the perfume image and content side by side. Go ahead even if it is hard to get. Nothing is easy to achieve. Nice time.

    Marked as helpful
    1