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

style(client):DTO property list UX #8343

Merged
merged 8 commits into from May 1, 2024

Conversation

yuval-hazaz
Copy link
Member

@yuval-hazaz yuval-hazaz commented Apr 29, 2024

part of #8306

PR Details

Improve the UX of the DTO form to make the property list more readable

  • Show the property list collapsed by default
  • Add the description of the DTO to the DTO list
  • Decrease the height of the description field to take less space
  • Show the optional description closed by default, even when there is a value
image

IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Copy link

nx-cloud bot commented Apr 29, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit f625f58. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@yuval-hazaz yuval-hazaz merged commit 825d6cb into feat/jovu-api May 1, 2024
8 of 9 checks passed
@yuval-hazaz yuval-hazaz deleted the style/dto-properties-ux branch May 1, 2024 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants