Skip to content

#56 add marker for gpt release#58

Merged
Asifdotexe merged 1 commit into
mainfrom
56-ui-add-indicator-of-ai-genesis
Jul 2, 2026
Merged

#56 add marker for gpt release#58
Asifdotexe merged 1 commit into
mainfrom
56-ui-add-indicator-of-ai-genesis

Conversation

@Asifdotexe

@Asifdotexe Asifdotexe commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features
    • Added support for shared milestones that can appear across all repositories, not just within a single repo.
    • Milestone markers now include improved visuals and hover/focus interactions for clearer highlighting.
    • Tooltip milestone matching now considers the full set of available milestones when showing milestone banners.
  • Chores
    • Updated ignore rules to exclude an additional generated documentation file.

@Asifdotexe Asifdotexe linked an issue Jul 2, 2026 that may be closed by this pull request
@Asifdotexe Asifdotexe merged commit 57c2c18 into main Jul 2, 2026
1 check passed
@Asifdotexe Asifdotexe deleted the 56-ui-add-indicator-of-ai-genesis branch July 2, 2026 15:03
@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3ae634f4-ced2-4f32-bca5-2887205962a2

📥 Commits

Reviewing files that changed from the base of the PR and between 679ac75 and 61a91af.

📒 Files selected for processing (3)
  • .gitignore
  • app.js
  • theseus.config.json

📝 Walkthrough

Walkthrough

This PR introduces global milestones alongside per-repository milestones in the Theseus visualizer. theseus.config.json gains a globalMilestones array; app.js reads and merges this into milestone rendering and tooltip logic, with updated marker visuals (line elements, hover/focus transitions). .gitignore also excludes a generated blog file.

Changes

Global milestones feature

Layer / File(s) Summary
Global milestone config and init
theseus.config.json, app.js
Adds a globalMilestones array (e.g., "ChatGPT Released") to config, and init() stores this array on the instance.
Milestone marker rendering with merged list
app.js
renderMilestoneMarkers() merges global and repo milestones, updates the early-exit condition, adds a line element per marker, restyles star/text, and extends enlarge/shrink transitions to animate the new line.
Tooltip milestone lookup
app.js
showTooltip() scans the merged global + repo milestone list instead of only repository milestones.
Ignore generated blog file
.gitignore
Adds docs/theseus_blog.md to ignore patterns.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • Asifdotexe/Theseus#29: Both PRs modify app.js’s milestone visualization/tooltip logic built on per-repo milestones.
  • Asifdotexe/Theseus#32: Both PRs modify app.js’s milestone marker rendering/hover/focus styling behavior.

Suggested labels: enhancement, ui/ux

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 56-ui-add-indicator-of-ai-genesis

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ui: add indicator of ai genesis

1 participant