Skip to content
Snippets Groups Projects

need important and no need for fit-content

Created by: flora8984461

Do need "!important" for no-gutters-mobile, otherwise it will be overridden by paddings classes, and I removed fit-content since no need for it if the paddings are removed.

I find I can remove the "padding-right-40" and "padding-left-40" since the class ".content-nav-tab-body." and "tab-content" gives a padding 40 on big screen and a padding 10 on mobile view.

@chrisguindon I wonder if you like to keep the 10 paddings on the mobile view? - If so, we do not need to add any new classes as long as the screen is bigger than 360px. The following are the view with 10 paddings: ( You can see, with 10 paddings and without fit-content, the button would be a little bit ugly on size 360px. ) image image

this following is a wider mobile with 375px: ( You can see, with 10 paddings and without fit-content, the button is good on size 375px. ) image

If you prefer to use 0 paddings, then we probably need this no-gutters-mobile class.

Signed-off-by: Yi Liu yi.liu@eclipse-foundation.org

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Created by: netlify[bot]

    Deploy preview for eclipsefdn-solstice-assets ready!

    Built with commit 5cd3dc6b02a428dfb247ed1fed3fee56311703b1

    https://deploy-preview-180--eclipsefdn-solstice-assets.netlify.app

  • Is there nothing else we can target to make the rule more specific to override the padding (if we still need it)? Important is bad and should be last resort.

    e.g.

    .tab-content.no-gutters-mobile {
        padding-left: 0;
        padding-right: 0;
    }

    and move it to the file for the content-nav-tab (as its still specific to this page)

  • Created by: flora8984461

    Is there nothing else we can target to make the rule more specific to override the padding (if we still need it)? Important is bad and should be last resort.

    e.g.

    .tab-content.no-gutters-mobile {
        padding-left: 0;
        padding-right: 0;
    }

    and move it to the file for the content-nav-tab (as its still specific to this page)

    That's much better and it works!

    Let's confirm should we keep the 10 paddings or not. If we want to use 0 paddings, I will use this one.

Please register or sign in to reply
Loading