8000 :recycle: Refactor modal/hide! function calls by EvaMarco · Pull Request #6415 · penpot/penpot · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

♻️ Refactor modal/hide! function calls #6415

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
May 7, 2025
Merged

♻️ Refactor modal/hide! function calls #6415

merged 1 commit into from
May 7, 2025

Conversation

EvaMarco
Copy link
Contributor
@EvaMarco EvaMarco commented May 7, 2025

Related Ticket

No ticket related

Summary

A little refactor of function calls

Steps to reproduce

Everything works as expected

Checklist

  • Choose the correct target branch; use develop by default.
  • Provide a brief summary of the changes introduced.
  • Add a detailed explanation of how to reproduce the issue and/or verify the fix, if applicable.
  • Include screenshots or videos, if applicable.
  • Add or modify existing integration tests in case of bugs or new features, if applicable.
  • Check CI passes successfully.
  • Update the CHANGES.md file, referencing the related GitHub issue, if applicable.

@EvaMarco EvaMarco force-pushed the eva-refactor-modal-hide branch from e801076 to d744522 Compare May 7, 2025 07:36
@EvaMarco EvaMarco marked this pull request as ready for review May 7, 2025 07:52
@niwinz niwinz merged commit 0f3a4db into develop May 7, 2025
9 checks passed
@niwinz niwinz deleted the eva-refactor-modal-hide branch May 7, 2025 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@EvaMarco @niwinz
0