Skip to content

(fix) preserve Study Rail height while moving - #9

Merged
Ammaar-Alam merged 4 commits into
mainfrom
fix/preserve-study-rail-height
Aug 5, 2026
Merged

(fix) preserve Study Rail height while moving#9
Ammaar-Alam merged 4 commits into
mainfrom
fix/preserve-study-rail-height

Conversation

@Ammaar-Alam

Copy link
Copy Markdown
Owner

Summary

  • preserve the Study Rail's rendered height when pointer or keyboard movement begins
  • clamp the rail's position instead of expanding it as it approaches the top
  • shrink only when a smaller viewport requires it and keep Home/reset anchor sizing unchanged
  • cover the behavior with unit geometry and real Chrome pointer-drag regressions

Reproduction

Before the fix, moving the rail from top: 220px to top: 16px increased its unit-test height cap from 564px to 768px. The Chrome fixture likewise expanded the rendered rail from 756px to 764px during an upward drag.

Verification

  • pnpm check
  • 144 unit tests passed
  • 60 Chrome tests passed
  • lint, typecheck, build, and release audit passed

(PR written by Codex)

@Ammaar-Alam
Ammaar-Alam marked this pull request as ready for review August 5, 2026 03:43

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b7650db645

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/ui/StudyRail.ts Outdated
@Ammaar-Alam

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 307213324b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/ui/StudyRail.ts
@Ammaar-Alam

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 355e386d09

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/ui/StudyRail.ts
@Ammaar-Alam

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: 5d2408dc94

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Ammaar-Alam
Ammaar-Alam merged commit b8a0ba0 into main Aug 5, 2026
1 check passed
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.

1 participant