Skip to content

Conversation

@weronikaolejniczak
Copy link
Contributor

@weronikaolejniczak weronikaolejniczak commented Mar 6, 2025

Summary

Closes #8407

This PR fixes the underscore names for Form subcategories. The difference comes from merging the reorganization PR and the guidelines PR.

Before After
Screenshot 2025-03-07 at 11 23 44 Screenshot 2025-03-07 at 11 22 11

I left Form controls > Guidelines page as is. It's a bit awkward because it's the only page there but it doesn't belong to any other section (Text controls / Selection controls etc.). One other idea I had was to add a Guidelines page to every specific "X controls" page but it would lead to a lot of duplicated content. If you have any idea how to improve, I'm open to updating it on this PR!

QA

  • Verify the new page names are correct
  • Verify all links redirecting to these pages work as expected

@weronikaolejniczak weronikaolejniczak requested a review from a team as a code owner March 6, 2025 14:50
@weronikaolejniczak weronikaolejniczak marked this pull request as draft March 6, 2025 14:50
@weronikaolejniczak weronikaolejniczak added the skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) label Mar 6, 2025
@weronikaolejniczak weronikaolejniczak force-pushed the fix/eui-plus-form-subcategories-name branch from e9a5dfa to 6a91017 Compare March 7, 2025 09:50
@weronikaolejniczak weronikaolejniczak marked this pull request as ready for review March 7, 2025 10:28
@kibanamachine
Copy link

Preview staging links for this PR:

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

Copy link
Contributor

@acstll acstll left a comment

Choose a reason for hiding this comment

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

This LGTM as is 🚢

I left Form controls > Guidelines page as is. It's a bit awkward because it's the only page there but it doesn't belong to any other section (Text controls / Selection controls etc.).

it's a bit awkward indeed but I don't think it's that bad… how about removing the form_controls folder, and placing this Guidelines page directly under Forms? 🤔

@weronikaolejniczak
Copy link
Contributor Author

@acstll I thought about that as well but those Guidelines do not describe Forms in general but specifically the Form Controls.

@acstll
Copy link
Contributor

acstll commented Mar 7, 2025

@acstll I thought about that as well but those Guidelines do not describe Forms in general but specifically the Form Controls.

then I think it's fine as it is 🙂

@weronikaolejniczak weronikaolejniczak merged commit c6330a3 into elastic:main Mar 7, 2025
6 checks passed
@weronikaolejniczak weronikaolejniczak deleted the fix/eui-plus-form-subcategories-name branch April 16, 2025 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[EUI+] Fix selection_controls and form_controls category names

4 participants