ronaldlamdev

Latest comments

• 

  ronaldlamdev•430

  @ronaldlamdevSubmitted over 2 years ago

  

  Space site with React Router Dom and Tailwind CSS

  #react#react-router#tailwind-css

  1

  20
  

  ronaldlamdev•430

  @ronaldlamdevPosted over 2 years ago

  Disregard my message about the stated issue. It's fixed now.

  0
• 

  Daniel•230

  @danielbrahSubmitted over 2 years ago

  

  Easybank Landing Page

  #sass/scss

  1

  21
  

  ronaldlamdev•430

  @ronaldlamdevPosted over 2 years ago

  Nice job. I also have a solution to this project which is right here (https://github.com/ronaldlamdev/easybank) where I used React Router Dom and Tailwind CSS.

  One thing I would add is make the h2s in your footer have hover animations to make it more user-friendly and interactive even if this isn't a real website.

  0
• 

  Rafael Nacle•80

  @rafaelnacleSubmitted over 2 years ago

  

  Responsive landing page of the e-commer page.

  1

  40
  

  ronaldlamdev•430

  @ronaldlamdevPosted over 2 years ago

  Nice job. I also finished this project today. I hope you don't mind me looking at your code to see how you got the cart to show quantity added.

  One issue I have is whenever I click on the thumbnail images, it opens the modal. I think this would be better if the open modal function was for when you click on the larger image, but that's just me.

  Marked as helpful

  1
• 

  Luis Fuentes•50

  @EleganteOsoSubmitted over 2 years ago

  

  HTML & CSS

  1

  40
  

  ronaldlamdev•430

  @ronaldlamdevPosted over 2 years ago

  Very nice job. Just to make your buttons seemingly more functional, I would add to each button "cursor : pointer". This will give your buttons that typical finger pointing icon like when you hover over a link on an actual website. This won't give your links a destination of course, but this is just for aesthetic purposes.

  Marked as helpful

  0
• 

  Sara Dunlop•450

  @RiscloverSubmitted almost 3 years ago

  

  Intro Component with Signup

  1

  30
  

  ronaldlamdev•430

  @ronaldlamdevPosted almost 3 years ago

  Try to add in @media (min-width: 1000px) a background image: url('images/bg-intro-desktop.png'). Hopefully this removes the column that you are talking about.

  Marked as helpful

  0
• 

  Daniela Darnea•180

  @mdanieladlaSubmitted almost 3 years ago

  

  Fylo landing page

  1

  40
  

  ronaldlamdev•430

  @ronaldlamdevPosted almost 3 years ago

  Great job on coding this challenge. There is one thing I would change with your email form. I would change the input for your email to text in your HTML. For example, I would change <input type="email"/> to <input type="text"/>. This will remove the default error message (not the red error message), but it'll make your form error cleaner and easier to read. Having two error messages is kinda redundant.

  I would also start using more id attributes in your HTML instead of repeatedly using class to name your tags. The benefit of this is that id attributes allow links to navigate to other sections of your code. For example, instead of class = "features", use id = "features". As for <li> features </li>, use <li href="#features"> features </li>. This way, the site itself will automatically scroll to the feature section of your site instead of having the user manually scroll to the site.

  Marked as helpful

  1