Skip to content

Change button label from 'implementation' to 'current' #1589

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
anthonyiscoding wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Change button label from 'implementation' to 'current' #1589

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion website/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4484,7 +4484,7 @@ <h1 class="hero-headline" id="hero-headline" aria-live="polite">
</span>
<div class="hero-viz-tabs" role="tablist" aria-label="Stage">
<button type="button" data-stage="mesh" class="on">problem space</button>
<button type="button" data-stage="actual">implementation</button>
<button type="button" data-stage="actual">current</button>
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot May 1, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Keep stage terminology consistent for data-stage="actual"

Line 4487 now says “current”, but the runtime stage label still resolves to “implementation” (Line 8249), so the same state is presented with two names.

🔧 Suggested fix 
-    actual: { name: 'implementation', count: String(actualEdges.length), accentCount: false, formula: '<math><mi>O</mi><mo>(</mo><mfrac><mrow><mi>n</mi><mo>(</mo><mi>n</mi><mo>\u2212</mo><mn>1</mn><mo>)</mo></mrow><mrow><mn>2</mn><mo>\u00b7</mo><mtext>your tolerance for integrations</mtext></mrow></mfrac><mo>)</mo></math>', formulaSub: '' },
+    actual: { name: 'current', count: String(actualEdges.length), accentCount: false, formula: '<math><mi>O</mi><mo>(</mo><mfrac><mrow><mi>n</mi><mo>(</mo><mi>n</mi><mo>\u2212</mo><mn>1</mn><mo>)</mo></mrow><mrow><mn>2</mn><mo>\u00b7</mo><mtext>your tolerance for integrations</mtext></mrow></mfrac><mo>)</mo></math>', formulaSub: '' },

As per coding guidelines, “Ensure the communications on the website are consistent with docs/ in functionality.”

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@website/index.html` at line 4487, The button with attribute
data-stage="actual" currently displays the text "current" which conflicts with
the runtime stage label that resolves to "implementation"; update the button’s
visible label to match the runtime terminology (change the button text from
"current" to "implementation") so the state shown by the element with
data-stage="actual" is consistent with the runtime stage label (the element that
resolves to "implementation").

<button type="button" data-stage="iii">iii</button>
</div>
</div>
Expand Down
Loading