chore(refactoring): finish remaining modifications after Bootstrap de…

…sign doc refactoring
Orange-OpenSource · May 13, 2022 · 61fca19 · 61fca19
1 parent 1266609
commit 61fca19
Show file tree

Hide file tree

Showing 9 changed files with 0 additions and 26 deletions.
diff --git a/scss/_navbar.scss b/scss/_navbar.scss
@@ -453,13 +453,11 @@
           display: none;
         }
 
-        //////////////////////////////////////////////////////////////////////////////
         //////////////////////////////////////////////////////////////////////////////
         //////////////////////////////////////////////////////////////////////////////
         // TODO: Is it really useful in Boosted?
         //////////////////////////////////////////////////////////////////////////////
         //////////////////////////////////////////////////////////////////////////////
-        //////////////////////////////////////////////////////////////////////////////
         .offcanvas {
           // stylelint-disable declaration-no-important
           position: static;

diff --git a/site/assets/js/code-examples.js b/site/assets/js/code-examples.js
@@ -18,15 +18,13 @@
   const btnTitle = 'Copy to clipboard'
   const btnEdit = 'Edit on StackBlitz'
 
-  // ////////////////////////////////////////////////////////
   // ////////////////////////////////////////////////////////
   // ////////////////////////////////////////////////////////
   // TODO: Bootstrap can remove width, height and
   // fill="currentColor" of <svg class="bi"... since ".bi"
   // already does that
   // ////////////////////////////////////////////////////////
   // ////////////////////////////////////////////////////////
-  // ////////////////////////////////////////////////////////
   // Boosted mod: removed unused svg height, width and fill="currentColor"
   const btnHtml = [
     '<div class="bd-code-snippet">',

diff --git a/site/assets/scss/_clipboard-js.scss b/site/assets/scss/_clipboard-js.scss
@@ -27,13 +27,11 @@
   border: 0;
   @include border-radius(.25rem);
 
-  ////////////////////////////////////////////////////////
   ////////////////////////////////////////////////////////
   ////////////////////////////////////////////////////////
   // TODO: see if we can't use a btn-* class instead
   ////////////////////////////////////////////////////////
   ////////////////////////////////////////////////////////
-  ////////////////////////////////////////////////////////
   &:hover {
     color: $primary;
   }
@@ -55,25 +53,21 @@
   z-index: 2;
   ////////////////////////////////////////////////////////
   ////////////////////////////////////////////////////////
-  ////////////////////////////////////////////////////////
   // TODO: check all cases
   // TODO: see if we can't use $border-something
   // TODO: px >>> rem
   ////////////////////////////////////////////////////////
   ////////////////////////////////////////////////////////
-  ////////////////////////////////////////////////////////
   margin-top: 2px; // Boosted mod
   margin-right: 2px; // Boosted mod
 }
 
-////////////////////////////////////////////////////////
 ////////////////////////////////////////////////////////
 ////////////////////////////////////////////////////////
 // Boosted mod: removed it in favor of using utility classes
 // in shortcodes > example.html
 ////////////////////////////////////////////////////////
 ////////////////////////////////////////////////////////
-////////////////////////////////////////////////////////
 
 /* .highlight-toolbar {
   .btn-clipboard {

diff --git a/site/assets/scss/_content.scss b/site/assets/scss/_content.scss
@@ -101,13 +101,11 @@
   fill: currentcolor;
 }
 
-/////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////
 // TODO: test into Bootstrap if there's not a conflict with .icon-link in features.css
 /////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////
-/////////////////////////////////////////////////////////////////
 .icon-link {
   display: flex;
   align-items: center;
@@ -121,13 +119,11 @@
     transition: .2s ease-in-out transform; // stylelint-disable-line property-disallowed-list
   }
 
-  /////////////////////////////////////////////////////////////////
   /////////////////////////////////////////////////////////////////
   /////////////////////////////////////////////////////////////////
   // TODO: Do we keep this animation in Boosted?
   /////////////////////////////////////////////////////////////////
   /////////////////////////////////////////////////////////////////
-  /////////////////////////////////////////////////////////////////
   &:hover {
     .bi {
       transform: translate3d(5px, 0, 0);

diff --git a/site/assets/scss/_layout.scss b/site/assets/scss/_layout.scss
@@ -1,10 +1,8 @@
 ///////////////////////////////////////////////
 ///////////////////////////////////////////////
-///////////////////////////////////////////////
 // TODO: check me! Not sure what's the purpose of --bs-gutter-x
 ///////////////////////////////////////////////
 ///////////////////////////////////////////////
-///////////////////////////////////////////////
 .bd-gutter {
   --bs-gutter-x: #{$bd-gutter-x};
 }

diff --git a/site/assets/scss/_masthead.scss b/site/assets/scss/_masthead.scss
@@ -71,12 +71,10 @@
   }
 }
 
-///////////////////////////////////////////////////////////
 ///////////////////////////////////////////////////////////
 ///////////////////////////////////////////////////////////
 // Boosted mod: no .masthead-followup-svg
 // Boosted mod: no .masthead-notice
 // TODO: check if it's the case in Bootstrap as well
 ///////////////////////////////////////////////////////////
 ///////////////////////////////////////////////////////////
-///////////////////////////////////////////////////////////
diff --git a/site/assets/scss/_navbar.scss b/site/assets/scss/_navbar.scss
@@ -1,10 +1,8 @@
 /////////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////////
-/////////////////////////////////////////////////////////////////////////////////
 // TODO: check all those rules but normally we wouldn't need anything in this file
 /////////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////////
-/////////////////////////////////////////////////////////////////////////////////
 .bd-navbar {
   // Boosted mod: no padding
   background-color: transparent;

diff --git a/site/assets/scss/_sidebar.scss b/site/assets/scss/_sidebar.scss
@@ -37,11 +37,9 @@
   margin-top: .125rem;
   //////////////////////////////////////////////////////////////////////////////
   //////////////////////////////////////////////////////////////////////////////
-  //////////////////////////////////////////////////////////////////////////////
   // TODO: adapt this rule after having changed the size of icons
   //////////////////////////////////////////////////////////////////////////////
   //////////////////////////////////////////////////////////////////////////////
-  //////////////////////////////////////////////////////////////////////////////
   margin-left: 1.125rem; // Boosted mod: changed value
   color: rgba($black, .65);
   text-decoration: if($link-decoration == none, null, none);

diff --git a/site/assets/scss/_subnav.scss b/site/assets/scss/_subnav.scss
@@ -1,12 +1,10 @@
 // Boosted mod: doesn't exist anymore in Bootstrap
 
-//////////////////////////////////////////////////////////////////////////////////////////
 //////////////////////////////////////////////////////////////////////////////////////////
 //////////////////////////////////////////////////////////////////////////////////////////
 // TODO: clean up and remove useless rules in this file
 //////////////////////////////////////////////////////////////////////////////////////////
 //////////////////////////////////////////////////////////////////////////////////////////
-//////////////////////////////////////////////////////////////////////////////////////////
 
 .bd-subnavbar {
   --bs-gutter-x: $bd-gutter-x;
@@ -33,14 +31,12 @@
     background-size: .75rem .75rem;
   }
 
-  //////////////////////////////////////////////////////////////////////////////////////////
   //////////////////////////////////////////////////////////////////////////////////////////
   //////////////////////////////////////////////////////////////////////////////////////////
   // TODO: since it is always displayed when there's a burger menu we should be able to
   // get rid of this @include (not its content which would become the default)
   //////////////////////////////////////////////////////////////////////////////////////////
   //////////////////////////////////////////////////////////////////////////////////////////
-  //////////////////////////////////////////////////////////////////////////////////////////
   @include media-breakpoint-up(md) {
     position: sticky;
     top: 60px; // Boosted mod: change value