Skip to content

Improve console output formatting in Recursion glossary#42478

Merged
hamishwillee merged 3 commits intomdn:mainfrom
muhammadfahaddev:improve-recursion-output
Dec 22, 2025
Merged

Improve console output formatting in Recursion glossary#42478
hamishwillee merged 3 commits intomdn:mainfrom
muhammadfahaddev:improve-recursion-output

Conversation

@muhammadfahaddev
Copy link
Copy Markdown
Contributor

Description

Motivation

Additional details

Related issues and pull requests

Copilot AI review requested due to automatic review settings December 20, 2025 05:05
@muhammadfahaddev muhammadfahaddev requested a review from a team as a code owner December 20, 2025 05:05
@muhammadfahaddev muhammadfahaddev requested review from hamishwillee and removed request for a team December 20, 2025 05:05
@github-actions github-actions Bot added Content:Glossary Glossary entries size/s [PR only] 6-50 LoC changed labels Dec 20, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves the formatting and organization of console output examples in the Recursion glossary documentation. The changes enhance readability by separating code from its output and adding descriptive section headers.

  • Adds descriptive subheaders (Factorial, Fibonacci, Reduce) for each common usage example
  • Moves inline output comments to separate "Output:" sections using proper plain code blocks
  • Improves consistency with the existing output formatting pattern used earlier in the document

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 22, 2025

Preview URLs

(comment last updated: 2025-12-22 00:45:44)

Comment thread files/en-us/glossary/recursion/index.md
Comment thread files/en-us/glossary/recursion/index.md Outdated
Comment thread files/en-us/glossary/recursion/index.md Outdated
Comment thread files/en-us/glossary/recursion/index.md
Comment thread files/en-us/glossary/recursion/index.md Outdated
Comment thread files/en-us/glossary/recursion/index.md
Copy link
Copy Markdown
Collaborator

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@muhammadfahaddev Thank you, and welcome to MDN.

FWIW I like the headings, but having comments for the console output is the standard way of doing example output in MDN. We wouldn't change this for just one page.

A good place to start is with our existing issues list, and in particular good first issues - these are things we know to be problems - fixing them is more useful than tidying layout.

@hamishwillee hamishwillee merged commit edcc531 into mdn:main Dec 22, 2025
7 checks passed
wbamberg added a commit to wbamberg/content that referenced this pull request Dec 24, 2025
…skeys-guide

* origin/add-passkeys-guide:
  Clarify fetch() error handling behavior for Response (mdn#42497)
  Add code example to Closure glossary entry (mdn#42477)
  42252 add new relative units ff release (mdn#42470)
  Remove duplicated word (mdn#42496)
  42252 add new relative units (mdn#42469)
  TrustedTypes: Range.createContextualFragment() (mdn#42493)
  Fix CSS, HTML, and HTTP landing pages to match sidebar content and order (mdn#42460)
  TT: CSP trusted types/require-trusted... fixes (mdn#42461)
  Improve console output formatting in Recursion glossary (mdn#42478)
  Add Sec-CH-Width + consistency fixes (mdn#42453)
  webextensions/manifest.json/options_page: typo (mdn#42394)
  webextensions/api/tabs/onupdated: Fix section link (mdn#42393)
  fix: add missing closing parenthesis (mdn#42482)
  docs: mark srcData as required in bufferSubData (mdn#42483)
  docs(learn): remove invalid type attribute from video example (mdn#42479)
  Drop 'read-only' from role (mdn#42474)
  Fix: Update bufferSubData parameters to match WebGL 1 spec mdn#42315 (mdn#42411)
  chore: Replace links with `cssxref` macro in the learn area (mdn#42446)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Glossary Glossary entries size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants