Skip to content

Add scroll-to-top dispatch after profile update - #472

Open
YuriSouzaDev wants to merge 1 commit into
he4rt:4.xfrom
YuriSouzaDev:fix/correcao-scroll-top-on-save-profile
Open

Add scroll-to-top dispatch after profile update#472
YuriSouzaDev wants to merge 1 commit into
he4rt:4.xfrom
YuriSouzaDev:fix/correcao-scroll-top-on-save-profile

Conversation

@YuriSouzaDev

Copy link
Copy Markdown
Contributor

This pull request introduces a small user experience improvement to the profile save process. After saving the profile, the page will now automatically scroll to the top, ensuring users see any relevant notifications or messages.

  • Added a call to dispatch the scroll-to-top event after saving profile skills in the save method of ProfilePage.php.

@YuriSouzaDev
YuriSouzaDev requested a review from a team August 3, 2026 16:39
@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 4c365374-9256-42d1-bcf1-0aba14e52eb2

📥 Commits

Reviewing files that changed from the base of the PR and between 3682549 and 92849f8.

📒 Files selected for processing (1)
  • app-modules/panel-app/src/Pages/ProfilePage.php

📝 Walkthrough

Walkthrough

ProfilePage::save() now dispatches the scroll-to-top event after synchronizing profile skills and before saving form relationships.

Possibly related PRs

Suggested reviewers: gvieira18

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the added scroll-to-top dispatch after a profile update.
Description check ✅ Passed The description accurately explains the scroll-to-top event and its location in ProfilePage.php.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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.

@stherzada stherzada left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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.

4 participants