Skip to content

Conversation

AlinaGoaga
Copy link
Contributor

@AlinaGoaga AlinaGoaga commented May 27, 2022

Closes #858, #809

I've also:

  • moved layers at the end of the profile list as discussed so that we don't mess up for alignment for lists that have a mix of profiles with and without layers
  • fixed the breadcrumbs style which was a bit off
  • brought back table filtering

@AlinaGoaga AlinaGoaga added the bug Something isn't working label May 27, 2022
@AlinaGoaga AlinaGoaga changed the title Bug fixes [UI] Profile version gets reset on selection of new profile / Form selection fields are cut off May 27, 2022
@AlinaGoaga AlinaGoaga marked this pull request as ready for review May 27, 2022 12:16
@AlinaGoaga AlinaGoaga requested a review from foot May 27, 2022 12:21
setVersion(profile.values[0].version as string);
setYaml(profile.values[0].version as string);
profile.values[0].selected = true;
const [selectedValue] = profile.values.filter(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

If there's a value already selected we keep that. If not, we set up the version to be the first in the list of available versions

@foot
Copy link
Collaborator

foot commented May 27, 2022

image

Two other small things we could fix here or in a follow up (I can see them on demo-01, not regression here):

  • Bit of space before the logo
  • Double header on details page 🤔

Copy link
Collaborator

@foot foot left a comment

Choose a reason for hiding this comment

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

Works great!

@AlinaGoaga
Copy link
Contributor Author

Coolio ty! Fixed those also

@AlinaGoaga AlinaGoaga merged commit ed993c4 into main May 27, 2022
@AlinaGoaga AlinaGoaga deleted the bug-fixes branch May 27, 2022 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Protocol Labs selection fields are getting cut off
2 participants