Skip to content

feat: add smoothScrollToTop and expose onSearchAction event on CodeEditor#755

Merged
AbhishekA1509 merged 12 commits into
developfrom
fix/app-details-manifest-layout
May 29, 2025
Merged

feat: add smoothScrollToTop and expose onSearchAction event on CodeEditor#755
AbhishekA1509 merged 12 commits into
developfrom
fix/app-details-manifest-layout

Conversation

@AbhishekA1509

@AbhishekA1509 AbhishekA1509 commented May 27, 2025

Copy link
Copy Markdown
Member

Description

feat: add smoothScrollToTop and expose onSearchAction event on CodeEditor

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Dev tested

Checklist

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

@AbhishekA1509 AbhishekA1509 changed the title Fix/app details manifest layout feat: add smoothScrollToTop and expose onSearchAction event on CodeEditor May 28, 2025
…rch functionality

feat(Helpers): update smoothScrollToTop easing function to use CUBIC_BEZIER_CURVE constant

feat(constants): define CUBIC_BEZIER_CURVE constant for consistent animation easing
Comment thread src/Shared/Components/CodeEditor/CodeEditor.tsx
RohitRaj011
RohitRaj011 previously approved these changes May 29, 2025
vivek-devtron
vivek-devtron previously approved these changes May 29, 2025
@AbhishekA1509 AbhishekA1509 dismissed stale reviews from vivek-devtron and RohitRaj011 via 769438b May 29, 2025 08:34
@AbhishekA1509 AbhishekA1509 merged commit a5a07ce into develop May 29, 2025
6 checks passed
@AbhishekA1509 AbhishekA1509 deleted the fix/app-details-manifest-layout branch May 29, 2025 09:18
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.

3 participants