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

Feature/UI/material3 theme #966

Open
wants to merge 86 commits into
base: dev
Choose a base branch
from
Open

Conversation

sofyakurilova
Copy link
Collaborator

No description provided.

@jboddey
Copy link
Collaborator

jboddey commented Jan 2, 2025

Some findings from the branch so far:

  • Settings page: If the selected port is missing (resulting in two callouts at the top), then the save button on the settings page cannot be reached at 100% page zoom.
  • Certificates settings: The page is cut off at the bottom when in 100% zoom.
  • Device modal. Cannot edit a device. Always get the error: 'A device with that MAC address already exists'

Other non-blocking issues:

  • The device edit modal is very blue. No spacing between testing journey type and the icon.

sofyakurilova and others added 24 commits January 2, 2025 10:38
* BE: Changes format for devices/format
FE: Form has no pages; form opens on button click.

Device actions is out of scope.

* Fix tests

* Fix tests
…ancel button when device has no changes; disable delete button for new device
# Conflicts:
#	modules/ui/src/app/pages/testrun/components/testrun-status-card/testrun-status-card.component.ts
#	modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.scss
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.

3 participants