Skip to content

Conversation

@kenny-not-dead
Copy link
Contributor

partial #5979

Excel allows you to enter a range through this input. I think this is a good starting point for implementing this input's functionality.

In the future, I think it might be possible to add the display of defined names and their assignments.

Before:

After:

Screencast.from.2025-11-12.22-41-53.webm

Pull Request Checklist

  • Related tickets or issues have been linked in the PR description (or missing issue).
  • Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
  • Unit tests have been added for the changes (if applicable).
  • Breaking changes have been documented (or no breaking changes introduced in this PR).

@github-actions
Copy link

github-actions bot commented Nov 12, 2025

View Deployment

📑 Demo (React@19) Demo (React@16) 📚 Storybook
🔗 Preview link 🔗 Preview link 🔗 Preview link

@jikkai jikkai requested a review from DR-Univer November 14, 2025 02:25
@kenny-not-dead
Copy link
Contributor Author

@DR-Univer
Is this PR still relevant or can it be closed?

@kenny-not-dead
Copy link
Contributor Author

team?

@jikkai
Copy link
Member

jikkai commented Dec 3, 2025

Sorry for the slow response. We've been focused on resolving some collaboration issues :)

The PR works well, but the viewport currently fails to scroll to the active selection. This is inconsistent with Excel and undermines the utility of the feature. Could you add logic to ensure the selection scrolls into view?

@kenny-not-dead
Copy link
Contributor Author

@jikkai
Could you please specify the scrolling behavior in more detail? Maybe you can make a short video?

@kenny-not-dead
Copy link
Contributor Author

No matter how hard I tried to define the required behavior, I couldn't figure out what exactly should happen in which cases. In Libre, it works the same way as in PR. Therefore, I would still be grateful for a demonstration of what needs to be added within the video, or a more detailed description of the flow.

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.

2 participants