490 custom sumo theme #373
Conversation
There was a problem hiding this comment.
Niet logischer om hier een image van de theme in te zetten? Of eventueel een sumo logo?
There was a problem hiding this comment.
Idd, vervangen door sumo logo. Dit wordt toch overschreven door image van klant thema
| @@ -0,0 +1,5 @@ | |||
| p { | |||
| margin: 0 0 20px; | |||
There was a problem hiding this comment.
Is het slim om met px te werken? Niet beter met em? Of een waarde van een bootstrap variabele te gebruiken?
There was a problem hiding this comment.
Weggehaald, we gebruiken standaard bootstrap idd, kwam uit Fork theme die gekopieerd was.
There was a problem hiding this comment.
Er zijn er nog hoor. Misschien eens zoeken naar px in de map src/Themes/Custom/assets/webpack/scss?
src/Themes/Custom/assets/webpack/scss/components/_breadcrumbs.scss
Outdated
Show resolved
Hide resolved
src/Themes/Custom/assets/webpack/scss/components/_privacyConsentDialog.scss
Outdated
Show resolved
Hide resolved
| {% extends 'Core/Layout/Templates/Base.html.twig' %} | ||
|
|
||
| {% block main %} | ||
|
|
|
@tijsverkoyen alles verwerkt |
| @@ -0,0 +1,3 @@ | |||
| .label + .label { | |||
| margin-left: 2px; | |||
There was a problem hiding this comment.
Niet beter met een variable?
| @@ -0,0 +1,17 @@ | |||
| .widget-media-library-lightbox { | |||
| line-height: 0; | |||
| -webkit-column-count: 5; | |||
There was a problem hiding this comment.
Zijn die prefixes nog nodig? https://caniuse.com/?search=column-count
| -moz-column-count: 5; | ||
| -moz-column-gap: 0; | ||
| column-count: 5; | ||
| column-gap: 10px; |
There was a problem hiding this comment.
Variabele? Gutter ofzoiets zou ik denken?
| border-bottom: 0; | ||
| color: $primary; | ||
| font-size: $font-size-sm; | ||
| padding: 0 0 15px; |
| @include media-breakpoint-down(lg) { | ||
| .nav-item { | ||
| .nav-link.dropdown-toggle { | ||
| padding: 8px $dropdown-item-padding-x 8px 0; |
There was a problem hiding this comment.
Niet beter met een variable?
| .navbar { | ||
| @include media-breakpoint-down(lg) { | ||
| .offcanvas { | ||
| padding-top: 84px; |
There was a problem hiding this comment.
Niet beter met een variable?
| background-color: $navbar-icon-color; | ||
| background-image: none; | ||
| height: $navbar-icon-line-height; | ||
| margin: 15px 0; |
There was a problem hiding this comment.
Niet beter met een variable?
| @@ -0,0 +1,7 @@ | |||
| .section-default { | |||
| padding: 50px 0 70px; | |||
There was a problem hiding this comment.
Niet beter met een variable?
Type
Pull request description
Adapt custom sumo theme for fork 6