Skip to content
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.

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

Moved the link dialog partials into view components #2766

Merged
merged 3 commits into from
Mar 5, 2024

Conversation

sascha-karnatz
Copy link
Contributor

@sascha-karnatz sascha-karnatz commented Mar 4, 2024

What is this pull request for?

Try to migrate the link dialog partials into view component to make it easier extendable and lesser repetitive. Add a configuration possibility to add new tabs to the link dialog, without overwriting or defacing.

This an extraction of PR #2756

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

@sascha-karnatz sascha-karnatz marked this pull request as draft March 4, 2024 11:34
@sascha-karnatz sascha-karnatz marked this pull request as ready for review March 4, 2024 12:11
@sascha-karnatz sascha-karnatz requested a review from a team March 4, 2024 12:11
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Very nice. thanks. I have some thoughts about a LinkDialog component

app/views/alchemy/admin/pages/link.html.erb Outdated Show resolved Hide resolved
Try to migrate the link dialog partials into view component to make it easier extendable and lesser repetitive.
Provide a configuration to configure the tabs in the link dialog. It is now possible to add new tabs without overwriting or defacing the Admin UI.
Move the last piece of the link view to a view component and remove the original view.
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Works perfect! Nice work

@tvdeyen tvdeyen added this to the 7.2 milestone Mar 5, 2024
@tvdeyen tvdeyen merged commit 4ba3105 into AlchemyCMS:main Mar 5, 2024
34 checks passed
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