Skip to content

[fix] Refactors event forms to improve type field handling#207

Merged
otc-zuul[bot] merged 2 commits into
mainfrom
i188
Jul 22, 2025
Merged

[fix] Refactors event forms to improve type field handling#207
otc-zuul[bot] merged 2 commits into
mainfrom
i188

Conversation

@Aloento
Copy link
Copy Markdown
Member

@Aloento Aloento commented Jul 21, 2025

Updates the EventEditor and NewForm components by moving the title input field to a consistent position within the forms.

The title field is now rendered after the type dropdown in the NewForm, enhancing user experience by maintaining a clear structure.

Redundant code for the type dropdown in EventEditor has been removed to streamline the component.

Updates the `EventEditor` and `NewForm` components by moving the title input field to a consistent position within the forms.

The title field is now rendered after the type dropdown in the `NewForm`, enhancing user experience by maintaining a clear structure.

Redundant code for the type dropdown in `EventEditor` has been removed to streamline the component.
@Aloento Aloento self-assigned this Jul 21, 2025
@Aloento Aloento linked an issue Jul 21, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@sgmv sgmv left a comment

Choose a reason for hiding this comment

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

Firefox:
image

Chromium:
image

Please fix the field borders.

Moves the ScaleDropdownSelect for event type to a new position in the form for improved layout and usability.

Ensures that the component remains correctly enabled or disabled based on the event type, maintaining validation and helper text functionality.

Enhances the user experience by streamlining the event input process.
@Aloento Aloento requested a review from sgmv July 21, 2025 14:06
@Aloento Aloento added the gate Merge PR label Jul 22, 2025
@otc-zuul otc-zuul Bot merged commit b72d99e into main Jul 22, 2025
9 checks passed
@otc-zuul otc-zuul Bot deleted the i188 branch July 22, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gate Merge PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move event type on the top in all forms

2 participants