Skip to content
  • Unlock Pro
  • Log in with GitHub
Profile
OverviewSolutions
33
Comments
25

nmorajda

@nmorajdaPoland945 points

Full stack developer with 7 years of freelance experience. Best wishes, Norbert Morajda phone: +48 665 665 025 website: https://abmstudio.pl

Latest solutions

  • Vanilla JS, Mobile First, SCSS, FlexBox


    nmorajda•945
    Submitted almost 4 years ago

    0 comments
  • Vanilla JS ES6, SCSS, FlexBox - Time tracking dashboard


    nmorajda•945
    Submitted almost 4 years ago

    0 comments
  • Calculator app site using vanilla JavaScript, HTML, SCSS,


    nmorajda•945
    Submitted almost 4 years ago

    2 comments
  • Vanilla JS, Mobile First, SCSS, FlexBox


    nmorajda•945
    Submitted almost 4 years ago

    0 comments
  • Mobile first, BEM, FlexBox - Order Summary Card


    nmorajda•945
    Submitted almost 4 years ago

    0 comments
  • Mobile toggle menu without JavaScript. Flexbox.


    nmorajda•945
    Submitted about 4 years ago

    1 comment
View more solutions

Latest comments

  • Paul Pabilonia•10
    @PaulPabilonia
    Submitted almost 4 years ago

    Simple CSS and HTML

    1
    nmorajda•945
    @nmorajda
    Posted almost 4 years ago
    1. Use correct indentation in the HTML code

    2. Reset default CSS styles for browser (now you have default margin / padding).

    3. Change

    .container {
      height: 100%;
    }
    

    on

    .container {
      min-height: 100vh;
    }
    

    and remove the padding, and you will have the element centered vertically as well.

    Marked as helpful
  • ThanhVuong0904•465
    @ThanhVuong0904
    Submitted almost 4 years ago

    HTML 5, SASS, JS, JSON server

    3
    nmorajda•945
    @nmorajda
    Posted almost 4 years ago

    Links don't work, I guess that's why:

    function getData(callback) {
       const API = "http://localhost:3000/tracking";
        ...
    

    and correct the errors visible in the report at the top.

    Happy codding :)

  • ThanhVuong0904•465
    @ThanhVuong0904
    Submitted almost 4 years ago

    HTML 5, SASS, JS, JSON server

    3
    nmorajda•945
    @nmorajda
    Posted almost 4 years ago
                        if(clickIndex === "weekly") {
                             timeFramesCurrent = timeframes.weekly.current;
                             timeFremesPre = timeframes.weekly.previous;
                        }
                             
                        else if (clickIndex === "daily") {
                             timeFramesCurrent = timeframes.daily.current;
                             timeFremesPre = timeframes.daily.previous;
                        }
                        else if (clickIndex === "monthly") {
                             timeFramesCurrent = timeframes.monthly.current;
                             timeFremesPre = timeframes.monthly.previous;
                        }
    

    can probably be written shorter and just as legible:

    const timeFramesCurrent = timeframes[clickIndex].current;
    const timeFremesPre = timeframes[clickIndex].previous;
    

    or maybe:

    const {current, previous} = timeframes[clickIndex]
    

    in the second case you have to rename the variables later in the code or

    const {current: timeFramesCurrent, previous: timeFremesPre} = timeframes[clickIndex]
    
    Marked as helpful
  • David Kalany•50
    @AmazingCukr
    Submitted almost 4 years ago

    Order summary component

    1
    nmorajda•945
    @nmorajda
    Posted almost 4 years ago

    Look fine on the desktop, but there are some small bugs in the code:

    <div class="Header"> ...
    

    Don't use uppercase letters at the beginning of the class name, id, etc.

    <div class="text"> ...
    

    This div element does nothing.

    Why not:

    <p class="text"> ....
    

    ?

    It is also not responsive and from a width of 400px a horizontal scroll bar starts to appear.

    You also have three errors in the report above.

  • Segundo Juan•95
    @JSegundo
    Submitted almost 4 years ago

    HTML & SCSS

    1
    nmorajda•945
    @nmorajda
    Posted almost 4 years ago

    It looks good ... but only on the desktop.

    It is not responsive.

    Use a media query for this purpose.

    Do not specify the height in px (usually the height of the element is to result from its content or the height of the parent element).

    If you use the section element, there should be some header inside. In this case, the IMHO section element is replaceable with a div element.

    Marked as helpful
  • Anis•135
    @anisgo
    Submitted almost 4 years ago

    Order Summary Responsive - Html and Css - flexbox - mediaquery.

    1
    nmorajda•945
    @nmorajda
    Posted almost 4 years ago

    The height of an element should rather result from its content, and not be fixed:

    height: 80vh;
    

    Use the min-height property if you must, but you don't have to.

    Marked as helpful
View more comments
Frontend Mentor logo

Stay up to datewith new challenges, featured solutions, selected articles, and our latest news

Frontend Mentor

  • Unlock Pro
  • Contact us
  • FAQs
  • Become a partner

Explore

  • Learning paths
  • Challenges
  • Solutions
  • Articles

Community

  • Discord
  • Guidelines

For companies

  • Hire developers
  • Train developers
© Frontend Mentor 2019 - 2025
  • Terms
  • Cookie Policy
  • Privacy Policy
  • License

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub