Skip to content
OverviewSolutions
15
Comments
9
Muhammad Saad
@Arman001

All comments

  • Brian•140
    @NoxlobinSubmitted about 1 year ago

    Hi everyone!

    I haven't really had any problems in this. However, I am unsure of the coding practices I used on this one. Please give me feedback on that.

    Profile card component

    2
    Muhammad Saad•280
    @Arman001Posted about 1 year ago

    Hi!

    Good work. It is not scaling down properly and on mobile it gets very compressed because of your auto scaling may be.

    Regards

    Saad

    0
  • Muhammad Saad•280
    @Arman001Posted about 1 year ago

    Hi!

    It is great work. If you want the hover effect on the question answer both then you can wrap them in a div and put hover effect on that. Otherwise on question it is working fine.

    Regards

    Saad

    0
  • Luis Herrera•795
    @Luis15HerrSubmitted about 1 year ago

    Hello everyone! 👋

    I'm thrilled to introduce my latest project, the 'Results Summary Component.' This marks my second project after a considerable break from development. As I dived back into coding, I focused on refining my implementation of the BEM methodology, and I'm eager to showcase my progress.😁

    If you have any questions or suggestions on how to enhance any aspect of the code, please don't hesitate to reach out. Your insights are invaluable to me, and I welcome any feedback with open arms.

    Thank you for taking the time to review the code, and I appreciate your support. Here's to a future of happy coding for all! 🚀

    Newsletter sign-up form with success message - Frontend Mentor

    1
    Muhammad Saad•280
    @Arman001Posted about 1 year ago

    Hi!

    Great work. Well designed and nice transitions. There is a small issue as you are not properly hiding your success_form , its dismiss button start showing at top when you are in the size 700 to 800. You can hide the whole form and show on button click with js. I hope it helps.

    Regards #Saad

    Marked as helpful
    1
  • Muhammad Saad•280
    @Arman001Posted about 1 year ago

    As @abhishek-baliyan-dev said, tailwind work for you better only if you knew vanilla css. Because until someone don;t what is padding, margin, flexbox, grid etc. It will not work. But which is the better one? If you have some practice with CSS, you can get used to tailwind easily. It will make things a lot faster without any doubt.

    Regards# Saad

    0
  • Muhammad Saad•280
    @Arman001Posted about 1 year ago

    Great work! It is not working on screens greater than 1440 due to your constraint during linking style sheet. I think it would be better if you remove it so it will also work on screens of greater sizes.

    Regards

    #SAAD

    0
  • Muhammad Saad•280
    @Arman001Posted about 1 year ago

    Hi!

    Great work there. It can be improved if you make all the content bigger so it will look better and easily readable. Following are links to my solved challenge:

    Netlify Live

    GitHub Repo

    I hope these will be helpful. Happy Designing and Coding.

    Regards: Saad

    0
  • Aisha Mohammed•30
    @ScazeeSubmitted about 1 year ago

    Hello everyone,

    I need some help. For some reason, for my .for-padding, I could not use justify-content or align-content. I had to use gap to separate them. I am not sure why. Maybe you can help me figure it out.

    'Stats page using flexbox

    2
    Muhammad Saad•280
    @Arman001Posted about 1 year ago

    Hi! Good Work there... Your problem was the usage of an extra div this is where i removed the extra div and changed class .remove_padding to .card_content. I also removed the top div but you can add it if you want but then it will be consider as a single element unlike separate h1 and p.

    <div class="card-container">
          <div class="card-content">
              <h1>Get <span>insights</span> that help your business grow.</h1>

              <p>
                Discover the benefits of data analytics and make better
                decisions regarding revenue, customer experience, and overall
                efficiency.
              </p>

            <div class="flex-container stats">
              <div class="stats-one">
                <h2>10k+</h2>
                <p class="stats-p">companies</p>
              </div>

              <div class="stats-two">
                <h2>314</h2>
                <p class="stats-p">templates</p>
              </div>

              <div class="stats-three">
                <h2>12m+</h2>
                <p class="stats-p">queries</p>
              </div>
            </div>
            
          </div>
        
        
        <div class="p-flex-item part-two">
          <img src="./images/image-header-desktop.jpg" class="image" ; />
        </div>
      </div>

    CSS is of .card-content and image is as follow where i changed flex:1; because it will give equal size both to image and .card-content.

    .image {
  width: 100%;
  height: 100%;
  flex: 1;

}

.card-content {
  padding: 60px;
  display: flex;
  flex-direction: column;
  justify-content: space-between;
  flex: 1;
}

    You can also check my solution here GitHub Link. I hope this will be helpful for you.

    Regards #SAAD

    0
  • Muhammad Saad•280
    @Arman001Posted about 1 year ago

    Hi Good Work!

    You can use patterns in your background on the body as below:

    background-color: var(--DARK-CYAN);
  background-image: url('../images/bg-pattern-top.svg'), url('../images/bg-pattern-bottom.svg') ;
  background-repeat: no-repeat, no-repeat ;
  background-position: right 50vw bottom 50vh, left 50vw top 50vh;

    For further details you can check my repository. Age Calculator GitHub

    Regards #Saad

    Marked as helpful
    0
  • Muhammad Saad•280
    @Arman001Posted about 1 year ago

    Hi! Good work. It seems mobile design is missing, may be you are currently working on it. Also it is scaling down not properly, you can improve it too for the better practice.

    Regards: Saad

    Marked as helpful
    0