Unify TODOs comments (#42641)

diff --git a/js/src/dom/selector-engine.js b/js/src/dom/selector-engine.js
index 7c28e178ebe18f82796cadea60c690bb3e884a08..d837fba472f08894f32a688b94d6ee554d1a92f2 100644 (file)
--- a/js/src/dom/selector-engine.js
+++ b/js/src/dom/selector-engine.js
@@ -75,7 +75,7 @@ const SelectorEngine = {
     return []
   },
 
-  // TODO: this is now unused; remove later along with prev()
+  // TODO: this is now unused; remove along with prev()
   next(element, selector) {
     let next = element.nextElementSibling
 

diff --git a/scss/_navbar.scss b/scss/_navbar.scss
index 3dc4bebf56c77e9c06fc4ba8a1ab7d576d55f52b..c52054ffe2648e0cc55f0d2b10979512ab58ed1a 100644 (file)
--- a/scss/_navbar.scss
+++ b/scss/_navbar.scss
@@ -6,7 +6,7 @@
 @use "mixins/tokens" as *;
 @use "mixins/transition" as *;
 
-// mdo-do: fix nav-link-height and navbar-brand-height, which we previously calculated with font-size, line-height, and block padding
+// TODO: fix nav-link-height and navbar-brand-height, which we previously calculated with font-size, line-height, and block padding
 
 // stylelint-disable custom-property-no-missing-var-function
 // scss-docs-start navbar-breakpoints

diff --git a/scss/_pagination.scss b/scss/_pagination.scss
index 9ddeed2a44608851cc68aaa5134ab2ebbfaea111..109a6423f9e30db62e0408b6e3942fd3f9e26db3 100644 (file)
--- a/scss/_pagination.scss
+++ b/scss/_pagination.scss
@@ -6,7 +6,7 @@
 @use "mixins/transition" as *;
 @use "mixins/tokens" as *;
 
-// mdo-do: Update pagination to support variant themes
+// TODO: update pagination to support variant themes (`.theme-*` role tokens)
 
 // stylelint-disable custom-property-no-missing-var-function
 $pagination-tokens: () !default;

diff --git a/scss/_root.scss b/scss/_root.scss
index b35cb1c56fa267073ef8f6e6f29a5a895d023e59..3d53bbd0340c2b6eac2d95c94072a5481ae4010a 100644 (file)
--- a/scss/_root.scss
+++ b/scss/_root.scss
@@ -4,7 +4,7 @@
 @use "functions" as *;
 @use "theme" as *;
 @use "mixins/tokens" as *;
-// mdo-do: do we need theme?
+// TODO: do we need theme output here?
 @layer colors, theme, config, root, reboot, layout, content, forms, components, custom, helpers, utilities;
 
 $root-tokens: () !default;

diff --git a/scss/_spinner.scss b/scss/_spinner.scss
index 5520c469264a8fe03103db372ae6a4ce7760a0b5..cc0bf9525c01294966d14cea2fca2724d01370a2 100644 (file)
--- a/scss/_spinner.scss
+++ b/scss/_spinner.scss
@@ -43,7 +43,7 @@ $spinner-grow-tokens: defaults(
 //
 
 @layer components {
-  // mdo-do: Refactor this to assume flex parent and remove `vertical-align`
+  // TODO: refactor this to assume flex parent and remove `vertical-align`
   .spinner-grow,
   .spinner-border {
     display: inline-block;

diff --git a/scss/_theme.scss b/scss/_theme.scss
index 0327ccfe51a7118df8819a3f587e6e67d78630c5..df71b290a51c9f99ed0a66bff66b26c61e5fa714 100644 (file)
--- a/scss/_theme.scss
+++ b/scss/_theme.scss
@@ -167,7 +167,7 @@ $theme-colors: (
 ) !default;
 // scss-docs-end theme-colors
 
-// mdo-do: consider using muted, subtle, ghost or something instead of linear scale?
+// TODO: decide $theme-bgs scale naming — semantic (muted/subtle/ghost) vs linear numeric
 $theme-bgs: (
   "body": light-dark(var(--white), var(--gray-975)),
   "1": light-dark(var(--gray-025), var(--gray-950)),

diff --git a/scss/content/_reboot.scss b/scss/content/_reboot.scss
index 578a88812e33823835d2ef21a8acec13c0738716..11a12c153bbe0bf7aacee126ae8d60bded401cdf 100644 (file)
--- a/scss/content/_reboot.scss
+++ b/scss/content/_reboot.scss
@@ -354,7 +354,7 @@ $reboot-mark-tokens: defaults(
     kbd {
       padding: 0;
       font-size: 1em;
-      font-weight: inherit; // mdo-do: check if this is needed
+      font-weight: inherit; // TODO: check if this is needed
     }
   }
 
@@ -396,7 +396,7 @@ $reboot-mark-tokens: defaults(
   // 3. Fix alignment for Safari
 
   th {
-    // font-weight: $table-th-font-weight; // 1 // mdo-do: it's null by default. maybe we remove?
+    // font-weight: $table-th-font-weight; // 1 // TODO: null by default — decide whether to remove
     text-align: inherit; // 2
     text-align: -webkit-match-parent; // 3
   }

diff --git a/scss/forms/_form-control.scss b/scss/forms/_form-control.scss
index 9eedb19866e546f5292c5b4b37904ef791e86c7b..69ee634041babc51fc6f8ed551757bdfcc31f5b1 100644 (file)
--- a/scss/forms/_form-control.scss
+++ b/scss/forms/_form-control.scss
@@ -121,7 +121,7 @@ $form-control-sizes: defaults(
     // Prevent excessive date input height in Webkit
     // https://github.com/twbs/bootstrap/issues/34433
 
-    // mdo-do: need to check this stuff out across browsers
+    // TODO: verify these rules across the supported browser matrix
     &::-webkit-datetime-edit {
       display: block;
       height: 1.5rem;

diff --git a/scss/layout/_grid.scss b/scss/layout/_grid.scss
index dab87eb7d386d776cb87faa16b07e53ecc07d352..33d92f605f1cb4755b87e346dab089b9d2410042 100644 (file)
--- a/scss/layout/_grid.scss
+++ b/scss/layout/_grid.scss
@@ -1,8 +1,7 @@
 @use "../config" as *;
 @use "../mixins/grid" as *;
 
-// mdo-do
-// - check gap utilities as replacement for gutter classes from v5
+// TODO: check gap utilities as replacement for gutter classes from v5
 
 @layer layout {
   @if $enable-grid-classes {
@@ -33,7 +32,7 @@
     @include make-cssgrid();
   }
 
-  // mdo-do: add to utilities?
+  // TODO: add to utilities?
   .grid-cols-subgrid {
     grid-template-columns: subgrid;
   }
@@ -51,7 +50,7 @@
   //   grid-column: auto;
   // }
 
-  // mdo-do: add to utilities?
+  // TODO: add to utilities?
   // .grid-cols-3 {
   //   --columns: 3;
   // }

diff --git a/site/src/content/docs/customize/color.mdx b/site/src/content/docs/customize/color.mdx
index 8cf53cdf5119ee67b70cfa8b9c22eb31ac362896..6565e1723c7896346dbb56abc22dbf412acaa80e 100644 (file)
--- a/site/src/content/docs/customize/color.mdx
+++ b/site/src/content/docs/customize/color.mdx
@@ -125,9 +125,7 @@ To remove a color, set the value to `null`.
 );
 ```
 
-<Callout type="warning">
-mdo-do: Update the content below
-</Callout>
+{/* TODO: rewrite the content below for the v6 color pipeline; drop v5-era references */}
 
 ## Generating utilities
 

diff --git a/site/src/content/docs/customize/components.mdx b/site/src/content/docs/customize/components.mdx
index a42b75653621fe83ed221640cd27bd533a0f7ea2..d337643618e6d2818eebce372cf71b2f69bc30fc 100644 (file)
--- a/site/src/content/docs/customize/components.mdx
+++ b/site/src/content/docs/customize/components.mdx
@@ -14,9 +14,7 @@ Check out [our Sass maps and loops docs]([[docsref:/customize/sass#maps-and-loop
 
 ## Theme variants
 
-<Callout type="info">
-@mdo-do: Add more info about theme variants here.
-</Callout>
+{/* TODO: write a real theme-variants section (how components consume --theme-* tokens, with a worked example) */}
 
 ## Responsive
 

diff --git a/site/src/content/docs/forms/form-control.mdx b/site/src/content/docs/forms/form-control.mdx
index af3b93447b093c38d32511db2cbe76c970c64fec..87d14626ef513b578c8feea4c46691c40bb42e59 100644 (file)
--- a/site/src/content/docs/forms/form-control.mdx
+++ b/site/src/content/docs/forms/form-control.mdx
@@ -372,7 +372,7 @@ We use a Sass map to define the sizes of the form controls, so you can easily ad
 
 <ScssDocs name="form-control-sizes" file="scss/forms/_form-control.scss" />
 
-{/* mdo-do: clean these up, possibly find new homes */}
+{/* TODO: clean these up, possibly find new homes */}
 
 {/* `$input-*` are shared across most of our form controls (and not buttons).
 

diff --git a/site/src/content/docs/utilities/aspect-ratio.mdx b/site/src/content/docs/utilities/aspect-ratio.mdx
index de98de9f808cdf79644e16f8df549a39e3e53cfc..8fb6fecb0f399ee078c104f77d7d9fb4c18bd54d 100644 (file)
--- a/site/src/content/docs/utilities/aspect-ratio.mdx
+++ b/site/src/content/docs/utilities/aspect-ratio.mdx
@@ -43,7 +43,7 @@ Swap the `.ratio-*` class for a different aspect ratio.
 
 ## Custom ratios
 
-mdo-do: do we bring these back?
+TODO: decide whether to restore custom ratios; delete this block or reinstate the section
 
 Each `.ratio-*` class includes a CSS custom property (or CSS variable) in the selector. You can override this CSS variable to create custom aspect ratios on the fly with some quick math on your part.