Skip to content

Conversation

@iam-vipin
Copy link
Member

@iam-vipin iam-vipin commented Jan 20, 2026

Description

Type of Change

  • Improvement (change that would cause existing functionality to not work as expected)

Screenshots and Media (if applicable)

image

References

Summary by CodeRabbit

  • Style
    • Reordered menu items in the toolbar options dropdown. Select options now appear earlier in the sequence for improved accessibility.
    • Note: One menu item now appears twice in the list.

✏️ Tip: You can customize this high-level summary in your review settings.

@iam-vipin iam-vipin self-assigned this Jan 20, 2026
@makeplane
Copy link

makeplane bot commented Jan 20, 2026

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

The optionsOrder array in the options dropdown menu has been reordered to display copy-markdown and version-history actions earlier in the sequence, while toggle-access is repositioned lower and appears twice in the array, affecting the menu item rendering order.

Changes

Cohort / File(s) Summary
Menu Item Reordering
apps/web/core/components/pages/editor/toolbar/options-dropdown.tsx
Reordered PageActions menu items: moved copy-markdown and version-history earlier (after sticky-toolbar), moved toggle-access lower with duplicate entry appearing both after make-a-copy and before export.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰✨ A shuffled menu, items rearranged with care,
Copy-markdown hops forward, version-history takes the air,
Toggle-access dances twice (perhaps by accident?),
The toolbar's new ballet—let's hope it's what we meant! 🎭

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly indicates the change is about fixing the order of dropdown options in pages, which directly matches the summary showing reordering of optionsOrder array for PageActions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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 and usage tips.

Copy link
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 reorders the dropdown menu options in the page editor toolbar to improve the user experience and logical grouping of actions.

Changes:

  • Reordered dropdown menu options to place "copy-markdown" and "version-history" higher in the list, moving "toggle-access" lower

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants