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

UI bugs party #1873

Merged
merged 17 commits into from
Nov 28, 2024
Merged

UI bugs party #1873

merged 17 commits into from
Nov 28, 2024

Conversation

BenElferink
Copy link
Contributor

@BenElferink BenElferink commented Nov 27, 2024

This pull request includes several updates to the frontend web application, focusing on adding new components, refactoring existing ones, and updating styles. The most important changes include the addition of new dropdown components, updates to modals and notifications, and various refactoring tasks to improve code readability and maintainability.

New Components and Features:

  • Added LanguageDropdown component to the dropdowns directory. (frontend/webapp/components/common/dropdowns/language-dropdown/index.tsx)
  • Exported the new LanguageDropdown from the dropdowns index. (frontend/webapp/components/common/dropdowns/index.ts)

Refactoring and Code Simplification:

  • Renamed and refactored build-card-from-action-spec.ts to build-card.ts and updated its usage. (frontend/webapp/containers/main/actions/action-drawer/build-card.ts, frontend/webapp/containers/main/actions/action-drawer/index.tsx) [1] [2]
  • Moved AllDrawers and AllModals components from containers to components directory and updated imports accordingly. (frontend/webapp/components/overview/all-drawers/index.tsx, frontend/webapp/components/overview/all-modals/index.tsx) [1] [2]

Style Updates:

  • Updated background color for HeaderContainer in both main and setup headers to use darker_grey. (frontend/webapp/components/main/header/index.tsx, frontend/webapp/components/setup/header/index.tsx) [1] [2]

Modals and Notifications:

  • Updated CancelWarning and DeleteWarning modal button texts to be more descriptive. (frontend/webapp/components/modals/cancel-warning/index.tsx, frontend/webapp/components/modals/delete-warning/index.tsx) [1] [2]
  • Refactored ToastList component and updated its export method. (frontend/webapp/components/notification/toast-list.tsx, frontend/webapp/components/notification/index.ts) [1] [2]

Utility and Hook Updates:

  • Added BACKEND_BOOLEAN utility to the ErrorDropdown component for better readability and consistency. (frontend/webapp/components/common/dropdowns/error-dropdown/index.tsx) [1] [2]

These changes collectively improve the structure, readability, and functionality of the frontend web application.

@BenElferink BenElferink enabled auto-merge (squash) November 27, 2024 10:44
@BenElferink BenElferink disabled auto-merge November 27, 2024 15:22
@BenElferink BenElferink enabled auto-merge (squash) November 28, 2024 09:31
@BenElferink BenElferink merged commit 1205f01 into main Nov 28, 2024
27 of 28 checks passed
@BenElferink BenElferink deleted the ui-bugs-party branch November 28, 2024 09:43
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