Skip to content

fix: index field wrapper extra div #3749

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

Merged
merged 3 commits into from
Mar 19, 2025
Merged

fix: index field wrapper extra div #3749

merged 3 commits into from
Mar 19, 2025

Conversation

Paul-Bob
Copy link
Contributor

Description

Fixes #3562

Removes an unnecessary <div> from the field wrapper, enabling content alignment with text-right. To maintain the functionality of the copyable feature, the Avo::ClipboardComponent has been converted into an inline-block.

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

@github-actions github-actions bot added the Fix label Mar 18, 2025
Copy link

qlty-cloud-legacy bot commented Mar 18, 2025

Code Climate has analyzed commit c0a2222 and detected 0 issues on this pull request.

View more on Code Climate.

@Paul-Bob Paul-Bob merged commit 2b7a09f into main Mar 19, 2025
20 checks passed
@Paul-Bob Paul-Bob deleted the fix/index_field_wrapper branch March 19, 2025 11:53
Copy link
Contributor

This PR has been merged into main. The functionality will be available in the next release.

Please check the release guide for more information.

@adrianthedev
Copy link
Collaborator

This doesn't impact the copyable option, right?

@Paul-Bob
Copy link
Contributor Author

This doesn't impact the copyable option, right?

No, the Avo::ClipboardComponent has been converted into an inline-block.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Aligning text on index view no longer works
2 participants